Evolution #1438
ferméAdd monthly membership
100%
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
Fichiers
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
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 :)
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
- Lié à Evolution #495: Contribution: durée d'inscription ajouté
Mis à jour par Manuel Her il y a plus de 4 ans
- Fichier 0001-Adhesion-en-mois-mensualisation-1438.patch 0001-Adhesion-en-mois-mensualisation-1438.patch ajouté
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
Mis à jour par Manuel Her il y a 9 mois
- Fichier modif_adhmensuelle.txt modif_adhmensuelle.txt ajouté
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
Mis à jour par Johan Cwiklinski il y a 9 mois
- Catégorie mis à Core
- Statut changé de Nouveau à In Progress
- Assigné à mis à 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).
Mis à jour par Johan Cwiklinski il y a 7 mois
- Sujet changé de Mensualisation - Fidéliser les adhérents à Add monthly membership
- Version cible mis à 1.1.0
Mis à jour par Johan Cwiklinski il y a 7 mois
- Statut changé de In Progress à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 58d36eb3ed02ef2a706c2c99b3cf19be47f4a541.
Mis à jour par Johan Cwiklinski il y a 7 mois
- Précède Evolution #1824: Handle reminders for monthly contributions ajouté