Project

General

Profile

Anomalie #1016

Cotisation par durée

Added by Michel LO almost 5 years ago. Updated 14 days ago.

Status:
Commentaire
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
02/22/2017
Due date:
% Done:

0%

Estimated time:
Version utilisée:
Vote:

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');

#1

Updated by Jean Joseph almost 4 years ago

Bonjour,
je rencontre la même difficulté.
Bien à vous.

#2

Updated by Guillaume Agnieray 14 days 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 ?

Also available in: Atom PDF