fix(user): drop stale select_related("global_role") in /user list view

Global_Role.user uses related_name="+" under legacy authorization
(dojo/authorization/models.py:52), so Dojo_User has no `global_role`
reverse accessor. The list view's select_related call was raising
FieldError and 500'ing /user. Drop the arg — the template's
{% if u.global_role.role %} silently evaluates falsy when the accessor
is missing, so the Global Role column simply renders blank, matching
the rest of the inert RBAC surface under legacy.

Author: devGregA

dojo/user/views.py

@@ -330,7 +330,7 @@ def change_password(request):
 def user(request):
     page_name = _("All Users")
     users = Dojo_User.objects.all() \
-        .select_related("usercontactinfo", "global_role") \
+        .select_related("usercontactinfo") \
         .order_by("username", "last_name", "first_name")
     users = UserFilter(request.GET, queryset=users)
     paged_users = get_page_items(request, users.qs, 25)

unittests/test_user_ui_timestamps.py

@@ -92,3 +92,13 @@ def test_password_reset_confirm_stamps_password_last_reset(self):
         self.assertEqual(resp.status_code, 302)
         uci = UserContactInfo.objects.get(user=user)
         self.assertEqual(uci.password_last_reset, fixed)
+
+    def test_user_list_page_renders(self):
+        # Regression: /user previously called select_related("global_role"),
+        # but Global_Role.user uses related_name="+" under legacy
+        # authorization, so Dojo_User has no `global_role` reverse accessor.
+        # The select_related raised FieldError and 500'd the page.
+        admin = Dojo_User.objects.get(username="admin")
+        self.client.force_login(admin)
+        resp = self.client.get(reverse("users"))
+        self.assertEqual(resp.status_code, 200)