#31 API DELETE

* adds support for DELETE method for all relevant objects
* improves get_obj_from_db functionality
* drops custom compensation logic for get_obj_from_db due to improvement of base method

api/utils/serializer/serializer.py

@@ -11,6 +11,8 @@ from abc import abstractmethod
 from django.contrib.gis import geos
 from django.contrib.gis.geos import GEOSGeometry
 
+from konova.utils.message_templates import DATA_UNSHARED
+
 
 class AbstractModelAPISerializer:
     model = None
@@ -66,7 +68,7 @@ class AbstractModelAPISerializer:
             # Return all objects
             del self.lookup["id"]
         else:
-            # Return certain objects
+            # Return certain object
             self.lookup["id"] = _id
         self.lookup["users__in"] = [user]
 
@@ -139,10 +141,14 @@ class AbstractModelAPISerializer:
         Returns:
 
         """
-        return self.model.objects.get(
+        obj = self.model.objects.get(
             id=id,
-            users__in=[user]
+            deleted__isnull=True,
         )
+        is_shared = obj.is_shared_with(user)
+        if not is_shared:
+            raise PermissionError(DATA_UNSHARED)
+        return obj
 
     @abstractmethod
     def initialize_objects(self, json_model, user):

api/utils/serializer/v1/compensation.py

@@ -129,23 +129,6 @@ class CompensationAPISerializerV1(AbstractModelAPISerializerV1, AbstractCompensa
 
             return obj.id
 
-    def get_obj_from_db(self, id, user):
-        """ Returns the object from database
-
-        Fails if id not found or user does not have shared access
-
-        Args:
-            id (str): The object's id
-            user (User): The API user
-
-        Returns:
-
-        """
-        return self.model.objects.get(
-            id=id,
-            intervention__users__in=[user]
-        )
-
     def update_model_from_json(self, id, json_model, user):
         """ Updates an entry for the model based on the contents of json_model
 

api/utils/serializer/v1/serializer.py

@@ -17,6 +17,7 @@ from codelist.settings import CODELIST_COMPENSATION_ACTION_ID, CODELIST_BIOTOPES
 from compensation.models import CompensationAction, UnitChoices, CompensationState
 from intervention.models import Responsibility, Legal
 from konova.models import Deadline, DeadlineType
+from konova.utils.message_templates import DATA_UNSHARED
 
 
 class AbstractModelAPISerializerV1(AbstractModelAPISerializer):
@@ -101,6 +102,27 @@ class AbstractModelAPISerializerV1(AbstractModelAPISerializer):
         modified_on = modified_on.timestamp if modified_on is not None else None
         return modified_on
 
+    def delete_entry(self, id, user):
+        """ Marks an entry as deleted
+
+        Args:
+            id (str): The entry's id
+            user (User): The API user
+
+        Returns:
+
+        """
+        entry = self.get_obj_from_db(id, user)
+        is_shared = entry.is_shared_with(user)
+        if not is_shared:
+            raise PermissionError(DATA_UNSHARED)
+        # Do not send mails if entry is deleting using API. THere could be hundreds of deletion resulting in hundreds of
+        # mails at once.
+        entry.mark_as_deleted(user, send_mail=False)
+        entry.refresh_from_db()
+        success = entry.deleted is not None
+        return success
+
 
 class DeductableAPISerializerV1Mixin:
     class Meta: