🎨 [#183] made internetaak serializer validate and model save method more precise in checking for status

Author: bart-maykin

src/openklant/components/klantinteracties/api/serializers/internetaken.py

@@ -72,16 +72,16 @@ class Meta:
 
     def validate(self, attrs):
         status = attrs.get("status", None)
-        if not status and self.instance:
+        if status is None and self.instance is not None:
             status = self.instance.status
 
         if attrs.get("afgehandeld_op") and status != Taakstatus.verwerkt:
             raise serializers.ValidationError(
                 {
                     "afgehandeld_op": _(
                         "De Internetaak kan geen afgehandeld op datum bevatten "
-                        "als de status nog in '{te_verwerken}' staat."
-                    ).format(te_verwerken=Taakstatus.te_verwerken.value)
+                        "als de status niet op '{verwerkt}' staat."
+                    ).format(verwerkt=Taakstatus.verwerkt)
                 }
             )
 

src/openklant/components/klantinteracties/api/tests/test_internetaken.py

@@ -147,7 +147,7 @@ def test_create_internetaak_with_status_te_verwerken(self):
             self.assertEqual(response_data["invalidParams"][0]["name"], "afgehandeldOp")
             self.assertEqual(
                 response_data["invalidParams"][0]["reason"],
-                "De Internetaak kan geen afgehandeld op datum bevatten als de status nog in 'te_verwerken' staat.",
+                "De Internetaak kan geen afgehandeld op datum bevatten als de status niet op 'verwerkt' staat.",
             )
 
     def test_create_internetaak_with_afgehandeld_op_date(self):
@@ -306,7 +306,7 @@ def test_update_internetaak(self):
             self.assertEqual(data["invalidParams"][0]["name"], "afgehandeldOp")
             self.assertEqual(
                 data["invalidParams"][0]["reason"],
-                "De Internetaak kan geen afgehandeld op datum bevatten als de status nog in 'te_verwerken' staat.",
+                "De Internetaak kan geen afgehandeld op datum bevatten als de status niet op 'verwerkt' staat.",
             )
 
     def test_partial_update_internetaak(self):

src/openklant/components/klantinteracties/models/internetaken.py

@@ -88,10 +88,12 @@ class Meta:
 
     def save(self, *args, **kwargs):
         number_generator(self, InterneTaak)
-        if not self.afgehandeld_op and self.status == Taakstatus.verwerkt:
-            self.afgehandeld_op = timezone.now()
-        if self.afgehandeld_op and self.status == Taakstatus.te_verwerken:
-            self.afgehandeld_op = None
+        if self.afgehandeld_op is None:
+            if self.status == Taakstatus.verwerkt:
+                self.afgehandeld_op = timezone.now()
+        else:
+            if self.status == Taakstatus.te_verwerken:
+                self.afgehandeld_op = None
         return super().save(*args, **kwargs)
 
     def __str__(self):