Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
K
kohinos-tav
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • agplv3
  • kohinos-tav
  • Merge Requests
  • !123

Merged
Opened Jan 08, 2025 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

calculate allocation with simplified household process

Edited Jan 15, 2025 by Damien Moulard
  • Discussion 20
  • Commits 12
  • Changes 18
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard @DamienM

    added 1 commit

    • 6b224702 - calculate allocation with simplified household process

    Compare with previous version

    Jan 15, 2025

    added 1 commit

    • 6b224702 - calculate allocation with simplified household process

    Compare with previous version

    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

    Compare with previous version

    Jan 15, 2025

    added 1 commit

    • c57cd235 - hide emlc ceiling functionnality in adherents list in simplified household process

    Compare with previous version

    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

    Jan 15, 2025

    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

    Compare with previous version

    Jan 17, 2025

    added 1 commit

    • 14c0725f - do not show subterritory in adherent admin in simplified mode

    Compare with previous version

    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
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 23, 2025
    src/Admin/AdherentAdmin.php
    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 commented Jan 17, 2025
      Master

      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

    Compare with previous version

    Jan 17, 2025

    added 1 commit

    • 444219d1 - orthographe colocation

    Compare with previous version

    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
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 23, 2025
    src/Admin/AdherentAdmin.php
    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 commented Jan 17, 2025
      Master

      J'ai corrigé cette orthographe.

      J'ai corrigé cette orthographe.
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 2 commits

    • 3a500fc1 - disable autocomplete on householdComposition to avoid issue with form wrongly…
    • 43f65113 - fix incorrect value of autocomplete attribute (off instead of false) to properly…

    Compare with previous version

    Jan 17, 2025

    added 2 commits

    • 3a500fc1 - disable autocomplete on householdComposition to avoid issue with form wrongly…
    • 43f65113 - fix incorrect value of autocomplete attribute (off instead of false) to properly…

    Compare with previous version

    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
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 23, 2025
    src/Admin/AdherentAdmin.php
    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 commented Jan 17, 2025
      Master

      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).
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 23, 2025
    src/Admin/AdherentAdmin.php
    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 commented Jan 17, 2025
      Master

      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).
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 23, 2025
    src/Admin/AdherentAdmin.php
    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 commented Jan 17, 2025
      Master

      J'ai ajouté l'autocomplete.

      J'ai ajouté l'autocomplete.
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on an old version of the diff Jan 17, 2025
    Resolved by Damien Moulard Jan 28, 2025
    src/Controller/UserComptoirController.php
    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 commented Jan 17, 2025
      Master

      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 commented Jan 20, 2025
      Master

      ç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 commented Jan 20, 2025
      Master

      appliquer également au cas household_based_allowance

      appliquer également au cas household_based_allowance
    • Damien Moulard @DamienM commented Jan 24, 2025
      Master

      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

      Jan 24, 2025

      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 commented Jan 27, 2025
      Master

      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 commented Jan 28, 2025
      Master

      tout à fait, c'était un oubli

      tout à fait, c'était un oubli
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Yvon Kerdoncuff Jan 27, 2025
    src/Controller/UserAdherentController.php
    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 commented Jan 17, 2025
      Master

      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 ?
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 17, 2025
    Resolved by Yvon Kerdoncuff Jan 27, 2025
    src/Utils/TAVCotisationUtils.php
    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 commented Jan 17, 2025
      Master

      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 commented Jan 24, 2025
      Master

      fait dans applyHouseholdAllowance, la fonction de versement de l'allocation

      fait dans applyHouseholdAllowance, la fonction de versement de l'allocation
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • 5051f980 - fix code error

    Compare with previous version

    Jan 20, 2025

    added 1 commit

    • 5051f980 - fix code error

    Compare with previous version

    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 started a discussion Jan 20, 2025
    Resolved by Yvon Kerdoncuff Jan 27, 2025
    • Damien Moulard @DamienM commented Jan 20, 2025
      Master

      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 commented Jan 24, 2025
      Master

      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
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion Jan 20, 2025
    Resolved by Yvon Kerdoncuff Jan 27, 2025
    • Damien Moulard @DamienM commented Jan 20, 2025
      Master

      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 commented Jan 24, 2025
      Master

      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é
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • 0c6901dd - fix code error

    Compare with previous version

    Jan 23, 2025

    added 1 commit

    • 0c6901dd - fix code error

    Compare with previous version

    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

    Compare with previous version

    Jan 24, 2025

    added 1 commit

    • 4ac38f0c - add button in adherents admin to recalculate household allowance & move verifs

    Compare with previous version

    added 1 commit <ul><li>4ac38f0c - add button in adherents admin to recalculate household allowance &amp; 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
  • Yvon Kerdoncuff
    @Yvon started a discussion on commit 4ac38f0c Jan 27, 2025
    Resolved by Damien Moulard Jan 28, 2025
    templates/themes/kohinos/tav/block/admin_recalculate_allocations.html.twig 0 → 100644
    1
    2 <li>
    • Yvon Kerdoncuff @Yvon commented Jan 27, 2025
      Master

      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 commented Jan 28, 2025
      Master

      bien vu

      bien vu
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 2 commits

    • 464a09f2 - remove useless call to calculateAllowanceAccordingToHousehold in…
    • 70f43ef3 - explain usage of button recalculate allocation

    Compare with previous version

    Jan 27, 2025

    added 2 commits

    • 464a09f2 - remove useless call to calculateAllowanceAccordingToHousehold in…
    • 70f43ef3 - explain usage of button recalculate allocation

    Compare with previous version

    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

    Jan 28, 2025

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    Jan 28, 2025

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 7e886ec1

    Jan 28, 2025

    mentioned in commit 7e886ec1

    mentioned in commit 7e886ec1912e3fc981b72471da042b652b3785dc
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: agplv3/kohinos-tav!123
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.