Anomalie #1590
closedIncohé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.
Files
Updated by Frederic CROZET about 3 years ago
- File Galette_01.JPG Galette_01.JPG added
- File Galette_02.JPG Galette_02.JPG added
Ce matin, voir images ci-jointes
C'est OK pour les membres en retard alors qu'il manque les nouveaux membres en retard.
Updated by Johan Cwiklinski about 3 years ago
- Category deleted (
Fields management)
Updated by Guillaume AGNIERAY almost 3 years ago
- Status changed from Nouveau to 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 ;)
Updated by Guillaume AGNIERAY almost 3 years ago
- Status changed from In Progress to 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.
Updated by Guillaume AGNIERAY almost 3 years ago
- Related to Anomalie #1612: Mistakes in reminders' unit tests added
Updated by Guillaume AGNIERAY almost 3 years ago
- Related to Anomalie #1614: due date reminder emails added
Updated by Guillaume AGNIERAY almost 3 years ago
- Related to deleted (Anomalie #1614: due date reminder emails)
Updated by Guillaume AGNIERAY almost 3 years ago
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
Updated by Frederic CROZET over 2 years ago
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 ;)
Updated by Guillaume AGNIERAY over 2 years ago
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.
Updated by Guillaume AGNIERAY over 2 years ago
- Status changed from Commentaire to Résolu
- % Done changed from 0 to 100
Appliqué par commit e89541901276f14915c3f38da7c650083f3db354.
Updated by Johan Cwiklinski about 1 year ago
- Status changed from Résolu to Fermé