Reverse Sorting, tests needed

Author: AllanKoder

app/Http/Controllers/ComputerScienceResourceController.php

@@ -180,6 +180,9 @@ public function index(Request $request)
         /// Handle Sorting
         $sortBy = $request->query('sort_by', 'top');
         $query = $this->resourceSortingManager->applySort($query, $sortBy);
+        if ($request->query('reverse', '0') == '1') {
+            $query = $this->resourceSortingManager->reverse($query);
+        }
 
         // Paginate and return
         $resources = $query->paginate(10)->appends($request->query());

app/Services/SortingManagers/SortingManager.php

@@ -20,4 +20,27 @@ public function applySort(Builder $query, string $sortBy): Builder
 
         return $query; // fallback: no sort applied
     }
+
+    /**
+     * Reverses the direction (ASC <-> DESC) of the *last* order clause in the query.
+     */
+    public function reverse(Builder $query): Builder
+    {
+        $orders = $query->getQuery()->orders;
+
+        if (empty($orders)) {
+            return $query;
+        }
+
+        $lastOrder = array_pop($orders);
+
+        if (isset($lastOrder['direction'])) {
+            $lastOrder['direction'] = strtolower($lastOrder['direction']) === 'asc' ? 'desc' : 'asc';
+        }
+
+        $orders[] = $lastOrder;
+        $query->getQuery()->orders = $orders;
+
+        return $query;
+    }
 }

app/SortingStrategies/DateSortingStrategy.php

@@ -16,13 +16,13 @@ public static function apply(Builder $query, string $sortBy): Builder
     {
         switch ($sortBy) {
             case 'latest':
-                $query->orderBy('created_at', 'desc');
+                $query->orderBy('created_at', 'DESC');
                 break;
             case 'oldest':
-                $query->orderBy('created_at', 'asc');
+                $query->orderBy('created_at', 'ASC');
                 break;
             case 'recently_updated':
-                $query->orderBy('updated_at', 'desc');
+                $query->orderBy('updated_at', 'DESC');
                 break;
         }
 

app/SortingStrategies/VoteSortingStrategy.php

@@ -25,19 +25,19 @@ public static function apply(Builder $query, string $sortBy): Builder
 
         switch ($sortBy) {
             case 'top':
-                $query->orderBy('upvote_summaries.score', 'desc');
+                $query->orderBy('upvote_summaries.score', 'DESC');
                 break;
 
             case 'bottom':
-                $query->orderBy('upvote_summaries.score', 'asc');
+                $query->orderBy('upvote_summaries.score', 'ASC');
                 break;
 
             case 'controversial':
-                $query->orderBy('upvote_summaries.controversy', 'asc');
+                $query->orderBy('upvote_summaries.controversy', 'ASC');
                 break;
 
             case 'total_votes':
-                $query->orderBy('upvote_summaries.total_votes', 'desc');
+                $query->orderBy('upvote_summaries.total_votes', 'DESC');
                 break;
 
             case 'hot':

resources/js/Components/Resources/FilterBar.vue

@@ -39,6 +39,7 @@ const selectedUpdates = ref(null);
 
 // sort_by options
 const selectedSorting = ref("top");
+const selectedReverse = ref(false);
 
 // date filters
 const createdFrom = ref(null);
@@ -80,6 +81,7 @@ onMounted(() => {
 
     // initialize sort_by
     selectedSorting.value = urlParams.get("sort_by") || "top";
+    selectedReverse.value = urlParams.get("reverse") === "1";
 
     createdFrom.value = urlParams.get("created_from")
         ? new Date(urlParams.get("created_from") + "T00:00:00")
@@ -106,7 +108,8 @@ onMounted(() => {
             createdTo.value !== null ||
             updatedFrom.value !== null ||
             updatedTo.value !== null ||
-            selectedSorting.value !== "top"
+            selectedSorting.value !== "top" ||
+            selectedReverse.value == true
         );
     }
 
@@ -166,6 +169,7 @@ function search() {
             updated_to:
                 updatedTo.value?.toISOString().slice(0, 10) || undefined,
             sort_by: selectedSorting.value || undefined,
+            reverse: selectedReverse.value == true ? true : undefined,
         }),
         { preserveScroll: true }
     );
@@ -201,6 +205,7 @@ function resetFilters() {
 
     // Sorting
     selectedSorting.value = "top";
+    selectedReverse.value = false;
 }
 </script>
 
@@ -424,7 +429,7 @@ function resetFilters() {
             <!-- Sorting Buttons -->
             <div class="w-full">
                 <h2 class="text-sm font-medium mb-2">Sorting</h2>
-                <div class="flex flex-wrap gap-2">
+                <div class="flex flex-wrap gap-2 mb-2">
                     <button
                         v-for="opt in resourceSortingLabels"
                         :key="opt.value"
@@ -440,6 +445,20 @@ function resetFilters() {
                         {{ opt.label }}
                     </button>
                 </div>
+
+                <h2 class="text-sm font-medium mb-2">Reverse Sorting</h2>
+                <button
+                    type="button"
+                    @click="selectedReverse = !selectedReverse"
+                    :class="[
+                        'px-3 py-1 rounded-full text-sm font-medium focus:outline-none',
+                        selectedReverse
+                        ? 'bg-blue-600 text-white'
+                        : 'bg-gray-200 text-gray-700 hover:bg-gray-300',
+                    ]"
+                    >
+                    Reverse
+                </button>
             </div>
         </div>