Fix up API default sort order

2025-07-25 10:41:24 +02:00 · 2015-02-06 12:20:58 +10:30
parent 3644b99bf6
commit 7b9c65b5bc
1 changed files with 5 additions and 4 deletions
--- a/framework/core/src/Flarum/Api/Actions/Base.php
+++ b/framework/core/src/Flarum/Api/Actions/Base.php
@@ -104,6 +104,11 @@ abstract class Base extends Controller
    protected function sort($options)
    {
        $criteria = (string) $this->input('sort', '');
+
+        if (! in_array($criteria, $options)) {
+            $criteria = reset($options);
+        }
+
        $order = $criteria ? 'asc' : null;

        if ($criteria && $criteria[0] == '-') {
@@ -111,10 +116,6 @@ abstract class Base extends Controller
            $criteria = substr($criteria, 1);
        }

-        if (! in_array($criteria, $options)) {
-            $criteria = reset($options);
-        }
-
        return [
            'by' => $criteria,
            'order' => $order,