Project

General

Profile

Actions

Evolution #1407

closed

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

Added by Manuel Her almost 5 years ago. Updated over 4 years 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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Johan Cwiklinski almost 5 years ago

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

Updated by Johan Cwiklinski almost 5 years 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.

Actions #3

Updated by Manuel Her almost 5 years 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

Actions #4

Updated by Johan Cwiklinski almost 5 years 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.

Actions #5

Updated by Johan Cwiklinski almost 5 years ago

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

Updated by Johan Cwiklinski almost 5 years ago

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

Updated by Johan Cwiklinski almost 5 years 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à :)

Actions #8

Updated by Manuel Her almost 5 years 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.

Actions #9

Updated by Manuel Her almost 5 years ago

Bonjour,

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

Manuel

Actions #10

Updated by Johan Cwiklinski almost 5 years 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

Actions #11

Updated by Johan Cwiklinski almost 5 years 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).

Actions #12

Updated by Johan Cwiklinski almost 5 years 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 :)

Actions #13

Updated by Johan Cwiklinski over 4 years ago

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

Updated by Johan Cwiklinski over 4 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF