Souhaits #1451
closed
Statut des adhérents et indication dans la colonne "Etat des cotisations".
Added by Roland Telle almost 3 years ago.
Updated almost 3 years ago.
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 ?
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).
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/');
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.
- 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).
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".
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
- Status changed from Résolu to Fermé
Also available in: Atom
PDF