From d8f3ce4203b16f60b6ffdd55edff1ca354f006e9 Mon Sep 17 00:00:00 2001
From: Marcus Furlong <furlongm@gmail.com>
Date: Wed, 17 Dec 2025 14:11:34 -0500
Subject: [PATCH] fix same module in different repos

---
 .../0005_alter_module_unique_together.py      | 19 +++++++++++++++++++
 modules/models.py                             |  2 +-
 modules/utils.py                              |  5 ++---
 3 files changed, 22 insertions(+), 4 deletions(-)
 create mode 100644 modules/migrations/0005_alter_module_unique_together.py

diff --git a/modules/migrations/0005_alter_module_unique_together.py b/modules/migrations/0005_alter_module_unique_together.py
new file mode 100644
index 00000000..046a0c54
--- /dev/null
+++ b/modules/migrations/0005_alter_module_unique_together.py
@@ -0,0 +1,19 @@
+# Generated by Django 4.2.25 on 2025-11-25 16:37
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('arch', '0001_initial'),
+        ('repos', '0006_mirror_errata_checksum_mirror_modules_checksum'),
+        ('modules', '0004_alter_module_options'),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name='module',
+            unique_together={('name', 'stream', 'version', 'context', 'arch', 'repo')},
+        ),
+    ]
diff --git a/modules/models.py b/modules/models.py
index 931a41c3..e1f8071b 100644
--- a/modules/models.py
+++ b/modules/models.py
@@ -35,7 +35,7 @@ class Module(models.Model):
     class Meta:
         verbose_name = 'Module'
         verbose_name_plural = 'Modules'
-        unique_together = ['name', 'stream', 'version', 'context', 'arch']
+        unique_together = ['name', 'stream', 'version', 'context', 'arch', 'repo']
         ordering = ['name', 'stream']
 
     def __str__(self):
diff --git a/modules/utils.py b/modules/utils.py
index 0d669478..248b8b45 100644
--- a/modules/utils.py
+++ b/modules/utils.py
@@ -25,10 +25,9 @@ def get_or_create_module(name, stream, version, context, arch, repo):
     """ Get or create a module object
         Returns the module
     """
-    created = False
-    m_arch, c = PackageArchitecture.objects.get_or_create(name=arch)
+    m_arch, _ = PackageArchitecture.objects.get_or_create(name=arch)
     try:
-        module, created = Module.objects.get_or_create(
+        module, _ = Module.objects.get_or_create(
             name=name,
             stream=stream,
             version=version,