konova/management/commands/recalculate_parcels.py

@@ -34,9 +34,7 @@ class Command(BaseKonovaCommand):
     def recalculate_parcels(self, options: dict):
         force_all = options.get("force_all", False)
 
-        geometry_objects = Geometry.objects.all().exclude(
+        geometry_objects = Geometry.objects.all()
-            geom=None
-        )
 
         if not force_all:
             # Fetch all intersections
@@ -48,17 +46,16 @@ class Command(BaseKonovaCommand):
                 "geometry__id",
                 flat=True
             )
-            intersected_geom_objs = Geometry.objects.filter(
+            # Filter those geometries out (they have intersections and do not need to be processed)
+            geometry_objects = geometry_objects.exclude(
                 id__in=geom_with_intersection_ids
             )
-            # Filter those geometries out (they have intersections and do not need to be processed)
-            geometry_objects = geometry_objects.difference(intersected_geom_objs)
 
         self._write_warning("=== Update parcels and districts ===")
         # Order geometries by size to process smaller once at first
-        geometries = geometry_objects.annotate(
+        geometries = geometry_objects.exclude(
-            area=Area("geom")
+            geom=None
-        ).order_by(
+        ).annotate(area=Area("geom")).order_by(
             'area'
         )
         self._write_warning(f"Process parcels for {geometries.count()} geometry entries now ...")

konova/models/parcel.py

@@ -159,6 +159,7 @@ class Parcel(UuidModel):
         return f"{self.parcel_group} | {self.flr} | {self.flrstck_zhlr} | {self.flrstck_nnr}"
 
     @classmethod
+    @transaction.atomic
     def make_unique(cls, **kwargs):
         """ Checks for duplicates of a Parcel, choose a (now) unique one,
         repairs relations for ParcelIntersection and removes duplicates.
@@ -187,7 +188,6 @@ class Parcel(UuidModel):
         intersection_objs = ParcelIntersection.objects.filter(
             parcel__in=parcel_objs
         )
-
         # Change each intersection, so they point on the 'true one' parcel from now on
         for intersection in intersection_objs:
             intersection.parcel = unique_parcel