Project

General

Profile

Actions

Anomalie #1566

closed

Différence entre "tous les adhérents" et "échéances proches"

Added by Frederic CROZET almost 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
09/20/2021
Due date:
% Done:

100%

Estimated time:
Version utilisée:

Description

Dans la vue liste des adhérents sans filtre, je retrouve 10 membres en "échéances proches".
Alors qu'avec le filtre "échéances proches", il m'en trouve seulement 9.
Dans la première vue, il compte une échéance proche pour un membre avec 29 jours restants alors qu'il devrait le compter qu'à partir du 30ième jour de retard... Non ??


Files

0001-Fix-close-expiries-closes-1566.patch (2.68 KB) 0001-Fix-close-expiries-closes-1566.patch Guillaume AGNIERAY, 09/20/2021 09:14 PM
avant.jpg (85 KB) avant.jpg Guillaume AGNIERAY, 09/20/2021 09:14 PM
apres.jpg (89.3 KB) apres.jpg Guillaume AGNIERAY, 09/20/2021 09:14 PM
0002-Fix-reminders-refs-1566.patch (1.02 KB) 0002-Fix-reminders-refs-1566.patch Guillaume AGNIERAY, 09/21/2021 02:49 PM

Updated by Guillaume AGNIERAY almost 3 years ago

Pour moi le problème est sur la vue filtrée parmi les échéances proches.
J'ai remarqué également que l'état indiqué au jour J de l'échéance, et celui à J-1, sont les mêmes : "Dernier jour!".
Voici un patch qui devrait corriger le filtre et affiner l'état affiché dans le tableau (voir captures d'écran avant/après).

Actions #2

Updated by Guillaume AGNIERAY almost 3 years ago

  • Category set to Core
  • Status changed from Nouveau to In Progress
Actions #3

Updated by Frederic CROZET almost 3 years ago

Guillaume Agnieray a écrit (#note-1):

Pour moi le problème est sur la vue filtrée parmi les échéances proches.
J'ai remarqué également que l'état indiqué au jour J de l'échéance, et celui à J-1, sont les mêmes : "Dernier jour!".
Voici un patch qui devrait corriger le filtre et affiner l'état affiché dans le tableau (voir captures d'écran avant/après).

Effectivement, c'est bien ça!
Tout fonctionne correctement désormais
Frédéric

Actions #4

Updated by Guillaume AGNIERAY almost 3 years ago

Je me demande si ça ne résout pas la demande #1016.

Et il doit y avoir un lien avec le problème de rappel automatique signalé sur la liste de discussion.
Ça se passe ici : source:/galette/lib/Galette/Repository/Reminders.php#L125

Actions #5

Updated by Guillaume AGNIERAY almost 3 years ago

Après relecture du thread sur la liste de discussion, je pense bien que les 2 problèmes sont liés.
La requête filtrant les membres avec une échéance proche et celle déterminant les rappels correspondants sont les mêmes.
Actuellement, lorsque des rappels sont enregistrés en base, le nombre est correct, mais donc, a priori, pas la liste des rappels.
Dans tous les cas, logiquement il faut que la liste des rappels et une recherche des échéances proches renvoient les mêmes résultats.
Voici donc un second patch qui devrait espérons le corriger la question des rappels automatiques.

Actions #6

Updated by Guillaume AGNIERAY almost 3 years ago

  • Status changed from In Progress to Commentaire
Actions #7

Updated by Johan Cwiklinski almost 3 years ago

  • Assignee set to Guillaume AGNIERAY
  • Target version set to 0.9.5.1
Actions #8

Updated by Guillaume AGNIERAY almost 3 years ago

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

Updated by Johan Cwiklinski almost 3 years ago

J'ai appliqué les correctifs proposés, qui me semblent corrects.

Les tests unitaires des rappels ont donc échoué, je les ai corrigés comme suit : https://github.com/galette/galette/commit/ceda0abdf1400923a8731e2dee7d800f2cbc2d7b

Concernant #1016, je ne pense pas, la date de fin d'adhésion en elle-même n'est pas modifiée, et l'adhérent est toujours considéré "à jour" (confer les tests justement ;))). Après, ce n'est peut-être pas correct (je doute qu'il y ait un réel consensus en fait) - mais c'est un autre sujet :)

Actions #10

Updated by Guillaume AGNIERAY almost 3 years ago

Johan Cwiklinski a écrit (#note-9):

Les tests unitaires des rappels ont donc échoué, je les ai corrigés comme suit : https://github.com/galette/galette/commit/ceda0abdf1400923a8731e2dee7d800f2cbc2d7b

Merci. J'y prêterai attention à l'avenir.

Concernant #1016, je ne pense pas, la date de fin d'adhésion en elle-même n'est pas modifiée, et l'adhérent est toujours considéré "à jour" (confer les tests justement ;))). Après, ce n'est peut-être pas correct (je doute qu'il y ait un réel consensus en fait) - mais c'est un autre sujet :)

D'accord. J'ai mal compris. Cette demande questionne en réalité le choix de la date d'échéance.
Je ne relance pas le sujet dans ce cas.

Actions #11

Updated by Johan Cwiklinski almost 3 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF