Misc cleanup

This commit is contained in:
Jeremy Stretch 2024-06-25 08:49:07 -04:00
parent 16420a32a1
commit f32f2ed1f3
2 changed files with 2 additions and 3 deletions

View File

@ -1,6 +1,5 @@
import logging import logging
import uuid import uuid
from urllib import parse
from django.conf import settings from django.conf import settings
from django.contrib import auth, messages from django.contrib import auth, messages

View File

@ -31,7 +31,7 @@ __all__ = (
class ConditionalLoginRequiredMixin(AccessMixin): class ConditionalLoginRequiredMixin(AccessMixin):
""" """
Requires a user to be authenticated if LOGIN_REQUIRED is True. Similar to Django's LoginRequiredMixin, but enforces authentication only if LOGIN_REQUIRED is True.
""" """
def dispatch(self, request, *args, **kwargs): def dispatch(self, request, *args, **kwargs):
if settings.LOGIN_REQUIRED and not request.user.is_authenticated: if settings.LOGIN_REQUIRED and not request.user.is_authenticated:
@ -42,7 +42,7 @@ class ConditionalLoginRequiredMixin(AccessMixin):
class ContentTypePermissionRequiredMixin(ConditionalLoginRequiredMixin): class ContentTypePermissionRequiredMixin(ConditionalLoginRequiredMixin):
""" """
Similar to Django's built-in PermissionRequiredMixin, but extended to check model-level permission assignments. Similar to Django's built-in PermissionRequiredMixin, but extended to check model-level permission assignments.
This is related to ObjectPermissionRequiredMixin, except that is does not enforce object-level permissions, This is related to ObjectPermissionRequiredMixin, except that it does not enforce object-level permissions,
and fits within NetBox's custom permission enforcement system. and fits within NetBox's custom permission enforcement system.
additional_permissions: An optional iterable of statically declared permissions to evaluate in addition to those additional_permissions: An optional iterable of statically declared permissions to evaluate in addition to those