api/utils/serializer/v1/compensation.py

@@ -11,7 +11,7 @@ from api.utils.serializer.v1.serializer import AbstractModelAPISerializerV1, Abs
 from compensation.models import Compensation
 from intervention.models import Intervention
 from konova.models import Geometry
-from konova.tasks import celery_update_parcels, celery_check_for_geometry_conflicts
+from konova.tasks import celery_update_parcels
 from konova.utils.message_templates import DATA_UNSHARED
 from user.models import UserActionLogEntry
 
@@ -128,7 +128,6 @@ class CompensationAPISerializerV1(AbstractModelAPISerializerV1, AbstractCompensa
         obj.log.add(obj.created)
 
         celery_update_parcels.delay(obj.geometry.id)
-        celery_check_for_geometry_conflicts.delay(obj.geometry.id)
 
         return obj.id
 

api/utils/serializer/v1/ecoaccount.py

@@ -13,7 +13,7 @@ from codelist.settings import CODELIST_CONSERVATION_OFFICE_ID, CODELIST_HANDLER_
 from compensation.models import EcoAccount
 from intervention.models import Legal, Responsibility, Handler
 from konova.models import Geometry
-from konova.tasks import celery_update_parcels, celery_check_for_geometry_conflicts
+from konova.tasks import celery_update_parcels
 from user.models import UserActionLogEntry
 
 
@@ -150,7 +150,6 @@ class EcoAccountAPISerializerV1(AbstractModelAPISerializerV1,
         obj.users.add(user)
 
         celery_update_parcels.delay(obj.geometry.id)
-        celery_check_for_geometry_conflicts.delay(obj.geometry.id)
 
         return obj.id
 

api/utils/serializer/v1/ema.py

@@ -13,7 +13,7 @@ from codelist.settings import CODELIST_CONSERVATION_OFFICE_ID, CODELIST_HANDLER_
 from ema.models import Ema
 from intervention.models import Responsibility, Handler
 from konova.models import Geometry
-from konova.tasks import celery_update_parcels, celery_check_for_geometry_conflicts
+from konova.tasks import celery_update_parcels
 from user.models import UserActionLogEntry
 
 
@@ -122,7 +122,6 @@ class EmaAPISerializerV1(AbstractModelAPISerializerV1, AbstractCompensationAPISe
         obj.users.add(user)
 
         celery_update_parcels.delay(obj.geometry.id)
-        celery_check_for_geometry_conflicts.delay(obj.geometry.id)
 
         return obj.id
 

api/utils/serializer/v1/intervention.py

@@ -13,7 +13,7 @@ from api.utils.serializer.v1.serializer import AbstractModelAPISerializerV1, \
 from compensation.models import Payment
 from intervention.models import Intervention, Responsibility, Legal, Handler
 from konova.models import Geometry
-from konova.tasks import celery_update_parcels, celery_check_for_geometry_conflicts
+from konova.tasks import celery_update_parcels
 from user.models import UserActionLogEntry
 
 
@@ -165,7 +165,6 @@ class InterventionAPISerializerV1(AbstractModelAPISerializerV1,
         obj.log.add(obj.created)
 
         celery_update_parcels.delay(obj.geometry.id)
-        celery_check_for_geometry_conflicts.delay(obj.geometry.id)
 
         return obj.id
 

api/utils/serializer/v1/serializer.py

@@ -392,8 +392,7 @@ class AbstractCompensationAPISerializerV1Mixin:
                 self._konova_code_from_json(e, CODELIST_COMPENSATION_ACTION_DETAIL_ID) for e in entry["action_details"]
             ]
             amount = float(entry["amount"])
-            # Mapping of old "qm" into "m²"
+            unit = entry["unit"]
-            unit = UnitChoices.m2.value if entry["unit"] == "qm" else entry["unit"]
             comment = entry["comment"]
 
             # Check on validity

konova/forms/geometry_form.py

@@ -15,7 +15,7 @@ from django.utils.translation import gettext_lazy as _
 
 from konova.forms.base_form import BaseForm
 from konova.models import Geometry
-from konova.tasks import celery_update_parcels, celery_check_for_geometry_conflicts
+from konova.tasks import celery_update_parcels
 from konova.sub_settings.lanis_settings import DEFAULT_SRID_RLP
 from user.models import UserActionLogEntry
 
@@ -82,12 +82,7 @@ class SimpleGeomForm(BaseForm):
             "MultiPolygon25D",
         ]
         for feature in features_json:
-            feature_geom = feature.get("geometry", feature)
+            feature_geom = json.dumps(feature.get("geometry", feature))
-            if feature_geom is None:
-                # Fallback for rare cases where a feature does not contain any geometry
-                continue
-
-            feature_geom = json.dumps(feature_geom)
             g = gdal.OGRGeometry(feature_geom, srs=DEFAULT_SRID_RLP)
 
             flatten_geometry = g.coord_dim > 2
@@ -146,9 +141,8 @@ class SimpleGeomForm(BaseForm):
                 geom=self.cleaned_data.get("geom", MultiPolygon(srid=DEFAULT_SRID_RLP)),
                 created=action,
             )
-        # Start parcel update and geometry conflict checking procedure in a background process
+        # Start the parcel update procedure in a background process
         celery_update_parcels.delay(geometry.id)
-        celery_check_for_geometry_conflicts.delay(geometry.id)
         return geometry
 
     def __flatten_geom_to_2D(self, geom):

konova/models/geometry.py

@@ -8,11 +8,13 @@ Created on: 15.11.21
 import json
 
 from django.contrib.gis.db.models import MultiPolygonField
+from django.contrib.gis.geos import Polygon
 from django.db import models, transaction
 from django.utils import timezone
 
 from konova.models import BaseResource, UuidModel
 from konova.sub_settings.lanis_settings import DEFAULT_SRID_RLP
+from konova.tasks import celery_check_for_geometry_conflicts
 from konova.utils.wfs.spatial import ParcelWFSFetcher
 
 
@@ -27,18 +29,7 @@ class Geometry(BaseResource):
 
     def save(self, *args, **kwargs):
         super().save(*args, **kwargs)
-
+        celery_check_for_geometry_conflicts.delay(self.id)
-    @property
-    def geom_small_buffered(self):
-        """
-        Returns a smaller buffered version of the geometry.
-        Can be used to shrink the geometry used for intersection purposes to avoid intersection detection on
-        neighbouring geometries.
-
-        Returns:
-
-        """
-        return self.geom.buffer(-0.001)
 
     def check_for_conflicts(self):
         """ Checks for new geometry overlaps
@@ -53,8 +44,9 @@ class Geometry(BaseResource):
             return None
 
         self.recheck_existing_conflicts()
+
         overlapping_geoms = Geometry.objects.filter(
-            geom__intersects=self.geom_small_buffered,
+            geom__intersects=self.geom,
         ).exclude(
             id=self.id
         ).distinct()
@@ -76,14 +68,14 @@ class Geometry(BaseResource):
         """
         all_conflicts_as_conflicting = self.conflicts_geometries.all()
         still_conflicting_conflicts = all_conflicts_as_conflicting.filter(
-            affected_geometry__geom__intersects=self.geom_small_buffered
+            affected_geometry__geom__intersects=self.geom
         )
         resolved_conflicts = all_conflicts_as_conflicting.exclude(id__in=still_conflicting_conflicts)
         resolved_conflicts.delete()
 
         all_conflicted_by_conflicts = self.conflicted_by_geometries.all()
         still_conflicting_conflicts = all_conflicted_by_conflicts.filter(
-            conflicting_geometry__geom__intersects=self.geom_small_buffered
+            conflicting_geometry__geom__intersects=self.geom
         )
         resolved_conflicts = all_conflicted_by_conflicts.exclude(id__in=still_conflicting_conflicts)
         resolved_conflicts.delete()

konova/utils/wfs/spatial.py

@@ -91,7 +91,7 @@ class ParcelWFSFetcher(AbstractWFSFetcher):
         geom = Geometry.objects.filter(
             id=self.geometry_id
         ).annotate(
-            smaller=Func(F('geom'), -0.001, function="ST_Buffer")  # same as geometry.geom_small_buffered but for QuerySet
+            smaller=Func(F('geom'), -0.001, function="ST_Buffer")
         ).annotate(
             gml=AsGML(MakeValid('smaller'))
         ).first()

locale/de/LC_MESSAGES/django.po

@@ -2307,8 +2307,8 @@ msgid "Server Error"
 msgstr ""
 
 #: templates/500.html:10
-msgid "Something happened. Admins have been informed. We are working on it!"
+msgid "Something happened. We are working on it!"
-msgstr "Irgendetwas ist passiert. Die Administratoren wurden informiert. Wir arbeiten daran!"
+msgstr "Irgendetwas ist passiert. Wir arbeiten daran!"
 
 #: templates/email/api/verify_token.html:7
 msgid "Hello support"

templates/500.html

@@ -7,7 +7,7 @@
         <h1 class="display-4">{% trans 'Server Error' %}</h1>
         <hr>
         <p class="lead">
-            {% trans 'Something happened. Admins have been informed. We are working on it!' %}
+            {% trans 'Something happened. We are working on it!' %}
         </p>
     </div>
 {% endblock %}