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

members/admin.py

@@ -843,7 +843,18 @@ def create_pair(request):
                 # 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
+                    # 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})
                     # On annule les rattrapages du child
                     api.update('res.partner', [child_id], {"makeups_to_do": 0})