From 053e39a390ef4df83f9d74dc3c2c7bdb4db86b09 Mon Sep 17 00:00:00 2001
From: Nicolas Oudard <nicolas.oudard@beta.gouv.fr>
Date: Thu, 9 Jan 2025 08:57:22 +0100
Subject: [PATCH] Prise en compte du commentaire de Fabien (#1191)

---
 qfdmo/models/acteur.py | 30 +++++++++++++-----------------
 1 file changed, 13 insertions(+), 17 deletions(-)

diff --git a/qfdmo/models/acteur.py b/qfdmo/models/acteur.py
index 4419d4878..852c09c53 100644
--- a/qfdmo/models/acteur.py
+++ b/qfdmo/models/acteur.py
@@ -2,6 +2,7 @@
 import random
 import string
 import uuid
+from copy import deepcopy
 from typing import Any, List, cast
 
 import opening_hours
@@ -601,12 +602,7 @@ def duplicate(self):
         if self.is_parent:
             raise Exception("Impossible de dupliquer un acteur parent")
 
-        # Get linked objects before removing pk
-        labels = self.labels.all()
-        acteur_services = self.acteur_services.all()
-        # Besoin de cette "list comprehension" pour executer la Queryset,
-        # sinon elle n'est pas executé avant que le revisionacteur soit modifié
-        proposition_services = [ps for ps in self.proposition_services.all()]
+        revision_acteur = deepcopy(self)
 
         acteur = Acteur.objects.get(identifiant_unique=self.identifiant_unique)
 
@@ -623,30 +619,30 @@ def duplicate(self):
         ]
 
         for field in fields_to_reset:
-            setattr(self, field, None)
-        for field in self._meta.fields:
+            setattr(revision_acteur, field, None)
+        for field in revision_acteur._meta.fields:
             if (
-                not getattr(self, field.name)
+                not getattr(revision_acteur, field.name)
                 and field.name not in fields_to_reset
                 and field.name not in fields_to_ignore
             ):
-                setattr(self, field.name, getattr(acteur, field.name))
-        self.save()
-        self.labels.set(labels)
-        self.acteur_services.set(acteur_services)
+                setattr(revision_acteur, field.name, getattr(acteur, field.name))
+        revision_acteur.save()
+        revision_acteur.labels.set(self.labels.all())
+        revision_acteur.acteur_services.set(self.acteur_services.all())
 
-        # recreate proposition_services for the new self
-        for proposition_service in proposition_services:
+        # recreate proposition_services for the new revision_acteur
+        for proposition_service in self.proposition_services.all():
             revision_proposition_service = revision_proposition_service = (
                 RevisionPropositionService.objects.create(
-                    acteur=self,
+                    acteur=revision_acteur,
                     action=proposition_service.action,
                 )
             )
             revision_proposition_service.sous_categories.set(
                 proposition_service.sous_categories.all()
             )
-        return self
+        return revision_acteur
 
     def __str__(self):
         return (