12589 review changes

netbox/templates/users/user.html

@@ -57,8 +57,20 @@
           {% endfor %}
         </ul>
       </div>
+
+      <div class="card">
+        <h5 class="card-header">{% trans "Assigned Permissions" %}</h5>
+        <ul class="list-group list-group-flush">
+          {% for perm in object.object_permissions.all %}
+            <li class="list-group-item">{{ perm }}</li>
+          {% empty %}
+            <li class="list-group-item text-muted">{% trans "None" %}</li>
+          {% endfor %}
+        </ul>
       </div>
     </div>
+
+  </div>
   {% if perms.extras.view_objectchange %}
     <div class="row">
       <div class="col-md-12">

netbox/users/forms/bulk_edit.py

@@ -1,12 +1,8 @@
 from django import forms
 from django.utils.translation import gettext_lazy as _
 
-from netbox.forms import NetBoxModelBulkEditForm
 from users.models import *
-from utilities.forms import add_blank_choice
-from utilities.forms.fields import CommentField, DynamicModelChoiceField, DynamicModelMultipleChoiceField
 from utilities.forms import BootstrapMixin
-from utilities.forms.widgets import DatePicker, NumberWithOptions
 
 __all__ = (
     'ObjectPermissionBulkEditForm',

netbox/users/forms/bulk_import.py

@@ -1,10 +1,7 @@
 from django import forms
 
-from users.models import *
+from users.models import NetBoxGroup
-from django.utils.translation import gettext_lazy as _
 from netbox.forms import NetBoxModelImportForm
-from utilities.forms import BootstrapMixin
-from utilities.forms.fields import CSVChoiceField, CSVModelChoiceField, SlugField
 
 __all__ = (
     'GroupImportForm',

netbox/users/migrations/0004_netboxgroup_netboxuser.py

@@ -39,4 +39,12 @@ class Migration(migrations.Migration):
                 ('objects', django.contrib.auth.models.UserManager()),
             ],
         ),
+        migrations.AlterModelOptions(
+            name='netboxgroup',
+            options={'ordering': ['name'], 'verbose_name': 'Group'},
+        ),
+        migrations.AlterModelOptions(
+            name='netboxuser',
+            options={'ordering': ['username'], 'verbose_name': 'User'},
+        ),
     ]

netbox/users/models.py

@@ -56,6 +56,10 @@ class NetBoxUserManager(UserManager.from_queryset(RestrictedQuerySet)):
     pass
 
 
+class NetBoxGroupManager(GroupManager.from_queryset(RestrictedQuerySet)):
+    pass
+
+
 class NetBoxUser(User):
     """
     Proxy contrib.auth.models.User for the UI
@@ -65,6 +69,7 @@ class NetBoxUser(User):
     class Meta:
         verbose_name = 'User'
         proxy = True
+        ordering = ['username',]
 
     def get_absolute_url(self):
         return reverse('users:netboxuser', args=[self.pk])
@@ -74,11 +79,12 @@ class NetBoxGroup(Group):
     """
     Proxy contrib.auth.models.User for the UI
     """
-    objects = RestrictedQuerySet.as_manager()
+    objects = NetBoxGroupManager()
 
     class Meta:
         verbose_name = 'Group'
         proxy = True
+        ordering = ['name',]
 
     def get_absolute_url(self):
         return reverse('users:netboxgroup', args=[self.pk])