cancel recurrent payment in kohinos from adherent admin
Check out, review, and merge locally
Step 1. Fetch and check out the branch for this merge request
git fetch origin git checkout -b 6367-cancel-recurring-payment origin/6367-cancel-recurring-payment
Step 2. Review the changes locally
Step 3. Merge the branch and fix any conflicts that come up
git checkout develop git merge --no-ff 6367-cancel-recurring-payment
Step 4. Push the result of the merge to GitLab
git push origin develop
Note that pushing to GitLab requires write access to this repository.
Tip: You can also checkout merge requests locally by following these guidelines.
-
Damien Moulard @DamienM
added 1 commit
- 438ea32d - change env variable name
added 1 commit <ul><li>438ea32d - change env variable name</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/104/diffs?diff_id=3108&start_sha=7e5dcd515e5fc56cc476dec8adcd8b0642ce2b87)Toggle commit list -
227 227 $now = new DateTime(); 228 228 if ($paymentEndDate && $paymentEndDate < $paymentEndDateDueToExpiry) { 229 229 //Compare now with payment end date 230 $reason = "Paiement récurrent en cours jusqu'à dernière échéance le " . $paymentEndDate->format('d/m/Y') . "."; 230 $reason = "Paiement récurrent en cours de " . $this.recurrenceAmount() . "€ jusqu'à dernière échéance le " . $paymentEndDate->format('d/m/Y') . "."; -
Yvon Kerdoncuff @Yvon commentedMaster
-
Yvon Kerdoncuff @Yvon commentedMaster
Je corrige directement sur ssa-gironde et je mets à jour ssagironde.coopdev.fr.
Je corrige directement sur ssa-gironde et je mets à jour ssagironde.coopdev.fr. -
Damien Moulard @DamienM commentedMaster
effectivement typo de ma part d'avoir rajouté les parenthèses (?)
effectivement typo de ma part d'avoir rajouté les parenthèses (?)
Please register or sign in to reply -
-
18 18 presta_extra_data: '%env(PRESTA_EXTRA_DATA)%' 19 19 household_based_allowance: '%env(HOUSEHOLD_BASED_ALLOWANCE)%' 20 20 ssa_friendly_flux_type_names: '%env(SSA_FRIENDLY_FLUX_TYPE_NAMES)%' 21 fix_balance_adherent_password: '%env(FIX_BALANCE_ADHERENT_PASSWORD)%' 21 extra_security_admin_password: '%env(EXTRA_SECURITY_ADMIN_PASSWORD)%' -
Yvon Kerdoncuff @Yvon commentedMaster
Le changement doit être repercuté également dans .env.dist -> corrigé sur ssa-gironde.
Le changement doit être repercuté également dans .env.dist -> corrigé sur ssa-gironde. -
Damien Moulard @DamienM commentedMaster
Bien vu, j'ai oublié
Bien vu, j'ai oublié
-
-
396 397 ->end() 397 398 ->end(); 398 399 } 400 401 if ($reason = $this->tavCotisationUtils->checkExistingRecurringPayment($adherent->getUser()->getEmail())) { 402 $formMapper 403 ->tab('General') 404 ->with('Informations de cotisation') 405 ->add('recurrentPaymentReason', HiddenType::class, [ 406 'mapped' => false, 407 'data' => $reason, 408 'attr' => [ 409 'class' => 'recurrentPaymentReason' 410 ] 411 ]) 412 ->add('cancelRecurrentPaymentPassword', TextType::class, [ -
Yvon Kerdoncuff @Yvon commentedMaster
Par homogénéité avec le mot de passe pour correction de solde je me suis permis de passer en type PasswordType (ssa-gironde).
Par homogénéité avec le mot de passe pour correction de solde je me suis permis de passer en type PasswordType (ssa-gironde). -
Damien Moulard @DamienM commentedMaster
Tout à fait bien vu
Tout à fait bien vu
-
-
444 473 $em->flush(); 445 474 } 446 475 } 476 477 // Cancel recurring payment 478 if( -
Yvon Kerdoncuff @Yvon commentedMaster
-
Yvon Kerdoncuff @Yvon commentedMaster
Je sais pas comment j'ai fait ça et c'est pénible à reproduire. Il y a peu de chance pour que quiconque se retrouve avec cette situation donc je propose de ne pas essayer de le corriger mais j'ouvre qd mm un ticket 6433 pour trace.
Je sais pas comment j'ai fait ça et c'est pénible à reproduire. Il y a peu de chance pour que quiconque se retrouve avec cette situation donc je propose de ne pas essayer de le corriger mais j'ouvre qd mm un ticket 6433 pour trace. -
Damien Moulard @DamienM commentedMaster
ok
ok
-
-
133 133 } 134 134 135 135 if($(".fixBalanceAdherentFormPart").length > 0) { 136 $(".fixBalanceAdherentFormPart").parent().parent().hide(); 137 $("div[id$='_idmlc']").append( 138 "<br/><input type='button' id='showFieldsToFixBalance' onclick='$(\".fixBalanceAdherentFormPart\").parent().parent().show();' value='Afficher les champs pour corriger le solde'> </input>" 136 if (!$(".fixBalanceAdherentFormPart").parent().parent().hasClass('has-error')) { 137 $(".fixBalanceAdherentFormPart").parent().parent().hide(); 138 } 139 $(".fixBalanceAdherentFormPart").first().parent().parent().before( 140 "<div class='form-group'><input class='btn btn-primary' type='button' id='showFieldsToFixBalance' value='Afficher les champs pour corriger le solde'></input></div>" 141 ); 142 143 $('#showFieldsToFixBalance').on('click', function (e) { -
Yvon Kerdoncuff @Yvon commentedMaster
C'est sympa de toggle, mais je pense que c'est pas une super idée car ça n'efface pas les champs, qui sont soumis même masqués, donc je suis pas convaincu, mais bon c'est un détail donc on peut laisser comme ça.
C'est sympa de toggle, mais je pense que c'est pas une super idée car ça n'efface pas les champs, qui sont soumis même masqués, donc je suis pas convaincu, mais bon c'est un détail donc on peut laisser comme ça. -
Damien Moulard @DamienM commentedMaster
Je vois, on pourrait vider les champs quand on les cache. Pas prioritaire effectivement.
Je vois, on pourrait vider les champs quand on les cache. Pas prioritaire effectivement.
-
-
Yvon Kerdoncuff @Yvon
merged
merged
mergedToggle commit list -
Damien Moulard @DamienM
resolved all discussions
resolved all discussions
resolved all discussionsToggle commit list