Projet

Général

Profil

Actions

Anomalie #1590

fermé

Incohérence rappels échéances proches et en retard

Ajouté par Frederic CROZET il y a environ 3 ans. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
04/11/2021
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

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

Galette.png (9,33 ko) Galette.png Vue des rappels Frederic CROZET, 04/11/2021 10:26
Galette2.png (40,3 ko) Galette2.png Liste des adhérents Frederic CROZET, 04/11/2021 10:29
Galette_01.JPG (125 ko) Galette_01.JPG Frederic CROZET, 05/11/2021 16:41
Galette_02.JPG (28,5 ko) Galette_02.JPG Frederic CROZET, 05/11/2021 16:41

Demandes liées 1 (0 ouverte1 fermée)

Lié à Galette - Anomalie #1612: Mistakes in reminders' unit testsFerméGuillaume AGNIERAY09/01/2022

Actions

Mis à jour par Frederic CROZET il y a environ 3 ans

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 environ 3 ans

  • Catégorie Fields management supprimé

Mis à jour par Guillaume AGNIERAY il y a presque 3 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.

  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 ;)

Mis à jour par Guillaume AGNIERAY il y a presque 3 ans

  • Statut changé de In Progress à Commentaire
Je pense/espère cette fois avoir identifié toutes les requêtes qui posaient problème entre les différentes vues.
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 presque 3 ans

Mis à jour par Guillaume AGNIERAY il y a presque 3 ans

Mis à jour par Guillaume AGNIERAY il y a presque 3 ans

Mis à jour par Guillaume AGNIERAY il y a presque 3 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 plus de 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 plus de 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 plus de 2 ans

  • Statut changé de Commentaire à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF