From 8896cb7dc9d760e4b3769b72c6af83b8a0b81eda Mon Sep 17 00:00:00 2001 From: Étienne Loks Date: Sun, 24 Jan 2021 13:08:31 +0100 Subject: Container: prevent parent association to himself --- archaeological_warehouse/models.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'archaeological_warehouse/models.py') diff --git a/archaeological_warehouse/models.py b/archaeological_warehouse/models.py index 5d06cefda..e67fe2bd2 100644 --- a/archaeological_warehouse/models.py +++ b/archaeological_warehouse/models.py @@ -1357,6 +1357,8 @@ class Container(DocumentItem, Merge, LightHistorizedItem, return actions def pre_save(self): + if self.parent == self: + self.parent = None if not self.collection_id and not self.collection: if self.location_id: self.collection_id = self.location_id @@ -1431,10 +1433,11 @@ class Container(DocumentItem, Merge, LightHistorizedItem, def container_post_save(sender, **kwargs): cached_label_and_geo_changed(sender=sender, **kwargs) + #TODO: to be deleted??? + """ if not kwargs.get('instance'): return instance = kwargs.get('instance') - #TODO: to be deleted??? for loca in ContainerLocalisation.objects.filter( container=instance).exclude( division__warehouse=instance.location).all(): @@ -1446,6 +1449,7 @@ def container_post_save(sender, **kwargs): continue loca.division = q.all()[0] loca.save() + """ def container_pre_delete(sender, **kwargs): -- cgit v1.2.3