Project

General

Profile

Souhaits #716

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

Added by Roland Telle about 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
10/02/2013
Due date:
% Done:

100%

Estimated time:
Vote:

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

Associated revisions

Revision 226ea599 (diff)
Added by Johan Cwiklinski about 6 years ago

Hide days for inactive late members; fixes #716

History

#1

Updated by Johan Cwiklinski about 6 years ago

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.

#2

Updated by Roland Telle about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • Assignee set to Johan Cwiklinski
  • Target version set to 0.7.6
  • Category set to 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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#5

Updated by Roland Telle about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF