Make some improvements as suggested by DjangoDoctor

assets/migrations/0019_fix_cabletype.py

@@ -0,0 +1,25 @@
+# Generated by Django 3.1.5 on 2021-02-08 16:02
+
+from django.db import migrations
+
+
+def add_default(apps, schema_editor):
+    CableType = apps.get_model('assets', 'CableType')
+    Connector = apps.get_model('assets', 'Connector')
+    for cable_type in CableType.objects.all():
+        if cable_type.plug is None:
+            cable_type.plug = Connector.first()
+        if cable_type.socket is None:
+            cable_type.socket = Connector.first()
+        cable_type.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('assets', '0018_auto_20200415_1940'),
+    ]
+
+    operations = [
+        migrations.RunPython(add_default, migrations.RunPython.noop)
+    ]

assets/migrations/0020_auto_20210208_1603.py

@@ -0,0 +1,50 @@
+# Generated by Django 3.1.5 on 2021-02-08 16:03
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('assets', '0019_fix_cabletype'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='assetstatus',
+            name='display_class',
+            field=models.CharField(blank=True, default='', help_text='HTML class to be appended to alter display of assets with this status, such as in the list.', max_length=80),
+            preserve_default=False,
+        ),
+        migrations.AlterField(
+            model_name='cabletype',
+            name='plug',
+            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='plug', to='assets.connector'),
+        ),
+        migrations.AlterField(
+            model_name='cabletype',
+            name='socket',
+            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='socket', to='assets.connector'),
+        ),
+        migrations.AlterField(
+            model_name='supplier',
+            name='address',
+            field=models.TextField(blank=True, default=''),
+        ),
+        migrations.AlterField(
+            model_name='supplier',
+            name='email',
+            field=models.EmailField(blank=True, default='', max_length=254),
+        ),
+        migrations.AlterField(
+            model_name='supplier',
+            name='notes',
+            field=models.TextField(blank=True, default=''),
+        ),
+        migrations.AlterField(
+            model_name='supplier',
+            name='phone',
+            field=models.CharField(blank=True, default='', max_length=15),
+        ),
+    ]

assets/models.py

@@ -10,44 +10,44 @@ from RIGS.models import RevisionMixin, Profile
 
 
 class AssetCategory(models.Model):
+    name = models.CharField(max_length=80)
+
     class Meta:
         verbose_name = 'Asset Category'
         verbose_name_plural = 'Asset Categories'
         ordering = ['name']
 
-    name = models.CharField(max_length=80)
-
     def __str__(self):
         return self.name
 
 
 class AssetStatus(models.Model):
+    name = models.CharField(max_length=80)
+    should_show = models.BooleanField(
+        default=True, help_text="Should this be shown by default in the asset list.")
+    display_class = models.CharField(max_length=80, blank=True, help_text="HTML class to be appended to alter display of assets with this status, such as in the list.")
+
     class Meta:
         verbose_name = 'Asset Status'
         verbose_name_plural = 'Asset Statuses'
         ordering = ['name']
 
-    name = models.CharField(max_length=80)
-    should_show = models.BooleanField(
-        default=True, help_text="Should this be shown by default in the asset list.")
-    display_class = models.CharField(max_length=80, blank=True, null=True, help_text="HTML class to be appended to alter display of assets with this status, such as in the list.")
-
     def __str__(self):
         return self.name
 
 
 @reversion.register
 class Supplier(models.Model, RevisionMixin):
+    name = models.CharField(max_length=80)
+    phone = models.CharField(max_length=15, blank=True, default="")
+    email = models.EmailField(blank=True, default="")
+    address = models.TextField(blank=True, default="")
+
+    notes = models.TextField(blank=True, default="")
+
     class Meta:
         ordering = ['name']
 
-    name = models.CharField(max_length=80)
-    phone = models.CharField(max_length=15, blank=True, null=True)
-    email = models.EmailField(blank=True, null=True)
-    address = models.TextField(blank=True, null=True)
-
-    notes = models.TextField(blank=True, null=True)
-
     def get_absolute_url(self):
         return reverse('supplier_list')
 
@@ -65,17 +65,16 @@ class Connector(models.Model):
         return self.description
 
 
-# Things are nullable that shouldn't be because I didn't properly fix the data structure when moving this to its own model...
 class CableType(models.Model):
-    class Meta:
-        ordering = ['plug', 'socket', '-circuits']
-
     circuits = models.IntegerField(default=1)
     cores = models.IntegerField(default=3)
     plug = models.ForeignKey(Connector, on_delete=models.CASCADE,
-                             related_name='plug', null=True)
+                             related_name='plug')
     socket = models.ForeignKey(Connector, on_delete=models.CASCADE,
-                               related_name='socket', null=True)
+                               related_name='socket')
+
+    class Meta:
+        ordering = ['plug', 'socket', '-circuits']
 
     def __str__(self):
         if self.plug and self.socket:
@@ -104,12 +103,6 @@ def get_available_asset_id(wanted_prefix=""):
 
 @reversion.register
 class Asset(models.Model, RevisionMixin):
-    class Meta:
-        ordering = ['asset_id_prefix', 'asset_id_number']
-        permissions = [
-            ('asset_finance', 'Can see financial data for assets')
-        ]
-
     parent = models.ForeignKey(to='self', related_name='asset_parent',
                                blank=True, null=True, on_delete=models.SET_NULL)
     asset_id = models.CharField(max_length=15, unique=True)
@@ -143,12 +136,18 @@ class Asset(models.Model, RevisionMixin):
 
     reversion_perm = 'assets.asset_finance'
 
-    def get_absolute_url(self):
-        return reverse('asset_detail', kwargs={'pk': self.asset_id})
+    class Meta:
+        ordering = ['asset_id_prefix', 'asset_id_number']
+        permissions = [
+            ('asset_finance', 'Can see financial data for assets')
+        ]
 
     def __str__(self):
         return "{} | {}".format(self.asset_id, self.description)
 
+    def get_absolute_url(self):
+        return reverse('asset_detail', kwargs={'pk': self.asset_id})
+
     def clean(self):
         errdict = {}
         if self.date_sold and self.date_acquired > self.date_sold: