Drop status column from media reports

api/api/migrations/0068_remove_audioreport_status_remove_imagereport_status.py

@@ -0,0 +1,21 @@
+# Generated by Django 4.2.11 on 2024-06-28 19:53
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('api', '0067_alter_audiodecisionthrough_media_obj_and_more'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='audioreport',
+            name='status',
+        ),
+        migrations.RemoveField(
+            model_name='imagereport',
+            name='status',
+        ),
+    ]

api/api/models/media.py

@@ -191,13 +191,6 @@ class AbstractMediaReport(models.Model):
 
     REPORT_CHOICES = [(MATURE, MATURE), (DMCA, DMCA), (OTHER, OTHER)]
 
-    STATUS_CHOICES = [
-        (PENDING, PENDING),
-        (MATURE_FILTERED, MATURE_FILTERED),
-        (DEINDEXED, DEINDEXED),
-        (NO_ACTION, NO_ACTION),
-    ]
-
     created_at = models.DateTimeField(auto_now_add=True)
 
     media_obj = models.ForeignKey(
@@ -226,12 +219,6 @@ class AbstractMediaReport(models.Model):
         null=True,
         help_text="The explanation on why media is being reported.",
     )
-    status = models.CharField(max_length=20, choices=STATUS_CHOICES, default=PENDING)
-    """
-    All statuses except ``PENDING`` are deprecated. Instead refer to the
-    property ``is_pending``.
-    """
-
     decision = models.ForeignKey(
         to="AbstractMediaDecision",
         on_delete=models.SET_NULL,

api/latest_migrations/api

@@ -2,4 +2,4 @@
 # If you have a merge conflict in this file, it means you need to run:
 #     manage.py makemigrations --merge
 # in order to resolve the conflict between migrations.
-0067_alter_audiodecisionthrough_media_obj_and_more
+0068_remove_audioreport_status_remove_imagereport_status

api/test/unit/models/test_media_report.py

@@ -13,7 +13,6 @@
     MATURE,
     MATURE_FILTERED,
     OTHER,
-    PENDING,
     AbstractDeletedMedia,
     AbstractSensitiveMedia,
 )
@@ -41,7 +40,7 @@ def test_pending_reports_have_no_subreport_models(
     media = media_type_config.model_factory.create()
     report = media_type_config.report_factory.create(media_obj=media, reason=reason)
 
-    assert report.status == PENDING
+    assert report.decision is None
     assert not media_type_config.sensitive_class.objects.filter(
         media_obj=media
     ).exists()