Souhaits #1451
fermé
Statut des adhérents et indication dans la colonne "Etat des cotisations".
100%
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 ?
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
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).
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
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/');
Mis à jour par Roland Telle il y a plus de 4 ans
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.
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
- Catégorie mis à Core
- Assigné à mis à Johan Cwiklinski
- Version cible mis à 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).
Mis à jour par Roland Telle il y a plus de 4 ans
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".
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit efda035c4378d2262f9dc781a3dfe5a88ace5fdb.