Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
T
third-party
  • 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 2
    • Merge Requests 2
  • 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
  • cooperatic-foodcoops
  • third-party
  • Merge Requests
  • !265

Merged
Opened Apr 12, 2024 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

fix computation of new makeups_to_do of titulaire when creating pair by taking…

fix computation of new makeups_to_do of titulaire when creating pair by taking into account already selected makeups of suppleant

  • Discussion 2
  • Commits 2
  • Pipelines 2
  • Changes 1
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion on the diff Apr 19, 2024
    Resolved by Yvon Kerdoncuff Apr 19, 2024
    members/admin.py
    843 843 # le suppléant a des rattrapages
    844 844 if child_makeups + parent_makeups <=2:
    845 845 # on transfert les rattrapages sur le parent
    846 api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child['makeups_to_do']})
    846
    • Damien Moulard @DamienM commented Apr 19, 2024
      Developer

      Je pense qu'on peut grandement simplifier le commentaire. "Comme on annule les services du suppléant, on ajoute en makeup_to_do du titulaire les rattrapages choisis ou non du suppléant"

      Je pense qu'on peut grandement simplifier le commentaire. "Comme on annule les services du suppléant, on ajoute en makeup_to_do du titulaire les rattrapages choisis ou non du suppléant"
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion on an old version of the diff Apr 19, 2024
    Last updated by Yvon Kerdoncuff Apr 19, 2024
    members/admin.py
    843 843 # le suppléant a des rattrapages
    844 844 if child_makeups + parent_makeups <=2:
    845 845 # on transfert les rattrapages sur le parent
    846 api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child['makeups_to_do']})
    846
    847 # Commentaires Yvon (12/04/2023, ticket 6001) :
    848 # Initiallement, on mettait à jour ici le nouveau nombre de rattrapages "makeups_to_do" du titulaire en
    849 # faisant la somme des makeups_to_do du titulaire et du suppléant.
    850 # Ce calcul se révèle insatisfaisant car les rattrapages déjà choisis du suppléant, qui sont comptés
    851 # dans child_makeups, ne retrouvent pas leur contrepartie en terme de rattrapages chez le titulaire.
    852 # Ceci provoque une incohérence entre le nombre de points et le nombre total de rattrapages.
    853 # On va donc les comptabiliser pour mettre à jour makeups_to_do du titulaire.
    854 # Autrement dit, on réintègre les rattrapages choisis mais pas encore réalisés du suppléant.
    855 # Idéalement, il faudrait aussi se pencher attentivement sur la question de l'état dans lequel
    • Damien Moulard @DamienM commented Apr 19, 2024
      Developer

      Pour la postérité, pas certain de la pertinence de mettre un commentaire dans le code pour une future amélioration. De fortes chances qu'on ne revienne jamais dessus. Je pencherais plus pour créer un ticket et rester sobre (autant que possible et nécessaire) dans les commentaires du code.

      Pour la postérité, pas certain de la pertinence de mettre un commentaire dans le code pour une future amélioration. De fortes chances qu'on ne revienne jamais dessus. Je pencherais plus pour créer un ticket et rester sobre (autant que possible et nécessaire) dans les commentaires du code.
    • Yvon Kerdoncuff @Yvon

      changed this line in version 2 of the diff

      Apr 19, 2024

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/265/diffs?diff_id=2946&start_sha=b0d4dc2ea0e05de5ccd6a4851d48c91c8e2b23f2#5b90c8970b97ef1e3043ecaaaf5dfd4cf2f2db68_855_847)
      Toggle commit list
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 28c47c33 - simplify comments

    Compare with previous version

    Apr 19, 2024

    added 1 commit

    • 28c47c33 - simplify comments

    Compare with previous version

    added 1 commit <ul><li>28c47c33 - simplify comments</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic-foodcoops/third-party/merge_requests/265/diffs?diff_id=2946&start_sha=b0d4dc2ea0e05de5ccd6a4851d48c91c8e2b23f2)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    Apr 19, 2024

    merged

    merged
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 3a3e7d21

    Apr 19, 2024

    mentioned in commit 3a3e7d21

    mentioned in commit 3a3e7d215f079178b93656159a61ebff260c98df
    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: cooperatic-foodcoops/third-party!265
×

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.