From 615ad8bddee75109fb8b78ee81c26576695e8fac Mon Sep 17 00:00:00 2001
From: Lova Andriarimalala <43842786+Xpirix@users.noreply.github.com>
Date: Thu, 3 Oct 2024 13:43:37 +0300
Subject: [PATCH] Fix plugins list sorting

---
 .../templates/plugins/plugin_list_sort.html   | 32 +++++++++++--------
 qgis-app/plugins/views.py                     | 25 ++++++++-------
 2 files changed, 31 insertions(+), 26 deletions(-)

diff --git a/qgis-app/plugins/templates/plugins/plugin_list_sort.html b/qgis-app/plugins/templates/plugins/plugin_list_sort.html
index aa0cdae..9b88a5a 100644
--- a/qgis-app/plugins/templates/plugins/plugin_list_sort.html
+++ b/qgis-app/plugins/templates/plugins/plugin_list_sort.html
@@ -4,6 +4,7 @@
         <label for="sort-options">{% trans "Sort by:" %}</label>
         <span class="select is-small">
             <select id="sort-options" onchange="updateSorting('sort', this.value);">
+                <option value=""{% if not request.GET.sort %}selected{% endif %}>-</option>
                 <option value="name" {% if request.GET.sort == 'name' %}selected{% endif %}>{% trans "Name" %}</option>
                 <option value="downloads" {% if request.GET.sort == 'downloads' %}selected{% endif %}>{% trans "Downloads" %}</option>
                 <option value="author" {% if request.GET.sort == 'author' %}selected{% endif %}>{% trans "Author" %}</option>
@@ -15,6 +16,7 @@
     <p class="control">
         <span class="select is-small">
             <select id="order-options" onchange="updateSorting('order', this.value);">
+                <option value="" {% if not request.GET.order %}selected{% endif %}>-</option>
                 <option value="asc" {% if request.GET.order == 'asc' %}selected{% endif %}>{% trans "Ascending" %}</option>
                 <option value="desc" {% if request.GET.order == 'desc' %}selected{% endif %}>{% trans "Descending" %}</option>
             </select>
@@ -24,20 +26,22 @@
 
 <script>
     function updateSorting(key, value) {
-        // Get current URL parameters
-        let params = new URLSearchParams(window.location.search);
-        
-        // Update the relevant parameter (sort/order) based on the selection
-        params.set(key, value);
-
-        // Preserve the other sorting option (sort/order)
-        if (key === 'sort' && !params.has('order')) {
-            params.set('order', 'asc');  // Default to ascending if no order is selected
-        } else if (key === 'order' && !params.has('sort')) {
-            params.set('sort', 'name');  // Default to sorting by name if no sort field is selected
+        if (value) {
+            // Get current URL parameters
+            let params = new URLSearchParams(window.location.search);
+            
+            // Update the relevant parameter (sort/order) based on the selection
+            params.set(key, value);
+    
+            // Preserve the other sorting option (sort/order)
+            if (key === 'sort' && !params.has('order')) {
+                params.set('order', 'asc');  // Default to ascending if no order is selected
+            } else if (key === 'order' && !params.has('sort')) {
+                params.set('sort', 'name');  // Default to sorting by name if no sort field is selected
+            }
+    
+            // Redirect to the updated URL
+            window.location.search = params.toString();
         }
-
-        // Redirect to the updated URL
-        window.location.search = params.toString();
     }
 </script>
\ No newline at end of file
diff --git a/qgis-app/plugins/views.py b/qgis-app/plugins/views.py
index acbe619..4ef36bb 100644
--- a/qgis-app/plugins/views.py
+++ b/qgis-app/plugins/views.py
@@ -836,18 +836,19 @@ def get_queryset(self):
         qs = self.get_filtered_queryset(qs)
 
         # Get the sort and order parameters from the URL (with default values)
-        sort_by = self.request.GET.get('sort', 'name')  # Default sort by name
-        sort_order = self.request.GET.get('order', 'asc')  # Default to ascending order
-
-        # Determine the correct sorting direction
-        if sort_order == 'desc':
-            sort_by = '-' + sort_by  # Prepend '-' to sort in descending order
-
-        # Validate the sort field
-        if sort_by.lstrip('-') in ['average_vote', 'latest_version_date'] or self._is_valid_field(sort_by.lstrip('-')):
-            qs = qs.order_by(sort_by)
-        elif not qs.ordered:
-            qs = qs.order_by(Lower("name"))
+        sort_by = self.request.GET.get('sort', None)  # Default sort by name
+        sort_order = self.request.GET.get('order', None)  # Default to ascending order
+
+        if sort_by and sort_order:
+            # Determine the correct sorting direction
+            if sort_order == 'desc':
+                sort_by = '-' + sort_by  # Prepend '-' to sort in descending order
+
+            # Validate the sort field
+            if sort_by.lstrip('-') in ['average_vote', 'latest_version_date'] or self._is_valid_field(sort_by.lstrip('-')):
+                qs = qs.order_by(sort_by)
+            elif not qs.ordered:
+                qs = qs.order_by(Lower("name"))
 
         return qs