Project

General

Profile

Actions

Evolution #1438

closed

Add monthly membership

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

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

100%

Estimated time:

Description

Bonjour,

Afin de fidéliser les adhérents, de plus en plus d'associations proposent la mensualisation.
Paypal propose ce service que j'ai utilisé/implémenté en dehors de Galette. J'utilise le plugin Paypal pour les callbacks IPN sans aucune modification (L'accès au formulaire classique est commenté dans le source).

En attendant que Galette propose cette fonctionnalité en interne, serait-il possible de poser la base de ce fonctionnement ?
L'essentiel est de pouvoir configurer des durées différentes pour une contribution (1, 2, 6, 12 mois, etc...)

Mon association souhaite pouvoir profiter des évolutions de galette sans devoir faire des diffs / patchs à chaque nouvelle version.

Je propose d'ajouter une durée d'extension à la configuration des types de contribution, qui se substitue aux paramètres des préférences pour le calcul de la fin d'adhésion.

Dans la table des types de contribution, on utiliserait 'cotis_extension' de cette façon :

-1: durée d'adhésion par défaut (config générale en année civile ou sur 12 mois)

0: pas de durée(c'est un Don)

1=>1 mois; 2=>2mois etc...

Qui peut le plus peut le moins. En stockant en entier il suffit de faire == 0 ou != 0 pour savoir si c'est un don ou non.

Le champ cotis_extension est très peu utilisé dans le source.

ça se résume à modifier 3 conditions ; par exemple :

->where(array('cotis_extension' => new Expression('true')))
par   - >where->notEqualTo('cotis_extension', 0)

cotis_extension changerait de tinyint(1) à tinyint(4) ;

Au moment de la mise à jour, il faudrait exécuter :

ALTER TABLE `galette_types_cotisation` CHANGE `cotis_extension` `cotis_extension` TINYINT(4) NOT NULL DEFAULT '0'; 
UPDATE `galette_types_cotisation` SET `cotis_extension`=-1 WHERE `cotis_extension`=1

Voici le code validé sur mon serveur de test :


diff -r -x '*.log' -x '*.rej' -x 'composer.*' '--exclude=data' '--exclude=plugins' '--exclude=vendor' galette/galette/lib/Galette/Entity/Contribution.php galette0941_avantmodif/galette/lib/Galette/Entity/Contribution.php
213a214,229
>         if ($preferences->pref_beg_membership != '') {
>             //case beginning of membership
>             list($j, $m) = explode('/', $preferences->pref_beg_membership);
>             $edate = new \DateTime($bdate->format('Y') . '-' . $m . '-' . $j);
>             while ($edate <= $bdate) {
>                 $edate->modify('+1 year');
>             }
> 
>             if ($preferences->pref_membership_offermonths > 0) {
>                 //count days until end of membership date
>                 $diff1 = (int)$bdate->diff($edate)->format('%a');
> 
>                 //count days beetween end of membership date and offered months
>                 $tdate = clone $edate;
>                 $tdate->modify('-' . $preferences->pref_membership_offermonths . ' month');
>                 $diff2 = (int)$edate->diff($tdate)->format('%a');
215,221c231,232
<         if ($preferences->pref_beg_membership != '') { //adhésion classique de date à date + 1 an 
<             if ($this->_type->extension==-1) //C'est une adhésion classique en année
<             {
<                 //case beginning of membership
<                 list($j, $m) = explode('/', $preferences->pref_beg_membership);
<                 $edate = new \DateTime($bdate->format('Y') . '-' . $m . '-' . $j);
<                 while ($edate <= $bdate) {
---
>                 //when number of days until end of membership is less than for offered months, it's free :)
>                 if ($diff1 <= $diff2) {
224,240d234
<     
<                 if ($preferences->pref_membership_offermonths > 0) {
<                     //count days until end of membership date
<                     $diff1 = (int)$bdate->diff($edate)->format('%a');
<     
<                     //count days beetween end of membership date and offered months
<                     $tdate = clone $edate;
<                     $tdate->modify('-' . $preferences->pref_membership_offermonths . ' month');
<                     $diff2 = (int)$edate->diff($tdate)->format('%a');
<     
<                     //when number of days until end of membership is less than for offered months, it's free :)
<                     if ($diff1 <= $diff2) {
<                         $edate->modify('+1 year');
<                     }
<                 }
<     
<                 $this->_end_date = $edate->format('Y-m-d');
242,248c236,238
<             else 
<             {
<                 $dext = new \DateInterval('P' . $this->_type->extension . 'M');
<                 $edate = $bdate->add($dext);
<                 $this->_end_date = $edate->format('Y-m-d');
<             }
<         } elseif ($preferences->pref_membership_ext != '') { //adhésion classique de date à date + N mois 
---
> 
>             $this->_end_date = $edate->format('Y-m-d');
>         } elseif ($preferences->pref_membership_ext != '') {
251,254c241
<                 if ($this->_type->extension==-1)
<                     $this->_extension = $preferences->pref_membership_ext;
<                 else
<                     $this->_extension = $this->_type->extension;
---
>                 $this->_extension = $preferences->pref_membership_ext;
534,535c521
<                 //->where(array('cotis_extension' => new Expression('true')))
<                 ->where->notEqualTo('cotis_extension', 0)
---
>                 ->where(array('cotis_extension' => new Expression('true')))
847,849c833
<             )->where->notEqualTo('cotis_extension', 0);
<             
<             /*->where(
---
>             )->where(
851c835
<             );*/
---
>             );
1255c1239
<                         if ($this->$rname->extension != 0) {//== 1) {
---
>                         if ($this->$rname->extension == 1) {
diff -r -x '*.log' -x '*.rej' -x 'composer.*' '--exclude=data' '--exclude=plugins' '--exclude=vendor' galette/galette/lib/Galette/Entity/ContributionsTypes.php galette0941_avantmodif/galette/lib/Galette/Entity/ContributionsTypes.php
130,131d129
<     
< 
diff -r -x '*.log' -x '*.rej' -x 'composer.*' '--exclude=data' '--exclude=plugins' '--exclude=vendor' galette/galette/lib/Galette/Entity/Entitled.php galette0941_avantmodif/galette/lib/Galette/Entity/Entitled.php
253,254c253
<                     //$select->where(array($this->fthird => new Expression('true')));
<                     $select->where->notEqualTo($this->fthird, 0);
---
>                     $select->where(array($this->fthird => new Expression('true')));
256,257c255
<                     //$select->where(array($this->fthird => new Expression('false')));
<                     $select->where->equalTo($this->fthird, 0);
---
>                     $select->where(array($this->fthird => new Expression('false')));
diff -r -x '*.log' -x '*.rej' -x 'composer.*' '--exclude=data' '--exclude=plugins' '--exclude=vendor' galette/galette/templates/default/editer_intitule.tpl galette0941_avantmodif/galette/templates/default/editer_intitule.tpl
28,38c28
<             
<                     <select name="{$field}">
<                         <option value="0" {if $entry->$field == 0}selected="selected"{/if}>{_T string="No"}</option>
<                         <option value="-1" {if $entry->$field == -1}selected="selected"{/if}>{_T string="By default"}</option>                        
<                         {for $m=12 to 2 step -1}
<                             <option value="{$m}" {if $entry->$field == $m}selected="selected"{/if} >{$m} {_T string="months"}</option>
<                         {/for}
<                         <option value="1" {if $entry->$field == 1}selected="selected"{/if} >1 {_T string="month"}</option>
<                     </select>
<                     
<                                 
---
>             <input type="checkbox" name="{$field}" id="{$field}" value="1"{if $entry->$field == 1} checked="checked"{/if} />
diff -r -x '*.log' -x '*.rej' -x 'composer.*' '--exclude=data' '--exclude=plugins' '--exclude=vendor' galette/galette/templates/default/gestion_intitule_content.tpl galette0941_avantmodif/galette/templates/default/gestion_intitule_content.tpl
36d35
< 
38,44c37,38
<                         <option value="-1">{_T string="By default"}</option>                        
<                         {for $m=12 to 2 step -1}
<                             <option value="{$m}">{$m} {_T string="months"}</option>
<                         {/for}
<                         <option value="1">1 {_T string="month"}</option>
<                            
<                     </select> 
---
>                         <option value="1">{_T string="Yes"}</option>
>                     </select>
83,88c77,78
<                     {if $entry.extra eq 0}
<                         {_T string="No"}
<                     {elseif $entry.extra eq -1}
<                         {_T string="By default"}
<                     {elseif $entry.extra eq 1}
<                         {$entry.extra} {_T string="month"}
---
>                     {if $entry.extra eq 1}
>                         {_T string="Yes"}
90c80
<                         {$entry.extra} {_T string="months"}
---
>                         {_T string="No"}

Manuel


Files

diff_2.txt (7.33 KB) diff_2.txt diff Manuel Her, 06/08/2020 09:20 AM
0001-Adhesion-en-mois-mensualisation-1438.patch (10.2 KB) 0001-Adhesion-en-mois-mensualisation-1438.patch Manuel Her, 08/17/2020 06:46 PM
modif_adhmensuelle.txt (15.2 KB) modif_adhmensuelle.txt Manuel Her, 02/23/2024 08:33 AM

Related issues 2 (0 open2 closed)

Related to Galette - Evolution #495: Contribution: durée d'inscriptionFermé01/23/2013

Actions
Precedes Galette - Evolution #1824: Handle reminders for monthly contributionsRejeté06/09/202006/09/2020

Actions
Actions #1

Updated by Johan Cwiklinski about 4 years ago

Je ne parviens pas à appliquer le patch fourni, je me prend systématiquement l'erreur :
patch: **** Only garbage was found in the patch input..

Bref, il serait bien plus simple de fourni un patch GIT ; je n'ai pas trop envie de répercuter les modifications une par une :)

Actions #2

Updated by Johan Cwiklinski about 4 years ago

Actions #3

Updated by Manuel Her about 4 years ago

Bonjour,

J'ai fait un patch GIT (ci-joint)

Il faut executer

ALTER TABLE `galette_types_cotisation` CHANGE `cotis_extension` `cotis_extension` TINYINT(4) NOT NULL DEFAULT '0'; 
UPDATE `galette_types_cotisation` SET `cotis_extension`=-1 WHERE `cotis_extension`=1

Manuel

Actions #4

Updated by Manuel Her 8 months ago

Bonjour,
Ci-joint un git diff pour la version 1.1.

cotis_extension=-1 : adhésion par défaut de galette
cotis_extension=0, c'est un don
cotis_extension>0, nombre de mois

Il faut executer :

ALTER TABLE `galette_types_cotisation` CHANGE `cotis_extension` `cotis_extension` TINYINT NOT NULL DEFAULT '0';
UPDATE `galette_types_cotisation` SET `cotis_extension`=-1 WHERE `cotis_extension`=1

Manuel

Actions #5

Updated by Johan Cwiklinski 8 months ago

  • Category set to Core
  • Status changed from Nouveau to In Progress
  • Assignee set to Manuel Her

Le plus simple serait sûrement d'ouvrir une PR sur le dépôt github, non ?

Pour le coup, j'avais complètement oublié qu'il y avait un patch fourni ; je n'ai donc jamais testé...

Sur la branche develop, des scripts de migration pour la 1.1.0 ont été ajoutés, les instructions SQL peuvent y être placées (pour mysql et postgres). Il faudrait également prévoir des tests unitaires, en plus de corriger cuex qui existent (je suppose que certains vont échouer).

Actions #6

Updated by Johan Cwiklinski 5 months ago

  • Subject changed from Mensualisation - Fidéliser les adhérents to Add monthly membership
  • Target version set to 1.1.0
Actions #7

Updated by Johan Cwiklinski 5 months ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
Actions #8

Updated by Johan Cwiklinski 5 months ago

  • Precedes Evolution #1824: Handle reminders for monthly contributions added
Actions #9

Updated by Johan Cwiklinski 5 months ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF