-
Damien Moulard @DamienM
added 1 commit
- 6b224702 - calculate allocation with simplified household process
added 1 commit
- 6b224702 - calculate allocation with simplified household process
added 1 commit <ul><li>6b224702 - calculate allocation with simplified household process</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3711&start_sha=011398afa4ee843454c19b53215daed2b2584fa2)Toggle commit list -
Damien Moulard @DamienM
added 1 commit
- c57cd235 - hide emlc ceiling functionnality in adherents list in simplified household process
added 1 commit
- c57cd235 - hide emlc ceiling functionnality in adherents list in simplified household process
added 1 commit <ul><li>c57cd235 - hide emlc ceiling functionnality in adherents list in simplified household process</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3712&start_sha=6b22470203a84ed057dc8d941580a77e1a09d99f)Toggle commit list -
Damien Moulard @DamienM
unmarked as a Work In Progress
unmarked as a Work In Progress
unmarked as a **Work In Progress**Toggle commit list -
Yvon Kerdoncuff @Yvon
added 1 commit
- 14c0725f - do not show subterritory in adherent admin in simplified mode
added 1 commit
- 14c0725f - do not show subterritory in adherent admin in simplified mode
added 1 commit <ul><li>14c0725f - do not show subterritory in adherent admin in simplified mode</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3720&start_sha=c57cd23535657bdbd9d40a07760d75ee4ffe6eab)Toggle commit list -
178 179 'required' => true, 179 180 'with_geoloc' => false, 180 181 'with_latlon' => false, 181 'with_subterritory' => $tav_env && $household_based_allowance, 182 'with_subterritory' => -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai fait cette correction qui permet de masquer le territoire en mode simplifié.
J'ai fait cette correction qui permet de masquer le territoire en mode simplifié.
Please register or sign in to reply -
-
Yvon Kerdoncuff @Yvon
added 1 commit
- 444219d1 - orthographe colocation
added 1 commit <ul><li>444219d1 - orthographe colocation</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3721&start_sha=14c0725f9d56e85fd56fc9060560821781aa5ba7)Toggle commit list -
255 'by_reference' => false, 256 'label' => "Enfant(s) à charge (pour calculer l'allocation)" 257 ]) 258 ->end() 259 ->end(); 226 if ($simplified_household_based_allowance) { 227 $formMapper 228 ->tab('General') 229 ->with('Foyer', ['class' => 'col-md-7']) 230 ->add('householdComposition',ChoiceType::class, [ 231 'choices' => [ 232 "Personne seule" => "Personne seule", 233 "Famille mono-parentale" => "Famille mono-parentale", 234 "Couple" => "Couple", 235 "Couple avec enfant(s)" => "Couple avec enfant(s)", 236 "Colocation" => "Colocation" -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai corrigé cette orthographe.
J'ai corrigé cette orthographe.
-
-
Yvon Kerdoncuff @Yvon
added 2 commits
added 2 commits
added 2 commits <ul><li>3a500fc1 - disable autocomplete on householdComposition to avoid issue with form wrongly…</li><li>43f65113 - fix incorrect value of autocomplete attribute (off instead of false) to properly…</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3722&start_sha=444219d1e45bb3a7c4a618974f8cfcdddb2cca7b)Toggle commit list -
265 "Famille mono-parentale" => "Famille mono-parentale", 266 "Couple avec enfant(s) à charge" => "Couple avec enfant(s) à charge", 267 "Autre" => "Autre" 268 ], 269 'label' => "Composition du foyer (pour information)", 270 'required' => true, 271 'placeholder' => "Choix de la composition du foyer", 272 ]) 273 ->add('householdAdultCount',IntegerType::class, [ 274 'label' => "Nombre total d'adultes dans le foyer (pour calculer l'allocation)", 275 'constraints' => [ 276 new GreaterThanOrEqual(['value' => 0]), 277 ], 278 'required' => true, 279 'attr' => [ 280 'autocomplete' => 'off' -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai basculé à "off" (l'autocomplete était à false ce qui ne marche pas).
J'ai basculé à "off" (l'autocomplete était à false ce qui ne marche pas).
-
-
237 ], 238 'label' => "Composition du foyer (pour information)", 239 'required' => true, 240 'attr' => [ 241 'autocomplete' => 'off' 242 ], 243 'placeholder' => "Choix de la composition du foyer", 244 ]) 245 ->add('householdCount',IntegerType::class, [ 246 'label' => "Nombre total de personnes que vous souhaitez engager dans l'expérimentation", 247 'constraints' => [ 248 new GreaterThanOrEqual(['value' => 0]), 249 ], 250 'required' => true, 251 'attr' => [ 252 'autocomplete' => 'off' -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai basculé à "off" (l'autocomplete était à false ce qui ne marche pas).
J'ai basculé à "off" (l'autocomplete était à false ce qui ne marche pas).
-
-
226 if ($simplified_household_based_allowance) { 227 $formMapper 228 ->tab('General') 229 ->with('Foyer', ['class' => 'col-md-7']) 230 ->add('householdComposition',ChoiceType::class, [ 231 'choices' => [ 232 "Personne seule" => "Personne seule", 233 "Famille mono-parentale" => "Famille mono-parentale", 234 "Couple" => "Couple", 235 "Couple avec enfant(s)" => "Couple avec enfant(s)", 236 "Colocation" => "Colocation" 237 ], 238 'label' => "Composition du foyer (pour information)", 239 'required' => true, 240 'attr' => [ 241 'autocomplete' => 'off' -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai ajouté l'autocomplete.
J'ai ajouté l'autocomplete.
-
-
221 // Some code duplication between household_based_allowance && simplified_household_based_allowance, for better readability 222 if ($allocationMethod == 'simplified_household_based_allowance') { 223 /* Process: simplified allowance based on household */ 224 $cotisationAmount = $destinataire->getCotisationAmount(); 225 226 // Verifications 227 if (is_null($cotisationAmount) || is_null($destinataire->getHouseholdCount()) ) { 228 $this->addFlash( 229 'error', 230 $this->translator->trans("Opération impossible : le profil de l'habitant.e est incomplet, veuillez le compléter dans l'interface d'administration.") 231 ); 232 233 return $this->redirectToRoute('index'); 234 } 235 236 if (is_null($destinataire->getAllocationAmount())) { -
Yvon Kerdoncuff @Yvon commentedMaster
Le fait de ne pas recalculer systématiquement le montant d'allocation donnera un résultat faux si on bascule entre les modes household "simplified" et "gironde-style". En effet, c'est la dernière valeur de l'allocation qui sera utilisée, et non la valeur qui correspond au paramétrage en cours. Le plus simple serait à mon avis de recalculer systématiquement. Le calcul est en effet extrêmement simple et ne nécessite pour ainsi dire pas de ressources supplémentaires, mais il évite de se faire des noeuds au cerveau.
Le fait de ne pas recalculer systématiquement le montant d'allocation donnera un résultat faux si on bascule entre les modes household "simplified" et "gironde-style". En effet, c'est la dernière valeur de l'allocation qui sera utilisée, et non la valeur qui correspond au paramétrage en cours. Le plus simple serait à mon avis de recalculer systématiquement. Le calcul est en effet extrêmement simple et ne nécessite pour ainsi dire pas de ressources supplémentaires, mais il évite de se faire des noeuds au cerveau. -
Damien Moulard @DamienM commentedMaster
ça me va : recalculer systématiquement même si montant déjà enregistré
ça me va : recalculer systématiquement même si montant déjà enregistré -
Damien Moulard @DamienM commentedMaster
appliquer également au cas household_based_allowance
appliquer également au cas household_based_allowance -
Damien Moulard @DamienM commentedMaster
vu ensemble, ajout plutôt d'un bouton pour recalculer le montant dans la liste d'adhérents en cas de changement de mode d'alloc
vu ensemble, ajout plutôt d'un bouton pour recalculer le montant dans la liste d'adhérents en cas de changement de mode d'alloc -
Damien Moulard @DamienM
changed this line in version 9 of the diff
changed this line in version 9 of the diff
changed this line in [version 9 of the diff](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3750&start_sha=0c6901dd175214283ecac893dce8e125fdc2bef9#d0c7dabdcb45c8ad7a3dbfa36cdd459c7b98ec64_236_236)Toggle commit list -
Yvon Kerdoncuff @Yvon commentedMaster
Suite à l'ajout du bouton, je vois que tu as supprimé l'appel à calculateAllowanceAccordingToHouseholdSimplified qui se trouvait dans comptoirEncaisserCotisationAction, mais pas l'appel à calculateAllowanceAccordingToHousehold qui se trouve juste après. Je pense qu'il s'agit d'un oubli donc j'ai pris l'initiative de faire la modification.
Suite à l'ajout du bouton, je vois que tu as supprimé l'appel à calculateAllowanceAccordingToHouseholdSimplified qui se trouvait dans comptoirEncaisserCotisationAction, mais pas l'appel à calculateAllowanceAccordingToHousehold qui se trouve juste après. Je pense qu'il s'agit d'un oubli donc j'ai pris l'initiative de faire la modification. -
Damien Moulard @DamienM commentedMaster
tout à fait, c'était un oubli
tout à fait, c'était un oubli
-
-
130 130 private function paiementCotisTavValidation($flux) { 131 131 132 132 $destinataire = $flux->getDestinataire(); 133 if($reason = $this->tavCotisationsUtils->preventCotisationDuplication($destinataire)) { 133 if($reason = $this->tavCotisationUtils->preventCotisationDuplication($destinataire)) { 134 134 return $reason; 135 135 } 136 136 137 137 // Look for cotisation data depending on active process 138 if (true == $this->getParameter('household_based_allowance')) { -
Yvon Kerdoncuff @Yvon commentedMaster
Pourquoi est-ce qu'ici on n'envisage pas le recalcul du montant d'allocation comme c'est le cas dans le cas d'un paiement comptoir ? Qu'est-ce qui justifie cette disymétrie ?
Pourquoi est-ce qu'ici on n'envisage pas le recalcul du montant d'allocation comme c'est le cas dans le cas d'un paiement comptoir ? Qu'est-ce qui justifie cette disymétrie ?
-
-
372 405 } 373 406 $this->em->flush(); 374 407 } 408 409 /** 410 * Check parameters and Adherent data in order to detect the right allocation method to use. 411 * 412 * Specific rule: if simplified_household_based_allowance is active BUT adherent's profile is incomplete for this method AND (s)he has a ProfilDeCotisation set, use ProfilDeCotisation 413 * (allows flowless transition from ProfilDeCotisation to simplified_household_based_allowance) 414 * 415 * @param Adherent $adherent 416 * @return String 'cotisation_profile' | 'household_based_allowance' | 'simplified_household_based_allowance 417 */ 418 public function getAppropriateAllocationMethod($adherent) { 419 if ($this->container->getParameter('simplified_household_based_allowance')) { 420 if ( -
Yvon Kerdoncuff @Yvon commentedMaster
Pourquoi ne vérifie-t-on pas si le AllocationAmount est bien renseigné également ? Est-ce qu'on gère ailleurs le fait de basculer sur le mode d'allocation standard quand allocation amount n'est pas renseigné ?
Est-ce que cette fonction est supposée définir la méthode d'allocation et de cotisation, ou bien seulement de cotisation ? ou bien seulement d'allocation ?
Pourquoi ne vérifie-t-on pas si le AllocationAmount est bien renseigné également ? Est-ce qu'on gère ailleurs le fait de basculer sur le mode d'allocation standard quand allocation amount n'est pas renseigné ? Est-ce que cette fonction est supposée définir la méthode d'allocation et de cotisation, ou bien seulement de cotisation ? ou bien seulement d'allocation ? -
Damien Moulard @DamienM commentedMaster
fait dans applyHouseholdAllowance, la fonction de versement de l'allocation
fait dans applyHouseholdAllowance, la fonction de versement de l'allocation
-
-
Damien Moulard @DamienM
added 1 commit
- 5051f980 - fix code error
added 1 commit <ul><li>5051f980 - fix code error</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3726&start_sha=43f65113a9c4f424939e792e3841c3401d84d9ac)Toggle commit list -
-
Damien Moulard @DamienM commentedMaster
Faire le recalcul du montant d'allocation dans tous les cas en cas de paiement payzen également (permet de recalculer le montant si on bascule puis rebascule entre les 2 modes household)
Faire le recalcul du montant d'allocation dans tous les cas en cas de paiement payzen également (permet de recalculer le montant si on bascule puis rebascule entre les 2 modes household) -
Damien Moulard @DamienM commentedMaster
Création d'un boutton à cet effet dans la liste des adhérents
Création d'un boutton à cet effet dans la liste des adhérents
-
-
-
Damien Moulard @DamienM commentedMaster
Voir pour mettre le calcul du montant de l'allocation dans getAppropriateAllocationMethod ? S'assurer alors qu'elle est bien appelée à chaque fois
Voir pour mettre le calcul du montant de l'allocation dans getAppropriateAllocationMethod ? S'assurer alors qu'elle est bien appelée à chaque fois -
Damien Moulard @DamienM commentedMaster
finalement je l'ai déplacé au moment du versement dans la méthode applyHouseholdAllowance, comme ça on est certains qu'un montant éventuellement null sera calculé
finalement je l'ai déplacé au moment du versement dans la méthode applyHouseholdAllowance, comme ça on est certains qu'un montant éventuellement null sera calculé
-
-
Damien Moulard @DamienM
added 1 commit
- 0c6901dd - fix code error
added 1 commit <ul><li>0c6901dd - fix code error</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3747&start_sha=5051f980f11912b9f752d538b5300631d2891ec1)Toggle commit list -
Damien Moulard @DamienM
added 1 commit
- 4ac38f0c - add button in adherents admin to recalculate household allowance & move verifs
added 1 commit
- 4ac38f0c - add button in adherents admin to recalculate household allowance & move verifs
added 1 commit <ul><li>4ac38f0c - add button in adherents admin to recalculate household allowance & move verifs</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3750&start_sha=0c6901dd175214283ecac893dce8e125fdc2bef9)Toggle commit list -
1 2 <li> -
Yvon Kerdoncuff @Yvon commentedMaster
J'ai ajouté un message au passage de la souris sur le bouton pour expliquer sa raison d'être.
J'ai ajouté un message au passage de la souris sur le bouton pour expliquer sa raison d'être. -
Damien Moulard @DamienM commentedMaster
bien vu
bien vu
-
-
Yvon Kerdoncuff @Yvon
added 2 commits
added 2 commits
added 2 commits <ul><li>464a09f2 - remove useless call to calculateAllowanceAccordingToHousehold in…</li><li>70f43ef3 - explain usage of button recalculate allocation</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/123/diffs?diff_id=3764&start_sha=4ac38f0cfc5792747abb7bb70ac37569119fe10a)Toggle commit list -
Damien Moulard @DamienM
resolved all discussions
resolved all discussions
resolved all discussionsToggle commit list -
Damien Moulard @DamienM
merged
merged
mergedToggle commit list -
Damien Moulard @DamienM
mentioned in commit 7e886ec1
mentioned in commit 7e886ec1
mentioned in commit 7e886ec1912e3fc981b72471da042b652b3785dcToggle commit list
Revert this merge request
This will create a new commit in order to revert the existing changes.