Anomalie #1566
fermé
Différence entre "tous les adhérents" et "échéances proches"
Ajouté par Frederic CROZET il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.
100%
Description
Dans la vue liste des adhérents sans filtre, je retrouve 10 membres en "échéances proches".
Alors qu'avec le filtre "échéances proches", il m'en trouve seulement 9.
Dans la première vue, il compte une échéance proche pour un membre avec 29 jours restants alors qu'il devrait le compter qu'à partir du 30ième jour de retard... Non ??
Fichiers
| 0001-Fix-close-expiries-closes-1566.patch (2,68 ko) 0001-Fix-close-expiries-closes-1566.patch | Guillaume AGNIERAY, 20/09/2021 21:14 | ||
| avant.jpg (85 ko) avant.jpg | Guillaume AGNIERAY, 20/09/2021 21:14 | ||
| apres.jpg (89,3 ko) apres.jpg | Guillaume AGNIERAY, 20/09/2021 21:14 | ||
| 0002-Fix-reminders-refs-1566.patch (1,02 ko) 0002-Fix-reminders-refs-1566.patch | Guillaume AGNIERAY, 21/09/2021 14:49 |
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Fichier 0001-Fix-close-expiries-closes-1566.patch 0001-Fix-close-expiries-closes-1566.patch ajouté
- Fichier avant.jpg avant.jpg ajouté
- Fichier apres.jpg apres.jpg ajouté
Pour moi le problème est sur la vue filtrée parmi les échéances proches.
J'ai remarqué également que l'état indiqué au jour J de l'échéance, et celui à J-1, sont les mêmes : "Dernier jour!".
Voici un patch qui devrait corriger le filtre et affiner l'état affiché dans le tableau (voir captures d'écran avant/après).
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Catégorie mis à Core
- Statut changé de Nouveau à In Progress
Mis à jour par Frederic CROZET il y a plus de 2 ans
Guillaume Agnieray a écrit (#note-1):
Pour moi le problème est sur la vue filtrée parmi les échéances proches.
J'ai remarqué également que l'état indiqué au jour J de l'échéance, et celui à J-1, sont les mêmes : "Dernier jour!".
Voici un patch qui devrait corriger le filtre et affiner l'état affiché dans le tableau (voir captures d'écran avant/après).
Effectivement, c'est bien ça!
Tout fonctionne correctement désormais
Frédéric
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
Je me demande si ça ne résout pas la demande #1016.
Et il doit y avoir un lien avec le problème de rappel automatique signalé sur la liste de discussion.
Ça se passe ici : source:/galette/lib/Galette/Repository/Reminders.php#L125
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
Après relecture du thread sur la liste de discussion, je pense bien que les 2 problèmes sont liés.
La requête filtrant les membres avec une échéance proche et celle déterminant les rappels correspondants sont les mêmes.
Actuellement, lorsque des rappels sont enregistrés en base, le nombre est correct, mais donc, a priori, pas la liste des rappels.
Dans tous les cas, logiquement il faut que la liste des rappels et une recherche des échéances proches renvoient les mêmes résultats.
Voici donc un second patch qui devrait espérons le corriger la question des rappels automatiques.
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Statut changé de In Progress à Commentaire
Mis à jour par Johan Cwiklinski il y a plus de 2 ans
- Assigné à mis à Guillaume AGNIERAY
- Version cible mis à 0.9.5.1
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Statut changé de Commentaire à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 622c809ff70e3a8292bd9647251429272d9b4eba.
Mis à jour par Johan Cwiklinski il y a plus de 2 ans
J'ai appliqué les correctifs proposés, qui me semblent corrects.
Les tests unitaires des rappels ont donc échoué, je les ai corrigés comme suit : https://github.com/galette/galette/commit/ceda0abdf1400923a8731e2dee7d800f2cbc2d7b
Concernant #1016, je ne pense pas, la date de fin d'adhésion en elle-même n'est pas modifiée, et l'adhérent est toujours considéré "à jour" (confer les tests justement ;))). Après, ce n'est peut-être pas correct (je doute qu'il y ait un réel consensus en fait) - mais c'est un autre sujet :)
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
Johan Cwiklinski a écrit (#note-9):
Les tests unitaires des rappels ont donc échoué, je les ai corrigés comme suit : https://github.com/galette/galette/commit/ceda0abdf1400923a8731e2dee7d800f2cbc2d7b
Merci. J'y prêterai attention à l'avenir.
Concernant #1016, je ne pense pas, la date de fin d'adhésion en elle-même n'est pas modifiée, et l'adhérent est toujours considéré "à jour" (confer les tests justement ;))). Après, ce n'est peut-être pas correct (je doute qu'il y ait un réel consensus en fait) - mais c'est un autre sujet :)
D'accord. J'ai mal compris. Cette demande questionne en réalité le choix de la date d'échéance.
Je ne relance pas le sujet dans ce cas.