Anomalie #1590
fermé
Incohérence rappels échéances proches et en retard
100%
Description
Je pensais que ce type d'erreur avait déjà été résolu mais il y a encore une incohérence pour le nombre d'échéances proches et en retard.
Dans la vue "rappel", j'ai 3 adhérents en échéance proche et 8 en retard (voir image ci-jointe).
Alors que dans la liste des membres en échéances proches, il y a seulement 2 adhérents et non 3 et seulement 8 membres dans la vue des membres en retard.
L'adhérent "en retard depuis aujourd'hui" (voir image ci-jointe) n'apparait pas dans les vues filtrés ni en échéances proches, ni en retard. Alors que le code couleur est bien correcte.
Fichiers
Mis à jour par Frederic CROZET il y a plus de 2 ans
- Fichier Galette_01.JPG Galette_01.JPG ajouté
- Fichier Galette_02.JPG Galette_02.JPG ajouté
Ce matin, voir images ci-jointes
C'est OK pour les membres en retard alors qu'il manque les nouveaux membres en retard.
Mis à jour par Johan Cwiklinski il y a plus de 2 ans
- Catégorie
Fields managementsupprimé
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Statut changé de Nouveau à In Progress
Frederic CROZET a écrit :
Je pensais que ce type d'erreur avait déjà été résolu mais il y a encore une incohérence pour le nombre d'échéances proches et en retard.
Oui #1566 mais pas totalement visiblement :)
Dans la vue "rappel", j'ai 3 adhérents en échéance proche et 8 en retard (voir image ci-jointe).
Alors que dans la liste des membres en échéances proches, il y a seulement 2 adhérents et non 3 et seulement 8 membres dans la vue des membres en retard.
L'adhérent "en retard depuis aujourd'hui" (voir image ci-jointe) n'apparait pas dans les vues filtrés ni en échéances proches, ni en retard. Alors que le code couleur est bien correcte.
Ça semble toujours uniquement concerner le cas des échéances atteintes le même jour que le rappel :/
Et logiquement les rappels sont aussi impactés et nécessitent les mêmes ajustements.
- Pour lister les adhérents "en retard depuis aujourd'hui" dans les vues filtrées "en retard", il faudrait normalement remplacer source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1104 par
->lessThanOrEqualTo( - Pour corriger le comptage des rappels dans la vue éponyme : source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1628 par
->lessThanOrEqualTo('a.date_echeance', $soon_date->format('Y-m-d')) - Et source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1629 par
->greaterThan('a.date_echeance', $now->format('Y-m-d')); - Et source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1666 par
->lessThanOrEqualTo('a.date_echeance', $now->format('Y-m-d')); - Pour corriger les rappels effectivement envoyés : source:galette/lib/Galette/Repository/Reminders.php@b7c91f64#L121 par
$select->where->LessThanOrEqualTo(
Il reste à regarder si des ajustements sont nécessaires dans les tests unitaires ;)
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Statut changé de In Progress à Commentaire
A priori, elles sont désormais toutes identiques et leurs résultats cohérents :
- les nombres d'adhérents entre la vue "rappels" et la liste des adhérents filtrée coïncident
- les couleurs correspondent bien aux statuts filtrés (à l'exception des "échéances proches" la veille de la date d'échéance : dans ce cas la ligne est rouge avec le message "Dernier jour!")
Voir : https://framagit.org/galette/galette/-/merge_requests/11/diffs
Rappel important : les rappels sont limités à 2 par état d'échéance, et à des intervalles définis. Donc les rappels comptabilisés dans la vue "rappels" ne correspondent pas à ceux qui seront effectivement envoyés et qui dépendent des quotas et cadences d'envoi autorisés. Ces nombres correspondent aux résultats de la vue filtrée qui est liée à chacun d'eux.
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Lié à Anomalie #1612: Mistakes in reminders' unit tests ajouté
Mis à jour par Guillaume AGNIERAY il y a plus de 2 ans
- Lié à Anomalie #1614: due date reminder emails ajouté
Mis à jour par Guillaume AGNIERAY il y a environ 2 ans
- Lié à Anomalie #1614: due date reminder emails supprimé
Mis à jour par Guillaume AGNIERAY il y a environ 2 ans
J'ai affiné encore un peu plus, et propose une nouvelle MR combinant au passage #1016
Actuellement, la date d'échéance d'un adhérent est la même que la date de fin d'adhésion d'une contribution.
Ce qui est logique.
Mais la date enregistrée, et affichée dans les listes, est en réalité la date de début de la prochaine période d'adhésion.
Je pense que c'est fait ainsi pour faciliter le calcul de la durée d'adhésion d'une contribution lorsque Galette est paramétrée par durée d'adhésion et non pas par date de début d'exercice.
Cependant, ça induit pas mal de confusion (en tout cas pour moi :D) ailleurs dans le code, dans les tests, les nom des variables, le calcul des jours restants ou expirés, etc.
De plus je suis d'accord avec l'intention de #1016 : ça devrait être la véritable date d'échéance (la date ultime après laquelle prend fin une adhésion) qui devrait apparaître dans les listes, les exports, les messages, etc.
Je pense que toutes les modifications proposées doivent avoir des impacts ailleurs (les transactions et les graphiques notamment) bien que les tests ne révèlent aucune erreur à ce sujet ; je ne me suis préoccupé que des rappels à ce stade.
Je me dis aussi que ces modifications nécessiteront inévitablement un update de la base pour soustraire 1 jour de la date de fin de toutes les contributions déjà enregistrées.
En espérant avoir correctement modifié les tests unitaires... : https://framagit.org/galette/galette/-/merge_requests/17
Mis à jour par Frederic CROZET il y a presque 2 ans
Hello,
Je reviens aux nouvelles concernant ce problème car je suis sur un cycle de rappel d'échéance des anciens adhérents et tous ne reçoivent pas de relance.
Le problème a été identifié et je remercie ceux qui sont intervenus pour proposer des solutions.
Je n'ai pas osé mettre en service les modifications sur ma galette en production
Sinon, je patiente en attendant la version 1.0.0 ;)
Mis à jour par Guillaume AGNIERAY il y a presque 2 ans
Frederic CROZET a écrit (#note-9):
Je n'ai pas osé mettre en service les modifications sur ma galette en production
Il ne vaut mieux pas ;)
Les dernières modifs ont pas mal d'incidences ailleurs que sur les rappels ; il reste du travail.
Avant d'aller plus loin, il est préférable d'attendre que Johan valide la direction prise pour tenter de résoudre le problème.
Mis à jour par Guillaume AGNIERAY il y a presque 2 ans
- Statut changé de Commentaire à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit e89541901276f14915c3f38da7c650083f3db354.