Change unique_together to use constraints, replace use of index_together

code_check.py

@@ -39,7 +39,7 @@ def status(line):
     cmd("black dash test_runner")
 
     status("Running ruff")
-    cmd("ruff dash")
+    cmd("ruff check dash")
 
     status("Running isort")
     cmd("isort dash")

dash/categories/migrations/0011_alter_category_unique_together_and_more.py

@@ -0,0 +1,26 @@
+# Generated by Django 5.1 on 2024-08-21 12:22
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("categories", "0010_alter_category_created_by_alter_category_modified_by_and_more"),
+        ("orgs", "0033_alter_orgbackend_unique_together_and_more"),
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name="category",
+            unique_together=set(),
+        ),
+        migrations.AddConstraint(
+            model_name="category",
+            constraint=models.UniqueConstraint(
+                fields=("name", "org"), name="categories_category_name_318a92307e6f39fb_uniq"
+            ),
+        ),
+    ]

dash/categories/models.py

@@ -50,7 +50,9 @@ def __str__(self):
 
     class Meta:
         ordering = ["name"]
-        unique_together = ("name", "org")
+        constraints = [
+            models.UniqueConstraint(fields=["name", "org"], name="categories_category_name_318a92307e6f39fb_uniq")
+        ]
         verbose_name_plural = _("Categories")
 
 

dash/dashblocks/migrations/0011_alter_dashblock_index_together_and_more.py

@@ -1,6 +1,6 @@
 # Generated by Django 4.1.7 on 2023-03-29 14:58
 
-from django.db import migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
@@ -10,12 +10,13 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.AlterIndexTogether(
-            name="dashblock",
-            index_together={("org", "is_active", "dashblock_type", "priority")},
+        migrations.AddIndex(
+            model_name="dashblock",
+            index=models.Index(
+                fields=["org", "is_active", "dashblock_type", "priority"], name="dashblocks__org_id_024805_idx"
+            ),
         ),
-        migrations.AlterIndexTogether(
-            name="dashblocktype",
-            index_together={("slug", "name")},
+        migrations.AddIndex(
+            model_name="dashblocktype", index=models.Index(fields=["slug", "name"], name="dashblocks__slug_c0c6c6_idx")
         ),
     ]

dash/dashblocks/models.py

@@ -49,7 +49,7 @@ def __str__(self):
 
     class Meta:
         ordering = ["name"]
-        indexes = [models.Index(fields=["slug", "name"])]
+        indexes = [models.Index(fields=["slug", "name"], name="dashblocks__slug_c0c6c6_idx")]
 
 
 class DashBlock(SmartModel):
@@ -149,7 +149,11 @@ def __str__(self):
 
     class Meta:
         ordering = ["dashblock_type", "title"]
-        indexes = [models.Index(fields=["org", "is_active", "dashblock_type", "priority"])]
+        indexes = [
+            models.Index(
+                fields=["org", "is_active", "dashblock_type", "priority"], name="dashblocks__org_id_024805_idx"
+            )
+        ]
 
 
 class DashBlockImage(SmartModel):

dash/orgs/migrations/0031_alter_orgbackend_index_together.py

@@ -1,6 +1,6 @@
 # Generated by Django 4.1.7 on 2023-03-29 13:57
 
-from django.db import migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
@@ -9,8 +9,8 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.AlterIndexTogether(
-            name="orgbackend",
-            index_together={("org", "is_active", "slug")},
-        ),
+        migrations.AddIndex(
+            model_name="orgbackend",
+            index=models.Index(fields=["org", "is_active", "slug"], name="orgs_orgbac_org_id_607508_idx"),
+        )
     ]

dash/orgs/migrations/0033_alter_orgbackend_unique_together_and_more.py

@@ -0,0 +1,35 @@
+# Generated by Django 5.1 on 2024-08-21 12:22
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("orgs", "0032_rename_orgbackend_org_is_active_slug_orgs_orgbac_org_id_607508_idx"),
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name="orgbackend",
+            unique_together=set(),
+        ),
+        migrations.AlterUniqueTogether(
+            name="taskstate",
+            unique_together=set(),
+        ),
+        migrations.AddConstraint(
+            model_name="orgbackend",
+            constraint=models.UniqueConstraint(
+                fields=("org", "slug"), name="orgs_orgbackend_org_id_slug_03bef4c1_uniq"
+            ),
+        ),
+        migrations.AddConstraint(
+            model_name="taskstate",
+            constraint=models.UniqueConstraint(
+                fields=("org", "task_key"), name="orgs_taskstate_org_id_70fc0194720c6cf5_uniq"
+            ),
+        ),
+    ]

dash/orgs/models.py

@@ -397,7 +397,9 @@ def get_time_taken(self):
         return (until - self.started_on).total_seconds()
 
     class Meta:
-        unique_together = ("org", "task_key")
+        constraints = [
+            models.UniqueConstraint(fields=["org", "task_key"], name="orgs_taskstate_org_id_70fc0194720c6cf5_uniq")
+        ]
 
 
 class OrgBackend(SmartModel):
@@ -415,5 +417,7 @@ def __str__(self):
         return self.slug
 
     class Meta:
-        unique_together = ("org", "slug")
+        constraints = [
+            models.UniqueConstraint(fields=["org", "slug"], name="orgs_orgbackend_org_id_slug_03bef4c1_uniq")
+        ]
         indexes = [models.Index(fields=["org", "is_active", "slug"])]

dash/tags/migrations/0003_alter_tag_unique_together_and_more.py

@@ -0,0 +1,24 @@
+# Generated by Django 5.1 on 2024-08-21 12:22
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("orgs", "0033_alter_orgbackend_unique_together_and_more"),
+        ("tags", "0002_alter_tag_created_by_alter_tag_modified_by"),
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name="tag",
+            unique_together=set(),
+        ),
+        migrations.AddConstraint(
+            model_name="tag",
+            constraint=models.UniqueConstraint(fields=("name", "org"), name="tags_tag_name_org_id_e64e146b_uniq"),
+        ),
+    ]

dash/tags/models.py

@@ -15,4 +15,4 @@ def __str__(self) -> str:
         return self.name
 
     class Meta:
-        unique_together = ("name", "org")
+        constraints = [models.UniqueConstraint(fields=["name", "org"], name="tags_tag_name_org_id_e64e146b_uniq")]

pyproject.toml

@@ -49,11 +49,13 @@ line-length = 119
 
 [tool.ruff]
 line-length = 120
-select = ["E", "F", "W"]
-ignore = ["E501", "F405"]
 fix = true
 exclude = ["./.tox/*", "./.venv/*", "./env/*", "*/migrations/*", "./build/*"]
 
+[tool.ruff.lint]
+select = ["E", "F", "W"]
+ignore = ["E501", "F405"]
+
 [tool.isort]
 multi_line_output = 3
 force_grid_wrap = 0