Project

General

Profile

Actions

Anomalie #1016

open

Cotisation par durée

Added by Michel LO over 5 years ago. Updated about 1 month ago.

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

100%

Estimated time:
Version utilisée:

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

Actions #1

Updated by Jean Joseph about 4 years ago

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

Actions #2

Updated by Guillaume Agniéray 6 months 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 ?

Actions #3

Updated by Guillaume Agniéray 5 months 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

Actions #4

Updated by Guillaume Agniéray about 1 month ago

  • Status changed from Commentaire to Résolu
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF