Use html_name in place of field name attribute #1743

Signed-off-by: tdruez <[email protected]>

Author: tdruez

scanpipe/templates/scanpipe/forms/project_clone_form.html

@@ -36,7 +36,7 @@
 </div>
 <div class="field">
   <label class="checkbox" for="{{ form.execute_now.id_for_label }}-clone">
-    <input type="checkbox" name="{{ form.execute_now.name }}" id="{{ form.execute_now.id_for_label }}-clone">
+    <input type="checkbox" name="{{ form.execute_now.html_name }}" id="{{ form.execute_now.id_for_label }}-clone">
     {{ form.execute_now.label }}
   </label>
   <p class="help">{{ form.execute_now.help_text }}</p>

scanpipe/templates/scanpipe/forms/project_settings_form.html

@@ -11,7 +11,7 @@
         </div>
       </div>
       <div class="field">
-        <label class="label" for="{{ form.name.id_for_label }}">
+        <label class="label" for="{{ form.purl.id_for_label }}">
           PURL
         </label>
         <div class="control">

scanpipe/templates/scanpipe/includes/search_field.html

@@ -4,7 +4,7 @@
   {% endif %}
   <form method="get">
     <p class="control has-icons-left">
-      <input class="input {{ extra_class }}" type="search" placeholder="{% if placeholder %}{{ placeholder }}{% else %}Search {{ filter.verbose_name_plural }}{% endif %}" name="{{ filter.form.search.name }}" value="{{ filter.form.search.value|default_if_none:'' }}">
+      <input class="input {{ extra_class }}" type="search" placeholder="{% if placeholder %}{{ placeholder }}{% else %}Search {{ filter.verbose_name_plural }}{% endif %}" name="{{ filter.form.search.html_name }}" value="{{ filter.form.search.value|default_if_none:'' }}">
       <span class="icon is-small is-left">
         <i class="fa-solid fa-search"></i>
       </span>

scanpipe/templates/scanpipe/modals/projects_archive_modal.html

@@ -38,15 +38,15 @@
             <hr>
             <div class="field include-all-field">
               <label class="checkbox" for="{{ archive_form.select_across.id_for_label }}">
-                <input type="checkbox" name="{{ archive_form.select_across.name }}" id="{{ archive_form.select_across.id_for_label }}">
+                <input type="checkbox" name="{{ archive_form.select_across.html_name }}" id="{{ archive_form.select_across.id_for_label }}">
                 Include all {{ paginator.count|intcomma }} projects
               </label>
               <p class="help">{{ outputs_download_form.select_across.help_text }}</p>
             </div>
           </div>
         {% endif %}
       </section>
-      <input type="hidden" name="{{ archive_form.url_query.name }}" value="{{ request.GET.urlencode }}">
+      <input type="hidden" name="{{ archive_form.url_query.html_name }}" value="{{ request.GET.urlencode }}">
       <input type="hidden" name="action" value="archive">
       <footer class="modal-card-foot is-justify-content-flex-end">
         <div class="buttons">

scanpipe/templates/scanpipe/modals/projects_delete_modal.html

@@ -22,7 +22,7 @@
       </p>
     </section>
     <form action="{% url 'project_action' %}" method="post" id="delete-projects-form">{% csrf_token %}
-      <input type="hidden" name="{{ action_form.url_query.name }}" value="{{ request.GET.urlencode }}">
+      <input type="hidden" name="{{ action_form.url_query.html_name }}" value="{{ request.GET.urlencode }}">
       <input type="hidden" name="action" value="delete">
       <footer class="modal-card-foot is-justify-content-flex-end">
         <div class="buttons">

scanpipe/templates/scanpipe/modals/projects_download_modal.html

@@ -19,15 +19,15 @@
             <hr>
             <div class="field">
               <label class="checkbox" for="{{ outputs_download_form.select_across.id_for_label }}">
-                <input type="checkbox" name="{{ outputs_download_form.select_across.name }}" id="{{ outputs_download_form.select_across.id_for_label }}">
+                <input type="checkbox" name="{{ outputs_download_form.select_across.html_name }}" id="{{ outputs_download_form.select_across.id_for_label }}">
                 Include all {{ paginator.count|intcomma }} projects
               </label>
               <p class="help">{{ outputs_download_form.select_across.help_text }}</p>
             </div>
           </div>
         {% endif %}
       </section>
