Project

General

Profile

Actions

Souhaits #1437

closed

Display sponsors in members list

Added by Manuel Her over 4 years ago. Updated 7 months ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
06/07/2020
Due date:
% Done:

100%

Estimated time:

Description

Bonjour,

Lorsque l'on a une personne au téléphone, il est bien de savoir si elle a adhéré ou donné dans l'année.
Il s'agit donc d'utiliser une couleur et idéalement un texte spécifique lorsque les membres ont fait un don mais pas encore adhéré. (dans la zone, n'a jamais cotisé / en retard ..)

ça se passe dans Adherent::getRowClass()

688,708d687  
<        //S'il y a des dons => on change la couleur
<        try {
<            $select = $this->zdb->select("cotisations");
<            $select->where('id_adh=' . $this->_id);
<            $select->where("YEAR (date_enreg) = " . date("Y"));
<                          
<            $results = $this->zdb->execute($select);
<        } catch (\Exception $e) {
<            Analog::log('An error occured when loading all contributions for #' . $this->_id . "Message:\n" . $e->getMessage(), Analog::ERROR);
<        }
<        
<        if ($this->_due_date == '' && count($results)>0) $this->_row_classes = ' cotis-give';

merci.
Manuel


Files

Actions #1

Updated by Johan Cwiklinski over 4 years ago

  • Target version deleted (0.9.4)
Actions #2

Updated by Johan Cwiklinski over 4 years ago

Le patch proposé ne peut être appliqué :

- ça ne fonctionnera pas sous PostgreSQL,
- il ne faut pas lever une exception (aucune raison pour qu'une erreur là soit bloquante),
- je pense que la requête ne devrait être jouée à tous les coups (probablement conditionné à minima sur $this->_due_date == '')
- ça aurait peut-être plus sa place dans la méthode checkDues

C'est donc très certainement plus compliqué que ce qui a été proposé ici. Il faudrait proposer un patch qui soit plus abouti, ou soumettre la demande au vote.

Actions #3

Updated by Manuel Her over 4 years ago

Bonjour,
Ci-joint un patch avec l'essentiel du code dans checkDues()

j'ai ajouté des commentaires pour une compréhension rapide de ce que j'ai fait.
J'ai changé la couleur dans .cotis-give pour la différencier de .cotis-late.
Compléter le texte retourné par getDues() lorsque la personne à fait un don depuis le début de l'exercice ou les 12 derniers mois

Manuel

Actions #4

Updated by Johan Cwiklinski 7 months ago

  • Subject changed from Une couleur et un texte spécifique lorsque les membres ont fait un don mais pas encore adhéré to Display sponsors in members list
  • Status changed from Nouveau to Fermé
  • Target version set to 1.1.0
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF