Galette

Ce matin, voir images ci-jointes
C'est OK pour les membres en retard alors qu'il manque les nouveaux membres en retard.

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.

1. 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(
2. 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'))
3. Et source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1629 par ->greaterThan('a.date_echeance', $now->format('Y-m-d'));
4. Et source:galette/lib/Galette/Repository/Members.php@b7c91f64#L1666 par ->lessThanOrEqualTo('a.date_echeance', $now->format('Y-m-d'));
5. 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 ;)

• 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.

Appliqué par commit e89541901276f14915c3f38da7c650083f3db354.