refactor is published to status

Author: sudip-khanal

api/test_views.py

@@ -89,11 +89,11 @@ def test_guest_user_permission(self):
         go_post_apis = [
             "/api/v2/dref/",
             "/api/v2/dref-final-report/",
-            f"/api/v2/dref-final-report/{id}/publish/",
+            f"/api/v2/dref-final-report/{id}/approve/",
             "/api/v2/dref-op-update/",
-            f"/api/v2/dref-op-update/{id}/publish/",
+            f"/api/v2/dref-op-update/{id}/approve/",
             "/api/v2/dref-share/",
-            f"/api/v2/dref/{id}/publish/",
+            f"/api/v2/dref/{id}/approve/",
             "/api/v2/flash-update/",
             "/api/v2/flash-update-file/multiple/",
             "/api/v2/local-units/",

dref/admin.py

@@ -230,7 +230,7 @@ def get_queryset(self, request):
 
     # NOTE: If the Dref Final report is unpublished, set Dref related to it as active
     def save_model(self, request, obj, form, change):
-        if not obj.is_published and obj.dref:
+        if obj.status != Dref.Status.APPROVED and obj.dref:
             obj.dref.is_active = True
             obj.dref.save(update_fields=["is_active"])
         super().save_model(request, obj, form, change)

dref/filter_set.py

@@ -15,15 +15,15 @@ class DrefFilter(filters.FilterSet):
 
     class Meta:
         model = Dref
-        fields = ["is_published"]
+        fields = ["status"]
 
 
 class DrefOperationalUpdateFilter(filters.FilterSet):
     dref = filters.ModelMultipleChoiceFilter(field_name="dref", queryset=Dref.objects.all().distinct())
 
     class Meta:
         model = DrefOperationalUpdate
-        fields = ["is_published"]
+        fields = ["status"]
 
 
 class BaseDrefFilterSet(filters.FilterSet):

dref/migrations/0084_alter_dref_status_alter_dreffinalreport_status_and_more.py renamed to dref/migrations/0084_dref_original_language_and_more.py

@@ -1,4 +1,4 @@
-# Generated by Django 4.2.19 on 2025-09-10 08:48
+# Generated by Django 4.2.19 on 2025-09-16 09:09
 
 from django.db import migrations, models
 
@@ -28,34 +28,55 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
+        migrations.AddField(
+            model_name="dref",
+            name="original_language",
+            field=models.CharField(
+                blank=True,
+                help_text="The language in which this record was first created.",
+                null=True,
+                verbose_name="Original language",
+            ),
+        ),
+        migrations.AddField(
+            model_name="dreffinalreport",
+            name="original_language",
+            field=models.CharField(
+                blank=True,
+                help_text="The language in which this record was first created.",
+                null=True,
+                verbose_name="Original language",
+            ),
+        ),
+        migrations.AddField(
+            model_name="drefoperationalupdate",
+            name="original_language",
+            field=models.CharField(
+                blank=True,
+                help_text="The language in which this record was first created.",
+                null=True,
+                verbose_name="Original language",
+            ),
+        ),
         migrations.AlterField(
             model_name="dref",
             name="status",
             field=models.IntegerField(
-                blank=True,
-                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")],
-                null=True,
-                verbose_name="status",
+                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")], default=1, verbose_name="status"
             ),
         ),
         migrations.AlterField(
             model_name="dreffinalreport",
             name="status",
             field=models.IntegerField(
-                blank=True,
-                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")],
-                null=True,
-                verbose_name="status",
+                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")], default=1, verbose_name="status"
             ),
         ),
         migrations.AlterField(
             model_name="drefoperationalupdate",
             name="status",
             field=models.IntegerField(
-                blank=True,
-                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")],
-                null=True,
-                verbose_name="status",
+                choices=[(1, "Draft"), (2, "Finalizing"), (3, "Finalized"), (4, "Approved")], default=1, verbose_name="status"
             ),
         ),
         migrations.RunPython(update_status, reverse_code=migrations.RunPython.noop),

dref/migrations/0085_remove_dref_is_published_and_more.py

@@ -0,0 +1,25 @@
+# Generated by Django 4.2.19 on 2025-09-16 09:13
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("dref", "0084_dref_original_language_and_more"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="dref",
+            name="is_published",
+        ),
+        migrations.RemoveField(
+            model_name="dreffinalreport",
+            name="is_published",
+        ),
+        migrations.RemoveField(
+            model_name="drefoperationalupdate",
+            name="is_published",
+        ),
+    ]

dref/models.py

@@ -264,13 +264,14 @@ class DisasterCategory(models.IntegerChoices):
         RED = 2, _("Red")
 
     class Status(models.IntegerChoices):
-        # NOTE: Updated statuses for clarity:
-        # IN_PROGRESS → DRAFT, COMPLETED → APPROVED.
-        # Added FINALIZING and FINALIZED for translation/finalization.
         DRAFT = 1, _("Draft")
+        """Draft: Initial stage content is being created and is not ready for review."""
         FINALIZING = 2, _("Finalizing")
+        """Finalizing: Content is in the translation process from the original language into English."""
         FINALIZED = 3, _("Finalized")
+        """Finalized: Translation is completed, content is ready for review, and updates to the original language are locked."""
         APPROVED = 4, _("Approved")
+        """Approved: The content has been reviewed, accepted, and is ready for use."""
 
     created_at = models.DateTimeField(verbose_name=_("created at"), auto_now_add=True)
     modified_at = models.DateTimeField(verbose_name=_("modified at"), default=timezone.now, null=True)
