Yvon Kerdoncuff · @Yvon · Damien Moulard · Yvon Kerdoncuff · Damien Moulard · Yvon Kerdoncuff
--- a/members/admin.py
+++ b/members/admin.py
                # le suppléant a des rattrapages
                if child_makeups + parent_makeups <=2:
                    # on transfert les rattrapages sur le parent
-                    api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child['makeups_to_do']})
+
--- a/members/admin.py
+++ b/members/admin.py
                # le suppléant a des rattrapages
                if child_makeups + parent_makeups <=2:
                    # on transfert les rattrapages sur le parent
-                    api.update("res.partner", [parent_id], {"makeups_to_do": parent['makeups_to_do'] + child['makeups_to_do']})
+
+                    # Commentaires Yvon (12/04/2023, ticket 6001) :
+                    # 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