Evolution #1407
closedAdhésion en année civile avec cadeau pour les 2 derniers mois
100%
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
Updated by Johan Cwiklinski almost 5 years ago
- Is duplicate of Souhaits #913: Offrir les 3 derniers mois de l'année added
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.
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
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.
Updated by Johan Cwiklinski almost 5 years ago
- Is duplicate of deleted (Souhaits #913: Offrir les 3 derniers mois de l'année)
Updated by Johan Cwiklinski almost 5 years ago
- Has duplicate Souhaits #913: Offrir les 3 derniers mois de l'année added
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à :)
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.
Updated by Manuel Her almost 5 years ago
Bonjour,
Du coup, allez-vous l'intégrer dans la prochaine version ?
Manuel
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
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).
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 :)
Updated by Johan Cwiklinski over 4 years ago
- Status changed from In Progress to Résolu
Appliqué par commit b52c47d1678b9365d9ec1d2546d19b496ea72f8e.
Updated by Johan Cwiklinski over 4 years ago
- Status changed from Résolu to Fermé