Projet

Général

Profil

Souhaits #716

Supprimer le décompte (Etat cotisations) des comptes désactivés.

Ajouté par Roland Telle il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
Début:
02/10/2013
Echéance:
% réalisé:

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) ?

Révisions associées

Révision 226ea599
Ajouté par Johan Cwiklinski il y a presque 4 ans

Hide days for inactive late members; fixes #716

Historique

#1 Mis à jour par Johan Cwiklinski il y a presque 4 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");
         }
             $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)")
+                );
+            }
+                $ret = _T("Never contributed");
+            }
         }
             $ret = _T("Last day!");
         }
@@ -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)")
+                );
+            }
+                $ret = _T("Late");
+            }
         }
             $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.

#2 Mis à jour par Roland Telle il y a presque 4 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

#3 Mis à jour par Johan Cwiklinski il y a presque 4 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 ;)

#4 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#5 Mis à jour par Roland Telle il y a presque 4 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 !

#6 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF