From f485a47b4849c1e756f3d3c5c56c3ccb49688a03 Mon Sep 17 00:00:00 2001
From: jeremystretch <jstretch@ns1.com>
Date: Mon, 18 Oct 2021 15:41:29 -0400
Subject: [PATCH] Tweak uniqueness constraints

---
 netbox/tenancy/migrations/0003_contacts.py | 11 +++++++----
 netbox/tenancy/models.py                   | 12 ++++++++----
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/netbox/tenancy/migrations/0003_contacts.py b/netbox/tenancy/migrations/0003_contacts.py
index dc6f6c668..6f77810f3 100644
--- a/netbox/tenancy/migrations/0003_contacts.py
+++ b/netbox/tenancy/migrations/0003_contacts.py
@@ -1,5 +1,3 @@
-# Generated by Django 3.2.8 on 2021-10-18 16:12
-
 import django.core.serializers.json
 from django.db import migrations, models
 import django.db.models.deletion
@@ -56,8 +54,8 @@ class Migration(migrations.Migration):
                 ('last_updated', models.DateTimeField(auto_now=True, null=True)),
                 ('custom_field_data', models.JSONField(blank=True, default=dict, encoder=django.core.serializers.json.DjangoJSONEncoder)),
                 ('id', models.BigAutoField(primary_key=True, serialize=False)),
-                ('name', models.CharField(max_length=100, unique=True)),
-                ('slug', models.SlugField(max_length=100, unique=True)),
+                ('name', models.CharField(max_length=100)),
+                ('slug', models.SlugField(max_length=100)),
                 ('description', models.CharField(blank=True, max_length=200)),
                 ('lft', models.PositiveIntegerField(editable=False)),
                 ('rght', models.PositiveIntegerField(editable=False)),
@@ -67,6 +65,7 @@ class Migration(migrations.Migration):
             ],
             options={
                 'ordering': ['name'],
+                'unique_together': {('parent', 'name')},
             },
         ),
         migrations.CreateModel(
@@ -95,4 +94,8 @@ class Migration(migrations.Migration):
             name='tags',
             field=taggit.managers.TaggableManager(through='extras.TaggedItem', to='extras.Tag'),
         ),
+        migrations.AlterUniqueTogether(
+            name='contact',
+            unique_together={('group', 'name')},
+        ),
     ]
diff --git a/netbox/tenancy/models.py b/netbox/tenancy/models.py
index f416d55b5..f53f7d0e6 100644
--- a/netbox/tenancy/models.py
+++ b/netbox/tenancy/models.py
@@ -117,12 +117,10 @@ class ContactGroup(NestedGroupModel):
     An arbitrary collection of Contacts.
     """
     name = models.CharField(
-        max_length=100,
-        unique=True
+        max_length=100
     )
     slug = models.SlugField(
-        max_length=100,
-        unique=True
+        max_length=100
     )
     parent = TreeForeignKey(
         to='self',
@@ -139,6 +137,9 @@ class ContactGroup(NestedGroupModel):
 
     class Meta:
         ordering = ['name']
+        unique_together = (
+            ('parent', 'name')
+        )
 
     def get_absolute_url(self):
         return reverse('tenancy:contactgroup', args=[self.pk])
@@ -216,6 +217,9 @@ class Contact(PrimaryModel):
 
     class Meta:
         ordering = ['name']
+        unique_together = (
+            ('group', 'name')
+        )
 
     def __str__(self):
         return self.name