Souhaits #716
ferméSupprimer le décompte (Etat cotisations) des comptes désactivés.
100%
Description
Les comptes désactivés sont pour la plupart ceux d'anciens membres qui n'ont pas renouvelé depuis quelque temps leur adhésion et qui sont classés là plutôt que de les détruire.
De ce fait, le décompte qui apparaît dans la colonne Etat cotisations est anachronique surtout quand il affiche le nombre de jours de retard sur un compte ancien (information inutile). De plus, sur petit écran, ce champ prend (inutilement) trois lignes.
Peut-on envisager de remplacer l'affichage du décompte par une expression simple genre "Ne cotise plus" (j'imagine mal un adhérent qui cotise dont la fiche serait classée dans les comptes désactivés) ?
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Est-ce que quelque chose comme ça conviendrait :
diff --git a/galette/lib/Galette/Entity/Adherent.php b/galette/lib/Galette/Entity/Adherent.php
index b1178cb..bca52e8 100644
--- a/galette/lib/Galette/Entity/Adherent.php
+++ b/galette/lib/Galette/Entity/Adherent.php
@@ -599,7 +599,7 @@ class Adherent
{
$ret = '';
if ( $this->isDueFree() ) {
- $ret = _T("Freed of dues");
+ $ret = _T("Freed of dues");
} else if ( $this->_due_date == '') {
$patterns = array('/%days/', '/%date/');
$cdate = new \DateTime($this->_creation_date);
@@ -607,11 +607,15 @@ class Adherent
$this->_oldness,
$cdate->format(_T("Y-m-d"))
);
- $ret = preg_replace(
- $patterns,
- $replace,
- _T("Never contributed: Registered %days days ago (since %date)")
- );
+ if ( $this->_active ) {
+ $ret = preg_replace(
+ $patterns,
+ $replace,
+ _T("Never contributed: Registered %days days ago (since %date)")
+ );
+ } else {
+ $ret = _T("Never contributed");
+ }
} else if ( $this->_days_remaining == 0 ) {
$ret = _T("Last day!");
} else if ( $this->_days_remaining < 0 ) {
@@ -621,11 +625,15 @@ class Adherent
$this->_days_remaining *-1,
$ddate->format(_T("Y-m-d"))
);
- $ret = preg_replace(
- $patterns,
- $replace,
- _T("Late of %days days (since %date)")
- );
+ if ( $this->_active ) {
+ $ret = preg_replace(
+ $patterns,
+ $replace,
+ _T("Late of %days days (since %date)")
+ );
+ } else {
+ $ret = _T("Late");
+ }
} else {
$patterns = array('/%days/', '/%date/');
$ddate = new \DateTime($this->_due_date);
En gros, si l'adhérent est en retard, ou n'a jamais contribué, on affiche juste « En retard » ou « N'a jamais contribué » sans le détail. S'il est à jour de cotisation, rien ne change.
Mis à jour par Roland Telle il y a environ 11 ans
Johan Cwiklinski a écrit :
Est-ce que quelque chose comme ça conviendrait :
[...] En gros, si l'adhérent est en retard, ou n'a jamais contribué, on affiche juste « En retard » ou « N'a jamais contribué » sans le détail. S'il est à jour de cotisation, rien ne change.
Le décompte (inutile pour ceux dont les comptes sont désactivés) a en effet disparu, le but est atteint (éviter l'affichage de milliers de jours...). Peut-être pourrait-on mettre un texte genre "Voir contributions" qui renverrait directement dans les contributions payées antérieurement ? [Mais c'est pour faire joli :-)]
Merci
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Assigné à mis à Johan Cwiklinski
- Version cible mis à 0.7.6
- Catégorie mis à Core
Roland Telle a écrit :
Johan Cwiklinski a écrit :
Est-ce que quelque chose comme ça conviendrait :
[...] En gros, si l'adhérent est en retard, ou n'a jamais contribué, on affiche juste « En retard » ou « N'a jamais contribué » sans le détail. S'il est à jour de cotisation, rien ne change.Le décompte (inutile pour ceux dont les comptes sont désactivés) a en effet disparu, le but est atteint (éviter l'affichage de milliers de jours...). Peut-être pourrait-on mettre un texte genre "Voir contributions" qui renverrait directement dans les contributions payées antérieurement ? [Mais c'est pour faire joli :-)]
Merci d'avoir testé :)
Pour le lien, il existe déjà en fin de ligne ; ce serait donc un peu redondant (et plus « compliqué » à mettre en oeuvre), je vote contre ;)
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 226ea59961b8e6ae7cac6c3f2eb7cc2f383624cd.
Mis à jour par Roland Telle il y a environ 11 ans
Johan Cwiklinski a écrit :
Pour le lien, il existe déjà en fin de ligne ; [...]
Comme quoi, il faut lire jusqu'au bout ! ;-)
je vote contre ;)
Moi aussi !