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
  • !104

Merged
Opened May 30, 2024 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

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.

  • Discussion 12
  • Commits 2
  • Changes 12
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard @DamienM

    added 1 commit

    • 438ea32d - change env variable name

    Compare with previous version

    May 30, 2024

    added 1 commit

    • 438ea32d - change env variable name

    Compare with previous version

    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
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 03, 2024
    Resolved by Damien Moulard Jun 04, 2024
    src/Entity/Payment.php
    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 commented Jun 03, 2024
      Master

      image

      ![image](/uploads/2f3bb7f698e61bf63c6726e6e0615192/image.png)
    • Yvon Kerdoncuff @Yvon commented Jun 03, 2024
      Master

      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 commented Jun 04, 2024
      Master

      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
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 03, 2024
    Resolved by Damien Moulard Jun 04, 2024
    config/services.yaml
    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 commented Jun 03, 2024
      Master

      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 commented Jun 04, 2024
      Master

      Bien vu, j'ai oublié

      Bien vu, j'ai oublié
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 03, 2024
    Resolved by Damien Moulard Jun 04, 2024
    src/Admin/AdherentAdmin.php
    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 commented Jun 03, 2024
      Master

      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 commented Jun 04, 2024
      Master

      Tout à fait bien vu

      Tout à fait bien vu
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 03, 2024
    Resolved by Damien Moulard Jun 04, 2024
    src/Admin/AdherentAdmin.php
    444 473 $em->flush();
    445 474 }
    446 475 }
    476
    477 // Cancel recurring payment
    478 if(
    • Yvon Kerdoncuff @Yvon commented Jun 03, 2024
      Master

      En jouant un peu j'ai réussi à provoquer des comportements bizarres : image

      En jouant un peu j'ai réussi à provoquer des comportements bizarres : ![image](/uploads/28278ab4c23db7598b8c01b1b6d5a94c/image.png)
    • Yvon Kerdoncuff @Yvon commented Jun 03, 2024
      Master

      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.

      Edited Jun 03, 2024 by Yvon Kerdoncuff
      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 commented Jun 04, 2024
      Master

      ok

      ok
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jun 03, 2024
    Resolved by Damien Moulard Jun 04, 2024
    assets/js/admin.js
    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 commented Jun 03, 2024
      Master

      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 commented Jun 04, 2024
      Master

      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.
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    merged

    Jun 03, 2024

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    resolved all discussions

    Jun 04, 2024

    resolved all discussions

    resolved all discussions
    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!104