Project

General

Profile

Actions

Souhaits #1451

closed

Statut des adhérents et indication dans la colonne "Etat des cotisations".

Added by Roland Telle over 4 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
07/08/2020
Due date:
% Done:

100%

Estimated time:

Description

Le statut Non membre se traduit très logiquement par n'a jamais cotisé.
Celui d'_ancien membre_ est actuellement traduit par En retard ce qui n'est pas très logique puisque la plupart du temps, c'est le fait de ne plus payer la cotisation qui déclenche le statut Ancien membre.

Serait-il possible que le statut Ancien membre déclenche le commentaire Ne cotise plus dans la colonne Etat des cotisations ?

Actions #1

Updated by Johan Cwiklinski over 4 years ago

Se baser sur le statut ne me semble pas une bonne idée ; puisque ce dernier peut être modifié, voire supprimé... En revanche, il est possible de le faire sur le statut du compte de l'adhérent (actif/inactif).

Actions #2

Updated by Johan Cwiklinski over 4 years ago

En gros, ça donnerait quelque chose comme :

diff --git a/galette/lib/Galette/Entity/Adherent.php b/galette/lib/Galette/Entity/Adherent.php
index fe9c2fdb..83d88bcb 100644
--- a/galette/lib/Galette/Entity/Adherent.php
+++ b/galette/lib/Galette/Entity/Adherent.php
@@ -498,7 +498,8 @@ class Adherent
                 if ($this->_days_remaining == 0) {
                     $this->_row_classes .= ' cotis-lastday';
                 } elseif ($this->_days_remaining < 0) {
-                    $this->_row_classes .= ' cotis-late';
+                    //check if member is still active
+                    $this->_row_classes .= $this->isActive() ? ' cotis-late' : ' cotis-old';
                 } elseif ($this->_days_remaining < 30) {
                     $this->_row_classes .= ' cotis-soon';
                 } else {
@@ -737,7 +738,7 @@ class Adherent
                     _T("Late of %days days (since %date)")
                 );
             } else {
-                $ret = _T("Late");
+                $ret = _T("No longer member");
             }
         } else {
             $patterns = array('/%days/', '/%date/');
Actions #3

Updated by Roland Telle over 4 years ago

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

Se baser sur le statut ne me semble pas une bonne idée ; puisque ce dernier peut être modifié, voire supprimé... En revanche, il est possible de le faire sur le statut du compte de l'adhérent (actif/inactif).

Ok. Je pensais bêtement que le statut actif/inactif était un simple drapeau. Tu as raison ; j'ai regardé ta proposition qui me semble convenir si elle tient compte du fait que des inactifs peuvent être à jour de leur cotisation à l'issue de quoi l'intéressé devient 'n'est plus membre' ou 'ne cotise plus' - à la place de 'en retard'.
Merci Johan.

Actions #4

Updated by Johan Cwiklinski over 4 years ago

  • Category set to Core
  • Assignee set to Johan Cwiklinski
  • Target version set to 0.9.4.1

Roland Telle a écrit (#note-3):

Ok. Je pensais bêtement que le statut actif/inactif était un simple drapeau. Tu as raison ; j'ai regardé ta proposition qui me semble convenir si elle tient compte du fait que des inactifs peuvent être à jour de leur cotisation à l'issue de quoi l'intéressé devient 'n'est plus membre' ou 'ne cotise plus' - à la place de 'en retard'.

Oui, tout à fait ; on ne passe à "No longer member" que si l'adhérent n'est pas exempt de cotisation d'une part, et que le nombre de jours restants à son adhésion est inférieur à 0 (en retard donc).

Actions #5

Updated by Roland Telle over 4 years ago

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

En gros, ça donnerait quelque chose comme :

[...]

Ca fonctionne très bien. Les comptes désactivés sont "n'a jamais cotisé" et "no longer member".

Actions #6

Updated by Johan Cwiklinski over 4 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
Actions #7

Updated by Johan Cwiklinski over 4 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF