Fix list generic endpoint default order by (#226)

cloudflare · Jan 8, 2025 · 5986b0d · 5986b0d
1 parent cd1ba5e
commit 5986b0d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "chanfana",
-  "version": "2.5.3",
+  "version": "2.5.4",
   "description": "OpenAPI 3 and 3.1 schema generator and validator for Hono, itty-router and more!",
   "main": "dist/index.js",
   "module": "dist/index.mjs",

diff --git a/src/endpoints/d1/list.ts b/src/endpoints/d1/list.ts
@@ -48,7 +48,7 @@ export class D1ListEndpoint<HandleArgs extends Array<object> = Array<object>> ex
       where = `WHERE ${conditions.join(" AND ")}`;
     }
 
-    let orderBy = `ORDER BY ${this.defaultOrderBy || this.meta.model.primaryKeys[0]} DESC`;
+    let orderBy = `ORDER BY ${this.defaultOrderBy || `${this.meta.model.primaryKeys[0]} DESC`}`;
     if (filters.options.order_by) {
       orderBy = `ORDER BY ${filters.options.order_by} ${filters.options.order_by_direction || "ASC"}`;
     }