Project

General

Profile

Anomalie #1566

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

Added by Frederic CROZET 2 months ago. Updated 2 months ago.

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

100%

Estimated time:
Version utilisée:
Vote:

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
#1

Updated by Guillaume Agnieray 2 months 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).

#2

Updated by Guillaume Agnieray 2 months ago

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

Updated by Frederic CROZET 2 months 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

#4

Updated by Guillaume Agnieray 2 months 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

#5

Updated by Guillaume Agnieray 2 months 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.

#6

Updated by Guillaume Agnieray 2 months ago

  • Status changed from In Progress to Commentaire
#7

Updated by Johan Cwiklinski 2 months ago

  • Assignee set to Guillaume Agnieray
  • Target version set to 0.9.5.1
#8

Updated by Guillaume Agnieray 2 months ago

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

Updated by Johan Cwiklinski 2 months 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 :)

#10

Updated by Guillaume Agnieray 2 months 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.

#11

Updated by Johan Cwiklinski 2 months ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF