Add sorting dropdown in the list #95

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

component_catalog/filters.py

@@ -307,7 +307,6 @@ class VulnerabilityFilterSet(DataspacedFilterSet):
         fields=[
             "max_score",
             "min_score",
-            "vulnerability_id",
             "affected_products_count",
             "affected_packages_count",
             "fixed_packages_length",

component_catalog/templates/component_catalog/vulnerability_list.html

@@ -13,4 +13,11 @@
       </div>
     </form>
   </li>
+{% endblock %}
+
+{% block top-right-buttons %}
+  {{ block.super }}
+  <div class="btn-group">
+    {{ filter.form.sort }}
+  </div>
 {% endblock %}

component_catalog/views.py

@@ -2491,7 +2491,8 @@ class VulnerabilityListView(
     table_headers = (
         Header("vulnerability_id", _("Vulnerability")),
         Header("aliases", _("Aliases")),
-        Header("score_range", _("Score"), help_text="Severity score range", filter="max_score"),
+        # Keep `max_score` to enable column sorting
+        Header("max_score", _("Score"), help_text="Severity score range", filter="max_score"),
         Header("summary", _("Summary")),
         Header("affected_products_count", _("Affected products"), help_text="Affected products"),
         Header("affected_packages_count", _("Affected packages"), help_text="Affected packages"),

purldb/templates/purldb/includes/filterset_modal.html → dje/templates/modals/filterset_modal.html

@@ -1,14 +1,14 @@
 {% load crispy_forms_tags %}
-<div id="purldb-filterset-modal" class="modal" tabindex="-1" role="dialog">
+<div id="filterset-modal" class="modal" tabindex="-1" role="dialog">
   <div class="modal-dialog modal-lg" role="document">
     <div class="modal-content">
       <div class="modal-header">
         <h5 class="modal-title">Filters</h5>
         <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
       </div>
-      <form autocomplete="off" method="{{ form_helper.form_method }}" class="{{ form_helper.form_class }}">
+      <form autocomplete="off" method="{{ filter_form_helper.form_method }}" class="{{ filter_form_helper.form_class }}">
         <div class="modal-body bg-body-tertiary">
-          {% crispy filter.form form_helper %}
+          {% crispy filter.form filter_form_helper %}
         </div>
         <div class="modal-footer">
           <input type="button" name="close" value="Close" class="btn btn-secondary" data-bs-dismiss="modal">

purldb/templates/purldb/purldb_list.html

@@ -18,11 +18,11 @@
 {% block top-right-buttons %}
   {{ block.super }}
    <div class="btn-group">
-    <button type="button" class="btn btn-outline-dark {% if filter.form.changed_data %}active{% endif %}" data-bs-toggle="modal" data-bs-target="#purldb-filterset-modal"><i class="fas fa-filter"></i> Filters</button>
+    <button type="button" class="btn btn-outline-dark {% if filter.form.changed_data %}active{% endif %}" data-bs-toggle="modal" data-bs-target="#filterset-modal"><i class="fas fa-filter"></i> Filters</button>
   </div>
 {% endblock %}
 
 {% block content %}
   {{ block.super }}
-  {% include 'purldb/includes/filterset_modal.html' %}
+  {% include 'modals/filterset_modal.html' %}
 {% endblock %}

purldb/tests/test_views.py

@@ -199,8 +199,8 @@ def test_purldb_list_view_filters(self):
         self.client.login(username=self.nexb_user.username, password="secret")
 
         response = self.client.get(list_url + "?sort=-name&type=pypi")
-        self.assertContains(response, 'data-bs-target="#purldb-filterset-modal"')
-        self.assertContains(response, 'id="purldb-filterset-modal"')
+        self.assertContains(response, 'data-bs-target="#filterset-modal"')
+        self.assertContains(response, 'id="filterset-modal"')
         self.assertContains(
             response, '<option value="-name" selected>Name (descending)</option>', html=True
         )

purldb/views.py

@@ -178,7 +178,7 @@ def get_queryset(self):
         return self.list_data.get("results", [])
 
     @staticmethod
-    def get_form_helper():
+    def get_filter_form_helper():
         helper = FormHelper()
         helper.form_method = "get"
         helper.form_tag = False
@@ -200,7 +200,7 @@ def get_context_data(self, **kwargs):
         )
 
         context["filter"] = self.filterset
-        context["form_helper"] = self.get_form_helper()
+        context["filter_form_helper"] = self.get_filter_form_helper()
 
         return context