@@ -334,7 +335,13 @@ class Status(models.IntegerChoices):
         verbose_name=_("If available please upload additional support documentation for targeting strategy"),
         related_name="dref_targeting_strategy_support_file",
     )
-    status = models.IntegerField(choices=Status.choices, verbose_name=_("status"), null=True, blank=True)
+    status = models.IntegerField(choices=Status.choices, verbose_name=_("status"), default=Status.DRAFT)
+    original_language = models.CharField(
+        blank=True,
+        null=True,
+        verbose_name=_("Original language"),
+        help_text="The language in which this record was first created.",
+    )  # NOTE: This field is set at creation with the active language.
     num_assisted = models.IntegerField(verbose_name=_("number of assisted"), blank=True, null=True)
     num_affected = models.IntegerField(verbose_name=_("number of affected"), blank=True, null=True)
     estimated_number_of_affected_male = models.IntegerField(
@@ -650,10 +657,6 @@ class Status(models.IntegerChoices):
         verbose_name=_("cover image"),
         related_name="cover_image_dref",
     )
-    is_published = models.BooleanField(
-        default=False,
-        verbose_name=_("Is published"),
-    )
     is_final_report_created = models.BooleanField(
         default=False,
         verbose_name=_("Is final report created"),
@@ -862,7 +865,13 @@ class DrefOperationalUpdate(models.Model):
     disaster_category = models.IntegerField(
         choices=Dref.DisasterCategory.choices, verbose_name=_("disaster category"), null=True, blank=True
     )
-    status = models.IntegerField(choices=Dref.Status.choices, verbose_name=_("status"), null=True, blank=True)
+    status = models.IntegerField(choices=Dref.Status.choices, verbose_name=_("status"), default=Dref.Status.DRAFT)
+    original_language = models.CharField(
+        blank=True,
+        null=True,
+        verbose_name=_("Original language"),
+        help_text="The language in which this record was first created.",
+    )  # NOTE: This field is set at creation with the active language.
     number_of_people_targeted = models.IntegerField(verbose_name=_("Number of people targeted"), blank=True, null=True)
     number_of_people_affected = models.IntegerField(verbose_name=_("number of people affected"), blank=True, null=True)
     estimated_number_of_affected_male = models.IntegerField(
@@ -1097,10 +1106,6 @@ class DrefOperationalUpdate(models.Model):
         null=True,
     )
     planned_interventions = models.ManyToManyField(PlannedIntervention, verbose_name=_("planned intervention"), blank=True)
-    is_published = models.BooleanField(
-        default=False,
-        verbose_name=_("Is published"),
-    )
     country = models.ForeignKey(
         Country,
         verbose_name=_("country"),
@@ -1299,7 +1304,13 @@ class DrefFinalReport(models.Model):
     disaster_category = models.IntegerField(
         choices=Dref.DisasterCategory.choices, verbose_name=_("disaster category"), null=True, blank=True
     )
-    status = models.IntegerField(choices=Dref.Status.choices, verbose_name=_("status"), null=True, blank=True)
+    status = models.IntegerField(choices=Dref.Status.choices, verbose_name=_("status"), default=Dref.Status.DRAFT)
+    original_language = models.CharField(
+        blank=True,
+        null=True,
+        verbose_name=_("Original language"),
+        help_text="The language in which this record was first created.",
+    )  # NOTE: This field is set at creation with the active language.
     number_of_people_targeted = models.IntegerField(verbose_name=_("Number of people targeted"), blank=True, null=True)
     number_of_people_affected = models.IntegerField(verbose_name=_("number of people affected"), blank=True, null=True)
     estimated_number_of_affected_male = models.IntegerField(
@@ -1483,7 +1494,6 @@ class DrefFinalReport(models.Model):
         verbose_name=_("Additional National Societies Actions"), null=True, blank=True
     )
     planned_interventions = models.ManyToManyField(PlannedIntervention, verbose_name=_("planned intervention"), blank=True)
-    is_published = models.BooleanField(verbose_name=_("Is Published"), default=False)
     country = models.ForeignKey(
         Country,
         verbose_name=_("country"),
@@ -1657,7 +1667,7 @@ def save(self, *args, **kwargs):
         super().save(*args, **kwargs)
 
     @staticmethod
-    def get_for(user, is_published=False):
+    def get_for(user, status=None):
         from dref.utils import get_dref_users
 
         # get the user in dref
@@ -1676,6 +1686,6 @@ def get_for(user, is_published=False):
         final_report_created_by = DrefFinalReport.objects.filter(created_by=user).distinct()
         union_query = final_report_users.union(final_report_created_by)
         queryset = DrefFinalReport.objects.filter(id__in=union_query.values("id")).distinct()
-        if is_published:
-            return queryset.filter(is_published=True)
+        if status == Dref.Status.APPROVED:
+            return queryset.filter(status=Dref.Status.APPROVED)
         return queryset

dref/permissions.py

@@ -1,7 +1,7 @@
 from django.contrib.auth.models import Permission
 from rest_framework import permissions
 
-from dref.models import DrefFinalReport, DrefOperationalUpdate
+from dref.models import Dref, DrefFinalReport, DrefOperationalUpdate
 from dref.utils import get_dref_users
 
 
@@ -42,6 +42,6 @@ def has_object_permission(self, request, view, obj):
         region = obj.country.region.name
         codename = f"dref_region_admin_{region}"
         user = request.user
-        if Permission.objects.filter(user=user, codename=codename).exists() and not obj.is_published:
+        if Permission.objects.filter(user=user, codename=codename).exists() and obj.status != Dref.Status.APPROVED:
             return True
         return False