Fix autofield warning messages

Changes to be committed:
	modified:   api/admin.py
	modified:   api/apps.py
	modified:   api/models.py
	modified:   api/signals.py
	modified:   bco_api/settings.py

Author: HadleyKing

bco_api/api/admin.py

@@ -1,3 +1,4 @@
+#!/usr/bin/env python3
 """Django Admin
 
 Registers models for the Django Admin app
@@ -6,8 +7,10 @@
 from django.contrib import admin
 from guardian.admin import GuardedModelAdmin
 from api.models import bco, prefix_table, new_users, Prefix
+from api.groups import GroupInfo
 
 admin.site.register(bco, GuardedModelAdmin)
 admin.site.register(prefix_table)
 admin.site.register(new_users)
+admin.site.register(GroupInfo)
 admin.site.register(Prefix)

bco_api/api/apps.py

@@ -1,24 +1,21 @@
-# Run code after start-up
-# TODO: move things from settings.py into here.
-# Source: https://stackoverflow.com/a/42744626/5029459
-# Source: https://docs.djangoproject.com/en/3.2/ref/applications/#django.apps.AppConfig.ready
+#!/usr/bin/env python3
+"""Run code after start-up
+TODO: move things from settings.py into here.
+Source: https://stackoverflow.com/a/42744626/5029459
+Source: https://docs.djangoproject.com/en/3.2/ref/applications/#django.apps.AppConfig.ready
+"""
 
 from django.apps import AppConfig
 from django.db.models.signals import post_migrate
+from api.signals import populate_models
 
+class ApiConfig(AppConfig):
+    """API Configuration
+    """
 
-class ApiConfig(
-        AppConfig
-        ):
+    default_auto_field = 'django.db.models.AutoField'
     name = 'api'
 
-    def ready(
-            self
-            ):
-        # Create the anonymous user if they don't exist.
-        from .signals import populate_models
-
-        post_migrate.connect(
-                populate_models,
-                sender=self
-                )
+    def ready(self):
+        """Create the anonymous user if they don't exist."""
+        post_migrate.connect(populate_models, sender=self)

bco_api/api/models.py

@@ -114,12 +114,12 @@ class new_users(models.Model):
     created = models.DateTimeField(default=timezone.now)
 
 
-# Link Prefix to groups and users.
-
-# Be careful about related_name.
-# Source: https://stackoverflow.com/questions/53651114/using-same-foreign-key-twice-in-a-model-in-django-as-different-fields
 class Prefix(models.Model):
-    """Which server is this prefix certified with?"""
+    """Link Prefix to groups and users.
+
+    Be careful about related_name.
+    Source: https://stackoverflow.com/questions/53651114/using-same-foreign-key-twice-in-a-model-in-django-as-different-fields
+    Which server is this prefix certified with?"""
 
     certifying_server = models.TextField(blank = True, null = True)
 
@@ -159,10 +159,10 @@ class Prefix(models.Model):
 
     # The actual prefix.
     prefix = models.CharField(max_length=5)
-    
+
     def __str__(self):
         """String for representing the BCO model (in Admin site etc.)."""
-        return "{}".format(self.prefix)
+        return f"{self.prefix}"
 
 # def get_first_name(self):
 #     return self.first_name

bco_api/api/signals.py

@@ -76,7 +76,7 @@ def populate_models(sender, **kwargs):
 
         DbUtils.DbUtils().write_object(
             p_app_label = 'api',
-            p_model_name = 'prefixes',
+            p_model_name = 'Prefix',
             p_fields = ['created_by', 'owner_group', 'owner_user', 'prefix'],
             p_data = {
                 'created_by': user,
@@ -129,7 +129,7 @@ def populate_models(sender, **kwargs):
             name = 'prefix_admins'
         ).permissions.add(
             Permission.objects.get(
-                codename = perm + '_prefixes'
+                codename = perm + '_prefix'
             )
         )
 

bco_api/bco_api/settings.py

@@ -260,3 +260,4 @@
 EMAIL_BACKEND = 'django.core.mail.backends.smtp.EmailBackend'
 EMAIL_HOST = 'localhost'
 EMAIL_PORT = 25
+DEFAULT_AUTO_FIELD = 'django.db.models.AutoField'