Project

General

Profile

Evolution #1407

Adhésion en année civile avec cadeau pour les 2 derniers mois

Added by Manuel Her 8 months ago. Updated 6 months ago.

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

100%

Estimated time:

Description

Bonjour,

Je souhaiterais que les adhésions faites à partir du 1er octobre soient effectives jusqu'au 31/12 de l'année suivante.

pref_beg_membership doit être configuré au 31/12 ou au 01/01?

Merci
Manuel


Related issues

Has duplicate Galette - Souhaits #913: Offrir les 3 derniers mois de l'annéeRejeté03/30/2015

Actions
#1

Updated by Johan Cwiklinski 8 months ago

  • Is duplicate of Souhaits #913: Offrir les 3 derniers mois de l'année added
#2

Updated by Johan Cwiklinski 8 months ago

  • Status changed from Nouveau to Rejeté

Aucune idée pour la date, ça dépend de ce que vous voulez... Le mieux serait sûrement de poser ce genre de question sur la liste, que d'autres puissent aider au besoin.

Je souhaiterais que les adhésions faites à partir du 1er octobre soient effectives jusqu'au 31/12 de l'année suivante.

Galette ne permet pas ça de manière automatique ; la date proposée est la bonne, à l'exception de l'année. La demande a déjà été faite (https://bugs.galette.eu/issues/913) mais n'a pas intéressé grand monde. J'ai crée un vote pour la demande.

#3

Updated by Manuel Her 8 months ago

Bonjour,

Voici comment je l'ai implémenté.
Le format de pref_beg_membership devient 'dd/mm;mois à offrir' et fonctionne aussi avec dd/mm.
Je l'ai testé en changeant la date de mon serveur de test.
Dans mon cas, j'ai configuré 01/01;2
Ca va faire gagner pas mal de temps à notre secrétaire qui doit modifier les dates des contributions paypal en fin d'année.

preferences.tpl ; line 192 :

                  
<input type="text" name="pref_beg_membership" id="pref_beg_membership" value="{$pref.pref_beg_membership}" maxlength="7"(au lieu de 5)
{if isset($required.pref_beg_membership) and $required.pref_beg_membership eq 1} required="required"{/if}/>
                    <span class="exemple">{_T string="(dd/mm) or (dd/mm;freemonth)"}</span>

Preferences.php; line 462 :


 case 'pref_beg_membership':
                        //$beg_membership = explode("/", $value);
                        //format is dd/mm;freemonth 
                        if (preg_match_all('#(\d{1,2})/(\d{1,2});?(\d)?$#mius', $value, $beg_membership, PREG_SET_ORDER))
                        {
                            $beg_membership = $beg_membership[0]; //je ne maitrise pas bien preg; il y peut être une façon plus propre de faire ça :)
                            array_shift($beg_membership); //supprimer le "catch" global
                        }
                         //   $this->errors[] .= "beg_membership =".var_export($beg_membership, true);
                        if (count($beg_membership) != 2 && count($beg_membership) != 3) {
                            $this->errors[] = _T("- Invalid format of beginning of membership.");
                        } else {
                            $now = getdate();
                            if (!checkdate($beg_membership[1], $beg_membership[0], $now['year'])) {
                                $this->errors[] = _T("- Invalid date for beginning of membership.");
                            }
                        }
                        break;

Contribution.php ; line 209 :


        global $preferences;

        $bdate = new \DateTime($this->_begin_date);
        if ($preferences->pref_beg_membership != '') {
            //case beginning of membership
            //list($j, $m) = explode('/', $preferences->pref_beg_membership);
            list($j, $m, $freemonth) = preg_split('#[/;]#', $preferences->pref_beg_membership.';0'); //format (dd/mm;0) or (dd/mm;freemonth)
            $edate = new \DateTime($bdate->format('Y') . '-' . $m . '-' . $j);
            while ($edate <= $bdate) {
                $edate->modify('+1 year');
            }
            //hm->        
            if($freemonth>0)
            {
                //comparaison en nombre de jours (le 1er essai en nombre de mois ne fonctionnait pas avec diff(); pb d'arrondi par excès?)
                //le nombre de mois entre la date de fin venant d'être calculé et la date de début est-il inférieur ou égal à $freemonth ?
                $diff1 = intval($bdate->diff($edate)->format('%a'));
                $tdate = clone $edate;
                $tdate->modify('-' . $freemonth . ' month');
                $diff2 = intval($edate->diff($tdate)->format('%a'));
                if($diff1<=$diff2) $edate->modify('+1 year');
                //echo "<br>interval=".$diff1."<br>interval2=".$diff2."<br>";
            }
            // <-hm 
            $this->_end_date = $edate->format('Y-m-d');
        } elseif ($preferences->pref_membership_ext != '') {

Manuel

#4

Updated by Johan Cwiklinski 8 months ago

  • Category set to Core
  • Status changed from Rejeté to Nouveau

En effet, c'est une piste intéressante, j'y regarderai de plus près :)

Du coup, je réouvre ce ticket et ferme l'autre.

#5

Updated by Johan Cwiklinski 8 months ago

  • Is duplicate of deleted (Souhaits #913: Offrir les 3 derniers mois de l'année)
#6

Updated by Johan Cwiklinski 8 months ago

  • Has duplicate Souhaits #913: Offrir les 3 derniers mois de l'année added
#7

Updated by Johan Cwiklinski 8 months ago

Globalement, je suis d'accord. Ne serait-il pas plus aisé pour les utilisateurs que cette information soit dans un champ à part, plutôt que d'utiliser une syntaxe spécifique ? (personnellement, ça ne me gêne pas plus que ça, mais c'est forcément plus compliqué ;)).

Sinon, au regard de l'implémentation actuelle ; je n'aurai probablement pas eu recours à du preg split. À mon avis, il faut regarder pour la présence d'un ; et dans ce cas, récupérer la valeur, et la virer de la chaine d'origine. Quelque chose comme :

if (strpos($value, ';') !== false)) {
    list $beg_membership, $freemonth = explode(';', $value);
    $beg_membership = explode('/', $beg_membership);
} else {
    $beg_membership = explode('/', $value);    
}

Et probablement une fonction pour traiter ça proprement puisque c'est utile à différents endroits.

Enfin, dans ce genre là :)

#8

Updated by Manuel Her 8 months ago

J'ai utilisé le ; pour éviter de confondre avec une date type mm/dd/yy.

Il serait préférable d'ajouter un champ pour une meilleure compréhension, initialisé à 0 par défaut.

#9

Updated by Manuel Her 7 months ago

Bonjour,

Du coup, allez-vous l'intégrer dans la prochaine version ?

Manuel

#10

Updated by Johan Cwiklinski 7 months ago

  • Target version set to 0.9.4
  • Vote set to https://vote.galette.eu/posts/63/offer-last-n-months

Je pense que ce pourrait être dans la prochaine version, oui

#11

Updated by Johan Cwiklinski 6 months ago

  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski
  • % Done changed from 0 to 100

Si tu peux tester https://github.com/galette/galette/pull/47

Il faut que j'ajoute des tests unitaires pour valider le fonctionnement (enfin... si j'y arrive).

#12

Updated by Johan Cwiklinski 6 months ago

Johan Cwiklinski a écrit :

Il faut que j'ajoute des tests unitaires pour valider le fonctionnement (enfin... si j'y arrive).

C'est fait :)

#13

Updated by Johan Cwiklinski 6 months ago

  • Status changed from In Progress to Résolu
#14

Updated by Johan Cwiklinski 6 months ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF