From e799821032b10f2789240fb7b01b852039a25fb5 Mon Sep 17 00:00:00 2001 From: Jeremy Stretch Date: Fri, 19 Jan 2024 16:52:09 -0500 Subject: [PATCH] Replace is_htmx() function with request.htmx --- netbox/extras/views.py | 13 +++++-------- netbox/netbox/views/generic/bulk_views.py | 4 ++-- netbox/netbox/views/generic/object_views.py | 11 +++++------ netbox/netbox/views/misc.py | 3 +-- netbox/utilities/htmx.py | 8 -------- 5 files changed, 13 insertions(+), 26 deletions(-) diff --git a/netbox/extras/views.py b/netbox/extras/views.py index 5a1fbf19e..f33b4c89b 100644 --- a/netbox/extras/views.py +++ b/netbox/extras/views.py @@ -1,5 +1,3 @@ -from django.apps import apps -from django.conf import settings from django.contrib import messages from django.contrib.auth.mixins import LoginRequiredMixin from django.contrib.contenttypes.models import ContentType @@ -20,7 +18,6 @@ from extras.dashboard.utils import get_widget_class from netbox.constants import DEFAULT_ACTION_PERMISSIONS from netbox.views import generic from utilities.forms import ConfirmationForm, get_field_value -from utilities.htmx import is_htmx from utilities.paginator import EnhancedPaginator, get_paginate_count from utilities.rqworker import get_workers_for_queue from utilities.templatetags.builtins.filters import render_markdown @@ -892,7 +889,7 @@ class DashboardWidgetAddView(LoginRequiredMixin, View): template_name = 'extras/dashboard/widget_add.html' def get(self, request): - if not is_htmx(request): + if not request.htmx: return redirect('home') initial = request.GET or { @@ -942,7 +939,7 @@ class DashboardWidgetConfigView(LoginRequiredMixin, View): template_name = 'extras/dashboard/widget_config.html' def get(self, request, id): - if not is_htmx(request): + if not request.htmx: return redirect('home') widget = request.user.dashboard.get_widget(id) @@ -983,7 +980,7 @@ class DashboardWidgetDeleteView(LoginRequiredMixin, View): template_name = 'generic/object_delete.html' def get(self, request, id): - if not is_htmx(request): + if not request.htmx: return redirect('home') widget = request.user.dashboard.get_widget(id) @@ -1173,7 +1170,7 @@ class ReportResultView(ContentTypePermissionRequiredMixin, View): report = module.reports[job.name] # If this is an HTMX request, return only the result HTML - if is_htmx(request): + if request.htmx: response = render(request, 'extras/htmx/report_result.html', { 'report': report, 'job': job, @@ -1347,7 +1344,7 @@ class ScriptResultView(ContentTypePermissionRequiredMixin, View): script = module.scripts[job.name]() # If this is an HTMX request, return only the result HTML - if is_htmx(request): + if request.htmx: response = render(request, 'extras/htmx/script_result.html', { 'script': script, 'job': job, diff --git a/netbox/netbox/views/generic/bulk_views.py b/netbox/netbox/views/generic/bulk_views.py index 615db6181..70436129d 100644 --- a/netbox/netbox/views/generic/bulk_views.py +++ b/netbox/netbox/views/generic/bulk_views.py @@ -22,7 +22,7 @@ from utilities.error_handlers import handle_protectederror from utilities.exceptions import AbortRequest, AbortTransaction, PermissionsViolation from utilities.forms import BulkRenameForm, ConfirmationForm, restrict_form_fields from utilities.forms.bulk_import import BulkImportForm -from utilities.htmx import is_embedded, is_htmx +from utilities.htmx import is_embedded from utilities.permissions import get_permission_for_model from utilities.utils import get_viewname from utilities.views import GetReturnURLMixin @@ -162,7 +162,7 @@ class ObjectListView(BaseMultiObjectView, ActionsMixin, TableMixin): table = self.get_table(self.queryset, request, has_bulk_actions) # If this is an HTMX request, return only the rendered table HTML - if is_htmx(request): + if request.htmx: if is_embedded(request): table.embedded = True # Hide selection checkboxes diff --git a/netbox/netbox/views/generic/object_views.py b/netbox/netbox/views/generic/object_views.py index 90b6e9495..10ea257bd 100644 --- a/netbox/netbox/views/generic/object_views.py +++ b/netbox/netbox/views/generic/object_views.py @@ -16,7 +16,6 @@ from extras.signals import clear_events from utilities.error_handlers import handle_protectederror from utilities.exceptions import AbortRequest, PermissionsViolation from utilities.forms import ConfirmationForm, restrict_form_fields -from utilities.htmx import is_htmx from utilities.permissions import get_permission_for_model from utilities.utils import get_viewname, normalize_querydict, prepare_cloned_fields from utilities.views import GetReturnURLMixin @@ -136,7 +135,7 @@ class ObjectChildrenView(ObjectView, ActionsMixin, TableMixin): table = self.get_table(table_data, request, has_bulk_actions) # If this is an HTMX request, return only the rendered table HTML - if is_htmx(request): + if request.htmx: return render(request, 'htmx/table.html', { 'object': instance, 'table': table, @@ -224,7 +223,7 @@ class ObjectEditView(GetReturnURLMixin, BaseObjectView): restrict_form_fields(form, request.user) # If this is an HTMX request, return only the rendered form HTML - if is_htmx(request): + if request.htmx: return render(request, 'htmx/form.html', { 'form': form, }) @@ -349,7 +348,7 @@ class ObjectDeleteView(GetReturnURLMixin, BaseObjectView): """ handle_protectederror(protected_objects, request, exc) - if is_htmx(request): + if request.htmx: return HttpResponse(headers={ 'HX-Redirect': obj.get_absolute_url(), }) @@ -378,7 +377,7 @@ class ObjectDeleteView(GetReturnURLMixin, BaseObjectView): return self._handle_protected_objects(obj, e.restricted_objects, request, e) # If this is an HTMX request, return only the rendered deletion form as modal content - if is_htmx(request): + if request.htmx: viewname = get_viewname(self.queryset.model, action='delete') form_url = reverse(viewname, kwargs={'pk': obj.pk}) return render(request, 'htmx/delete_form.html', { @@ -480,7 +479,7 @@ class ComponentCreateView(GetReturnURLMixin, BaseObjectView): instance = self.alter_object(self.queryset.model(), request) # If this is an HTMX request, return only the rendered form HTML - if is_htmx(request): + if request.htmx: return render(request, 'htmx/form.html', { 'form': form, }) diff --git a/netbox/netbox/views/misc.py b/netbox/netbox/views/misc.py index c7255916c..781b40494 100644 --- a/netbox/netbox/views/misc.py +++ b/netbox/netbox/views/misc.py @@ -14,7 +14,6 @@ from netbox.forms import SearchForm from netbox.search import LookupTypes from netbox.search.backends import search_backend from netbox.tables import SearchTable -from utilities.htmx import is_htmx from utilities.paginator import EnhancedPaginator, get_paginate_count __all__ = ( @@ -96,7 +95,7 @@ class SearchView(View): }).configure(table) # If this is an HTMX request, return only the rendered table HTML - if is_htmx(request): + if request.htmx: return render(request, 'htmx/table.html', { 'table': table, }) diff --git a/netbox/utilities/htmx.py b/netbox/utilities/htmx.py index d12478b67..d85d817cf 100644 --- a/netbox/utilities/htmx.py +++ b/netbox/utilities/htmx.py @@ -2,17 +2,9 @@ from urllib.parse import urlparse __all__ = ( 'is_embedded', - 'is_htmx', ) -def is_htmx(request): - """ - Returns True if the request was made by HTMX; False otherwise. - """ - return 'Hx-Request' in request.headers - - def is_embedded(request): """ Returns True if the request indicates that it originates from a URL different from