Anomalie #1016
closedCotisation par durée
100%
Description
Dans une cotisation par durée, l'ajout de la durée entraîne un jour de trop car la date finale calculée est celle ddu début de préiode de cotisation suivante au lieu d'être la fin de cotisation.
Si la date de début de cotisation est le 22 février 2017, la date calculée de fin de cotisation de 12 mois doit être le 21 février et non pas le 22 février 2018
dans _retrieveEndDate, il faut calculer non pas en ajoutant la durée mais en ajoutant la durée - 1 jour
suggestion :
remplacer
//case membership extension
$dext = new \DateInterval('P' . $this->_extension . 'M’);
$edate = $bdate->add($dext);
$this->_end_date = $edate->format('Y-m-d');
par
//case membership extension
$dext = new \DateInterval('P' . $this->_extension . 'M');
$moinsunjour = new \DateInterval("P1D");
$moinsunjour->invert=1;
$edate = $bdate->add($dext);
$edate = $bdate->add($moinsunjour);
$this->_end_date = $edate->format('Y-m-d');
Updated by Jean Joseph over 6 years ago
Bonjour,
je rencontre la même difficulté.
Bien à vous.
Updated by Guillaume AGNIERAY almost 3 years ago
- Status changed from Nouveau to Commentaire
J'ai cet inconvénient aussi dans le cas de cotisations par date de début d'exercice.
Sur la liste des adhérents, la date d'échéance affichée est aussi celle du premier jour de l'exercice suivant.
Voici une tentative pour corriger ça dans les 2 cas de figure : https://framagit.org/galette/galette/-/merge_requests/12
J'apporte au passage un changement sur l'édition des cotisations par durée.
Actuellement, la durée est toujours ajoutée à la date de début de cotisation pour recalculer la date d'échéance.
J'aimerai de mon côté pouvoir étendre la période en renseignant simplement le nombre de mois supplémentaires souhaités.
Par exemple renseigner 4 mois dans le champ et pas 16.
À ce stade, si j'ai bien ajusté les tests unitaires, tout semble fonctionner correctement.
Si je n'oublie rien, il reste encore :- à régler le cas de la modification d'une contribution où seules les autres infos que la durée sont changées (le montant uniquement par exemple)
- à préparer l'update pour les données existantes : il faudra soustraire 1 jour de toutes les dates des cotisations déjà enregistrées.
P.S. Pour moi la version utilisée est la 9.6.1. Dois-je changer la version utilisée dans les propriétés de la demande ?
Updated by Guillaume AGNIERAY over 2 years ago
Guillaume Agnieray a écrit (#note-2):
Voici une tentative pour corriger ça dans les 2 cas de figure : https://framagit.org/galette/galette/-/merge_requests/12
J'ai revu ma copie en traitant aussi au passage #1590 : https://framagit.org/galette/galette/-/merge_requests/17
J'apporte au passage un changement sur l'édition des cotisations par durée.
Actuellement, la durée est toujours ajoutée à la date de début de cotisation pour recalculer la date d'échéance.J'aimerai de mon côté pouvoir étendre la période en renseignant simplement le nombre de mois supplémentaires souhaités.
Par exemple renseigner 4 mois dans le champ et pas 16.
J'ai abandonné cette idée. Ça n'a en réalité pas grand intérêt :p
Updated by Guillaume AGNIERAY over 2 years ago
- Status changed from Commentaire to Résolu
- % Done changed from 0 to 100
Appliqué par commit e89541901276f14915c3f38da7c650083f3db354.
Updated by Guillaume AGNIERAY 11 months ago
Cette demande est résolue depuis plus d'un an. Je pense qu'elle peut être fermée.
Updated by Guillaume AGNIERAY 11 months ago
- Related to Anomalie #1762: Contribution start date overload added