Souhaits #1451
fermé
Statut des adhérents et indication dans la colonne "Etat des cotisations".
Ajouté par Roland Telle il y a plus de 4 ans.
Mis à jour il y a plus de 4 ans.
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.
- 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).
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".
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
- Statut changé de Résolu à Fermé
Formats disponibles : Atom
PDF