Merge pull request #50 from apisearch-io/fix/fixed-sort-by-basic-fields

Fixed when sorting by id or type

Author: mmoreram

Query/SortBy.php

@@ -251,7 +251,7 @@ public function byFieldValue(
     ): SortBy {
         $this->sortsBy[] = [
             'type' => self::TYPE_FIELD,
-            "indexed_metadata.$field" => [
+            Filter::getFilterPathByField($field) => [
                 'order' => $order,
             ],
         ];

Tests/Query/SortByTest.php

@@ -116,6 +116,33 @@ public function testSortByFieldValue()
         );
     }
 
+    /**
+     * Test sort by field value.
+     */
+    public function testSortByFieldValueBasicFields()
+    {
+        $sortBy = SortBy::create();
+        $sortBy->byFieldValue('id', SortBy::ASC);
+        $sortBy->byFieldValue('type', SortBy::ASC);
+        $this->assertEquals(
+            [
+                [
+                    'type' => SortBy::TYPE_FIELD,
+                    'uuid.id' => [
+                        'order' => SortBy::ASC,
+                    ],
+                ],
+                [
+                    'type' => SortBy::TYPE_FIELD,
+                    'uuid.type' => [
+                        'order' => SortBy::ASC,
+                    ],
+                ],
+            ],
+            $sortBy->all()
+        );
+    }
+
     /**
      * Test sortby nested field and filter.
      */