Exemption : filtre dans la liste des membres #709
Conversation
| 'required' => false, | ||
| 'choices' => [ | ||
| 'exempté' => "exempted", | ||
| // 'Non exempté' => "not_exempted", |
There was a problem hiding this comment.
C'est normal que ça soit commenté ? Pour les autres filtre équivalent, il y a toujours la version positive et négative. C'est peut-être mieux d'uniformiser ?
There was a problem hiding this comment.
J'ai essayé de l'implémenter sans succès (cf description dans la PR)
There was a problem hiding this comment.
Car "non exempté" = "0 exemption" OU "X exemptions dont aucune isCurrent"
There was a problem hiding this comment.
Car "non exempté" = "0 exemption" OU "X exemptions dont aucune isCurrent"
Pour moi c'est la 2ème, aucune exemption valide actuellement
There was a problem hiding this comment.
Pour moi c'est les 2 ^^ (le OU/OR est à retranscrire en SQL)
There was a problem hiding this comment.
Je viens de faire une proposition de modification dans le dernier commit. Je te laisse regarder.
There was a problem hiding this comment.
ok niquel merci ! c'était plus simple que j'imaginais ^^
(j'ai rajouté un mini commit pour cleanup)
| @@ -376,6 +386,10 @@ public function processSearchFormData($form,&$qb) { | |||
| $qb = $qb->andWhere('o.frozen = :frozen') | |||
| ->setParameter('frozen', $form->get('frozen')->getData()-1); | |||
| } | |||
| if ($form->get('exempted')->getData() == "exempted") { | |||
There was a problem hiding this comment.
pourquoi ne pas utiliser un boolean comme les autres filtres ?
There was a problem hiding this comment.
Les autres filtres utilisent des booléen car ce sont des champs booléen.
Et plus généralement je trouve bizarre de mettre comme choices des 1 et des 2 puis faire -1 par magie...
D'ailleurs si on passe un jour cette page de filtre avec la méthode GET, les paramètres s'afficheront dans l'URL, je préfère du texte explicite que des numéros ^^
There was a problem hiding this comment.
Je viens de faire une proposition de modification dans le dernier commit. Je te laisse regarder.
There was a problem hiding this comment.
ok why not, mais ca ne me parait toujours pas logique de manipuler des 2 et des 1 🤷
There was a problem hiding this comment.
Yep je suis d'accord avec ta remarque, faudrait changer partout. La au moins c'est homogène :)
* Exemption: add filter in Member list * Implement not exempted filter Co-authored-by: Albin PETIT <albin.petit@gmail.com>
Quoi ?
Dans la page admin "Liste des membres", pouvoir filtrer par membre exempté
(comment filtrer sur les membres qui ne sont pas exemptés ? plus compliqué, à faire dans une future PR)
Capture d'écran