Duplicate action : deassociate documents

2 files changed, 6 insertions, 0 deletions

diff --git a/archaeological_finds/models_finds.py b/archaeological_finds/models_finds.py
index 8d4c65f1a..157841ab7 100644
--- a/archaeological_finds/models_finds.py
+++ b/archaeological_finds/models_finds.py
@@ -1999,6 +1999,8 @@ class Find(BulkUpdatedItem, ValueGetter, DocumentItem, BaseHistorizedItem,
             if bf:
                 new.base_finds.add(bf.duplicate(
                     user=user, data={"label": new.label, "external_id": None}))
+            # remove documents for this kind of duplicate (data entry)
+            new.documents.clear()
         return new
 
     @classmethod
diff --git a/archaeological_finds/tests.py b/archaeological_finds/tests.py
index 178a5f035..e39ebda32 100644
--- a/archaeological_finds/tests.py
+++ b/archaeological_finds/tests.py
@@ -1211,6 +1211,9 @@ class FindQATest(FindInit, TestCase):
         default_desc = "Description for duplicate"
         find.description = default_desc
         find.save()
+        d = Document.objects.create()
+        d.finds.add(find)
+
         c = Client()
         url = reverse('find-qa-duplicate', args=[find.pk])
         response = c.get(url)
@@ -1241,6 +1244,7 @@ class FindQATest(FindInit, TestCase):
         new_bf = models.BaseFind.objects.order_by('-pk').all()[0]
         base_bf = find.get_first_base_find()
         self.assertEqual(new_bf.context_record, base_bf.context_record)
+        self.assertNotIn(d, list(new.documents.all()))
 
     def test_bulk_update(self):
         c = Client()