-      <input type="hidden" name="{{ outputs_download_form.url_query.name }}" value="{{ request.GET.urlencode }}">
+      <input type="hidden" name="{{ outputs_download_form.url_query.html_name }}" value="{{ request.GET.urlencode }}">
       <input type="hidden" name="action" value="download">
       <footer class="modal-card-foot is-justify-content-flex-end">
         <div class="buttons">

scanpipe/templates/scanpipe/modals/projects_report_modal.html

@@ -19,15 +19,15 @@
             <hr>
             <div class="field include-all-field">
               <label class="checkbox" for="{{ report_form.select_across.id_for_label }}">
-                <input type="checkbox" name="{{ report_form.select_across.name }}" id="{{ report_form.select_across.id_for_label }}">
+                <input type="checkbox" name="{{ report_form.select_across.html_name }}" id="{{ report_form.select_across.id_for_label }}">
                 Include all {{ paginator.count|intcomma }} projects
               </label>
               <p class="help">{{ report_form.select_across.help_text }}</p>
             </div>
           </div>
         {% endif %}
       </section>
-      <input type="hidden" name="{{ report_form.url_query.name }}" value="{{ request.GET.urlencode }}">
+      <input type="hidden" name="{{ report_form.url_query.html_name }}" value="{{ request.GET.urlencode }}">
       <input type="hidden" name="action" value="report">
       <footer class="modal-card-foot is-justify-content-flex-end">
         <div class="buttons">

scanpipe/templates/scanpipe/modals/projects_reset_modal.html

@@ -40,15 +40,15 @@
             <hr>
             <div class="field include-all-field">
               <label class="checkbox" for="{{ reset_form.select_across.id_for_label }}">
-                <input type="checkbox" name="{{ reset_form.select_across.name }}" id="{{ reset_form.select_across.id_for_label }}">
+                <input type="checkbox" name="{{ reset_form.select_across.html_name }}" id="{{ reset_form.select_across.id_for_label }}">
                 Include all {{ paginator.count|intcomma }} projects
               </label>
               <p class="help">{{ outputs_download_form.select_across.help_text }}</p>
             </div>
           </div>
         {% endif %}
       </section>
-      <input type="hidden" name="{{ action_form.url_query.name }}" value="{{ request.GET.urlencode }}">
+      <input type="hidden" name="{{ reset_form.url_query.html_name }}" value="{{ request.GET.urlencode }}">
       <input type="hidden" name="action" value="reset">
       <footer class="modal-card-foot is-justify-content-flex-end">
         <div class="buttons">

scanpipe/tests/test_views.py

@@ -174,13 +174,22 @@ def test_scanpipe_views_project_list_modal_forms_include_url_query(self):
         url = reverse("project_list")
         response = self.client.get(url)
 
-        expected = '<input type="hidden" name="url_query" value="">'
-        self.assertContains(response, expected, html=True)
+        expected_html_names = [
+            "url_query",
+            "download-url_query",
+            "report-url_query",
+            "archive-url_query",
+            "reset-url_query",
+        ]
+        for html_name in expected_html_names:
+            expected = f'<input type="hidden" name="{html_name}" value="">'
+            self.assertContains(response, expected, html=True)
 
         url_query = "name=search_value"
         response = self.client.get(url + "?" + url_query)
-        expected = f'<input type="hidden" name="url_query" value="{url_query}">'
-        self.assertContains(response, expected, html=True)
+        for html_name in expected_html_names:
+            expected = f'<input type="hidden" name="{html_name}" value="{url_query}">'
+            self.assertContains(response, expected, html=True)
 
     @mock.patch("scanpipe.views.ProjectListView.get_paginate_by")
     def test_scanpipe_views_project_list_modal_forms_include_show_on_all_checked(

scanpipe/views.py

@@ -1351,14 +1351,12 @@ def get_project_queryset(selected_project_ids=None, action_form=None):
         An instance of BaseProjectActionForm can be provided as the ``action_form``
         argument for the support of ``select_across``.
         """
-        if action_form:
-            select_across = action_form.cleaned_data.get("select_across")
+        if action_form and action_form.cleaned_data.get("select_across"):
             # url_query may be empty for a "select everything"
             url_query = action_form.cleaned_data.get("url_query", "")
-            if select_across:
-                project_filterset = ProjectFilterSet(data=QueryDict(url_query))
-                if project_filterset.is_valid():
-                    return project_filterset.qs
+            project_filterset = ProjectFilterSet(data=QueryDict(url_query))
+            if project_filterset.is_valid():
+                return project_filterset.qs
 
         if selected_project_ids:
             return Project.objects.filter(uuid__in=selected_project_ids)