Fixed order by bug in WooCommerce api listing

Denhac · Jan 30, 2024 · 7cbb8da · 7cbb8da
1 parent 957ad60
commit 7cbb8da
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/app/External/WooCommerce/Api/WooCommerceApiMixin.php b/app/External/WooCommerce/Api/WooCommerceApiMixin.php
@@ -37,6 +37,10 @@ private function getWithPaging($url, array $options = [], ApiProgress $progress
             $options[RequestOptions::QUERY] = [];
         }
 
+        // If you don't order by id, the default is post_date which causes issues when two posts have the exact same timestamp.
+        // Forcing the order here isn't my favorite thing, but it can always be sorted later
+        $options[RequestOptions::QUERY]['orderby'] = 'id';
+
         if (! Arr::has($options[RequestOptions::QUERY], 'per_page')) {
             $options[RequestOptions::QUERY]['per_page'] = 100;
         }