From 4e57eca4738f3c1081452e47be0b4379506755fc Mon Sep 17 00:00:00 2001 From: Johanna England Date: Tue, 8 Oct 2024 14:44:56 +0200 Subject: [PATCH] Uncrispify service forms in seedDB --- python/nav/web/seeddb/page/service/edit.py | 14 ++++---------- .../web/templates/seeddb/service_netbox_form.html | 9 ++++++--- .../templates/seeddb/service_property_form.html | 13 ++++++++++--- 3 files changed, 20 insertions(+), 16 deletions(-) diff --git a/python/nav/web/seeddb/page/service/edit.py b/python/nav/web/seeddb/page/service/edit.py index 7266aca056..63fe182111 100644 --- a/python/nav/web/seeddb/page/service/edit.py +++ b/python/nav/web/seeddb/page/service/edit.py @@ -22,10 +22,9 @@ from django.db import transaction from django.urls import reverse -from crispy_forms.helper import FormHelper - from nav.models.service import Service, ServiceProperty from nav.models.manage import Netbox +from nav.web.crispyforms import set_flat_form_attributes from nav.web.servicecheckers import get_description, load_checker_classes from nav.web.message import new_message, Messages from nav.web.seeddb.page.service import ServiceInfo @@ -46,8 +45,7 @@ def __init__(self, *args, **kwargs): widget=forms.Select(attrs={'class': 'select2'}), ) - self.helper = FormHelper(self) - self.helper.form_tag = False + self.attrs = set_flat_form_attributes() @staticmethod def _build_checker_choices(): @@ -77,10 +75,7 @@ class ServiceForm(forms.Form): handler = forms.CharField(widget=forms.HiddenInput) netbox = forms.IntegerField(widget=forms.HiddenInput) - def __init__(self, *args, **kwargs): - super(ServiceForm, self).__init__(*args, **kwargs) - self.helper = FormHelper(self) - self.helper.form_tag = False + attrs = set_flat_form_attributes() class ServicePropertyForm(forms.Form): @@ -99,8 +94,7 @@ def __init__(self, *args, **kwargs): for arg, descr in opt_args: self.fields[arg] = forms.CharField(required=False, help_text=descr) - self.helper = FormHelper(self) - self.helper.form_tag = False + self.attrs = set_flat_form_attributes() def service_edit(request, service_id=None): diff --git a/python/nav/web/templates/seeddb/service_netbox_form.html b/python/nav/web/templates/seeddb/service_netbox_form.html index a933e10c2e..a2024f9c1f 100644 --- a/python/nav/web/templates/seeddb/service_netbox_form.html +++ b/python/nav/web/templates/seeddb/service_netbox_form.html @@ -1,5 +1,4 @@ {% extends "seeddb/base.html" %} -{% load crispy_forms_tags %} {% block content %} @@ -16,11 +15,15 @@

Add new service checker - {% crispy choice_form %} + {% if choice_form.attrs %} + {% include 'custom_crispy_templates/_form_content.html' with form=choice_form %} + {% else %} + {{ choice_form }} + {% endif %}
- + diff --git a/python/nav/web/templates/seeddb/service_property_form.html b/python/nav/web/templates/seeddb/service_property_form.html index fff30ea07e..4c9c50bfe1 100644 --- a/python/nav/web/templates/seeddb/service_property_form.html +++ b/python/nav/web/templates/seeddb/service_property_form.html @@ -1,5 +1,4 @@ {% extends "seeddb/base.html" %} -{% load crispy_forms_tags %} {% block content %} @@ -20,9 +19,17 @@

Attributes - {% crispy service_form %} + {% if service_form.attrs %} + {% include 'custom_crispy_templates/_form_content.html' with form=service_form %} + {% else %} + {{ service_form }} + {% endif %} {% if property_form.fields %} - {% crispy property_form %} + {% if property_form.attrs %} + {% include 'custom_crispy_templates/_form_content.html' with form=property_form %} + {% else %} + {{ property_form }} + {% endif %} {% else %}
There are no attributes for this service.