Merge pull request #558 from furlongm/module-uniqueness

use correct module uniqeness constraint

modules/migrations/0002_auto_20240204_2214.py

@@ -0,0 +1,33 @@
+# Generated by Django 3.2.23 on 2024-02-04 22:14
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('arch', '0001_initial'),
+        ('modules', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='module',
+            name='context',
+            field=models.CharField(max_length=255),
+        ),
+        migrations.AlterField(
+            model_name='module',
+            name='name',
+            field=models.CharField(max_length=255),
+        ),
+        migrations.AlterField(
+            model_name='module',
+            name='stream',
+            field=models.CharField(max_length=255),
+        ),
+        migrations.AlterUniqueTogether(
+            name='module',
+            unique_together={('name', 'stream', 'version', 'context', 'arch')},
+        ),
+    ]

modules/migrations/0003_alter_module_options.py

@@ -0,0 +1,17 @@
+# Generated by Django 3.2.23 on 2024-02-04 22:40
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('modules', '0002_auto_20240204_2214'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='module',
+            options={'ordering': ('name', 'stream'), 'verbose_name': 'Module', 'verbose_name_plural': 'Modules'},
+        ),
+    ]

modules/models.py

@@ -24,21 +24,22 @@
 
 class Module(models.Model):
 
-    name = models.CharField(unique=True, max_length=255)
-    stream = models.CharField(unique=True, max_length=255)
+    name = models.CharField(max_length=255)
+    stream = models.CharField(max_length=255)
     version = models.CharField(max_length=255)
-    context = models.CharField(unique=True, max_length=255)
+    context = models.CharField(max_length=255)
     arch = models.ForeignKey(PackageArchitecture, on_delete=models.CASCADE)
     repo = models.ForeignKey(Repository, on_delete=models.CASCADE)
     packages = models.ManyToManyField(Package, blank=True)
 
     class Meta:
         verbose_name = 'Module'
         verbose_name_plural = 'Modules'
-        ordering = ('name',)
+        unique_together = ('name', 'stream', 'version', 'context', 'arch',)
+        ordering = ('name', 'stream',)
 
     def __str__(self):
-        return self.name
+        return f'{self.name}-{self.stream}'
 
     def get_absolute_url(self):
         return reverse('modules:module_detail', args=[str(self.id)])