Commit 28c47c33 by Yvon

simplify comments

parent b0d4dc2e
Pipeline #3388 failed with stage
in 1 minute 5 seconds
...@@ -844,16 +844,7 @@ def create_pair(request): ...@@ -844,16 +844,7 @@ def create_pair(request):
if child_makeups + parent_makeups <=2: if child_makeups + parent_makeups <=2:
# on transfert les rattrapages sur le parent # on transfert les rattrapages sur le parent
# Commentaires Yvon (12/04/2023, ticket 6001) : # Comme on annule les services du suppléant, on ajoute en makeup_to_do du titulaire les rattrapages choisis ou non du suppléant
# Initiallement, on mettait à jour ici le nouveau nombre de rattrapages "makeups_to_do" du titulaire en
# faisant la somme des makeups_to_do du titulaire et du suppléant.
# Ce calcul se révèle insatisfaisant car les rattrapages déjà choisis du suppléant, qui sont comptés
# dans child_makeups, ne retrouvent pas leur contrepartie en terme de rattrapages chez le titulaire.
# Ceci provoque une incohérence entre le nombre de points et le nombre total de rattrapages.
# On va donc les comptabiliser pour mettre à jour makeups_to_do du titulaire.
# Autrement dit, on réintègre les rattrapages choisis mais pas encore réalisés du suppléant.
# Idéalement, il faudrait aussi se pencher attentivement sur la question de l'état dans lequel
# est laissé le suppléant après une désolidarisation du binôme, mais c'est une autre histoire
api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child_makeups}) api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child_makeups})
# On annule les rattrapages du child # On annule les rattrapages du child
api.update('res.partner', [child_id], {"makeups_to_do": 0}) api.update('res.partner', [child_id], {"makeups_to_do": 0})
......
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 to comment