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

Merged
Opened Nov 21, 2022 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

set payment code from adherent homepage

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

    added 1 commit

    • 10db1f60 - remove useless code from SetPaymentCodeFormType

    Compare with previous version

    Nov 21, 2022

    added 1 commit

    • 10db1f60 - remove useless code from SetPaymentCodeFormType

    Compare with previous version

    added 1 commit <ul><li>10db1f60 - remove useless code from SetPaymentCodeFormType</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/5/diffs?diff_id=1913&start_sha=d710982b55305610f043a67bae751e50b9821502)
    Toggle commit list
  • Félicie
    @felicie started a discussion on an old version of the diff Nov 22, 2022
    Last updated by Damien Moulard Nov 22, 2022
    src/Controller/UserAdherentController.php
    161
    162 /**
    163 * @Route("/adherent/set-payment-code", name="adherentSetPaymentCode")
    164 * @IsGranted("ROLE_ADHERENT")
    165 */
    166 public function setPaymentCodeAction(Request $request, UserPasswordEncoderInterface $encoder)
    167 {
    168 $adherent = $this->getUser()->getAdherent();
    169 $form = $this->createForm(SetPaymentCodeFormType::class, $adherent);
    170 $form->handleRequest($request);
    171
    172 if ($form->isSubmitted() && $form->isValid()) {
    173 $data = $form->getData();
    174 $plainCode = $data->getPaymentCode();
    175
    176 if (is_numeric($plainCode) && strlen($plainCode) > 4 && strlen($plainCode) < 8) {
    • Félicie @felicie commented Nov 22, 2022

      Je pense que c'est plutôt des >= et des <= qu'il faut mettre ? Pour prendre en compte les code à 4 chiffres et les codes à 8 chiffres !

      Edited Nov 22, 2022
      Je pense que c'est plutôt des >= et des <= qu'il faut mettre ? Pour prendre en compte les code à 4 chiffres et les codes à 8 chiffres !
    • Damien Moulard @DamienM commented Nov 22, 2022
      Master

      Tout à fait !

      Tout à fait !
    • Damien Moulard @DamienM

      changed this line in version 3 of the diff

      Nov 22, 2022

      changed this line in version 3 of the diff

      changed this line in [version 3 of the diff](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/5/diffs?diff_id=1916&start_sha=10db1f60b9c6bc29210505758df581dea7ca8b8f#20b371c10322140a760bee75eaa25edf7d7185fc_176_176)
      Toggle commit list
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • 65e49214 - fix check payment code between 4 & 8 digits

    Compare with previous version

    Nov 22, 2022

    added 1 commit

    • 65e49214 - fix check payment code between 4 & 8 digits

    Compare with previous version

    added 1 commit <ul><li>65e49214 - fix check payment code between 4 &amp; 8 digits</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/5/diffs?diff_id=1916&start_sha=10db1f60b9c6bc29210505758df581dea7ca8b8f)
    Toggle commit list
  • Félicie @felicie

    merged

    Nov 22, 2022

    merged

    merged
    Toggle commit list
  • Félicie @felicie

    mentioned in commit db5e52e4

    Nov 22, 2022

    mentioned in commit db5e52e4

    mentioned in commit db5e52e4addbb951ca0127b90ad021323548ebf9
    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!5
×

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.