Project

General

Profile

Actions

Anomalie #1590

closed

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

Added by Frederic CROZET about 3 years ago. Updated about 1 year ago.

Status:
Fermé
Priority:
Bas
Assignee:
-
Category:
-
Target version:
-
Start date:
11/04/2021
Due date:
% Done:

100%

Estimated time:
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.


Files

Galette.png (9.33 KB) Galette.png Vue des rappels Frederic CROZET, 11/04/2021 10:26 AM
Galette2.png (40.3 KB) Galette2.png Liste des adhérents Frederic CROZET, 11/04/2021 10:29 AM
Galette_01.JPG (125 KB) Galette_01.JPG Frederic CROZET, 11/05/2021 04:41 PM
Galette_02.JPG (28.5 KB) Galette_02.JPG Frederic CROZET, 11/05/2021 04:41 PM

Related issues 1 (0 open1 closed)

Related to Galette - Anomalie #1612: Mistakes in reminders' unit testsFerméGuillaume AGNIERAY01/09/2022

Actions

Updated by Frederic CROZET about 3 years ago

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

Actions #2

Updated by Johan Cwiklinski about 3 years ago

  • Category deleted (Fields management)
Actions #3

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.

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

Actions #4

Updated by Guillaume AGNIERAY almost 3 years ago

  • Status changed from In Progress to 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.

Actions #5

Updated by Guillaume AGNIERAY almost 3 years ago

Actions #6

Updated by Guillaume AGNIERAY almost 3 years ago

Actions #7

Updated by Guillaume AGNIERAY almost 3 years ago

Actions #8

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

Actions #9

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

Actions #10

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.

Actions #11

Updated by Guillaume AGNIERAY over 2 years ago

  • Status changed from Commentaire to Résolu
  • % Done changed from 0 to 100
Actions #12

Updated by Johan Cwiklinski about 1 year ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF