Ensure all models have an ordering

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

vulnerabilities/migrations/0027_alter_all_models_to_add_ordering.py

@@ -0,0 +1,37 @@
+# Generated by Django 4.0.7 on 2022-09-08 14:51
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('vulnerabilities', '0026_alter_package_purl'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='advisory',
+            options={'ordering': ['aliases', 'date_published', 'unique_content_id']},
+        ),
+        migrations.AlterModelOptions(
+            name='package',
+            options={'ordering': ['purl']},
+        ),
+        migrations.AlterModelOptions(
+            name='packagerelatedvulnerability',
+            options={'ordering': ['package', 'vulnerability'], 'verbose_name_plural': 'PackageRelatedVulnerabilities'},
+        ),
+        migrations.AlterModelOptions(
+            name='vulnerability',
+            options={'ordering': ['vulnerability_id'], 'verbose_name_plural': 'Vulnerabilities'},
+        ),
+        migrations.AlterModelOptions(
+            name='vulnerabilityrelatedreference',
+            options={'ordering': ['vulnerability', 'reference']},
+        ),
+        migrations.AlterModelOptions(
+            name='vulnerabilityseverity',
+            options={'ordering': ['reference', 'scoring_system', 'value']},
+        ),
+    ]

vulnerabilities/models.py

@@ -60,6 +60,7 @@ class Vulnerability(models.Model):
 
     class Meta:
         verbose_name_plural = "Vulnerabilities"
+        ordering = ["vulnerability_id"]
 
     def __str__(self):
         return self.vulnerability_id
@@ -118,10 +119,7 @@ class VulnerabilityReference(models.Model):
     )
 
     class Meta:
-        unique_together = (
-            "url",
-            "reference_id",
-        )
+        unique_together = ["url", "reference_id"]
         ordering = ["reference_id", "url"]
 
     def __str__(self):
@@ -145,7 +143,8 @@ class VulnerabilityRelatedReference(models.Model):
     )
 
     class Meta:
-        unique_together = ("vulnerability", "reference")
+        unique_together = ["vulnerability", "reference"]
+        ordering = ["vulnerability", "reference"]
 
 
 class Package(PackageURLMixin):
@@ -174,14 +173,8 @@ def purl(self):
         return self.package_url
 
     class Meta:
-        unique_together = (
-            "type",
-            "namespace",
-            "name",
-            "version",
-            "qualifiers",
-            "subpath",
-        )
+        unique_together = ["type", "namespace", "name", "version", "qualifiers", "subpath"]
+        ordering = ["purl"]
 
     def __str__(self):
         return self.package_url
@@ -261,9 +254,10 @@ class PackageRelatedVulnerability(models.Model):
     )
 
     class Meta:
-        unique_together = ("package", "vulnerability")
+        unique_together = ["package", "vulnerability"]
         verbose_name_plural = "PackageRelatedVulnerabilities"
         indexes = [models.Index(fields=["fix"])]
+        ordering = ["package", "vulnerability"]
 
     def __str__(self):
         return f"{self.package.package_url} {self.vulnerability.vulnerability_id}"
@@ -324,11 +318,8 @@ class VulnerabilitySeverity(models.Model):
     value = models.CharField(max_length=50, help_text="Example: 9.0, Important, High")
 
     class Meta:
-        unique_together = (
-            "reference",
-            "scoring_system",
-            "value",
-        )
+        unique_together = ["reference", "scoring_system", "value"]
+        ordering = ["reference", "scoring_system", "value"]
 
 
 class Alias(models.Model):
@@ -414,11 +405,8 @@ class Advisory(models.Model):
     )
 
     class Meta:
-        unique_together = (
-            "aliases",
-            "unique_content_id",
-            "date_published",
-        )
+        unique_together = ["aliases", "unique_content_id", "date_published"]
+        ordering = ["aliases", "date_published", "unique_content_id"]
 
     def save(self, *args, **kwargs):
         checksum = hashlib.md5()