Merge pull request #132 from supabase/fix/order-column

Fix ordering on JSON columns
supabase · Nov 12, 2020 · c92493e · c92493e
2 parents 43b093d + 548ea28
commit c92493e
Show file tree

Hide file tree

Showing 3 changed files with 39 additions and 1 deletion.
diff --git a/src/lib/PostgrestTransformBuilder.ts b/src/lib/PostgrestTransformBuilder.ts
@@ -24,7 +24,7 @@ export default class PostgrestTransformBuilder<T> extends PostgrestBuilder<T> {
     const key = typeof foreignTable === 'undefined' ? 'order' : `"${foreignTable}".order`
     this.url.searchParams.set(
       key,
-      `"${column}".${ascending ? 'asc' : 'desc'}.${nullsFirst ? 'nullsfirst' : 'nullslast'}`
+      `${column}.${ascending ? 'asc' : 'desc'}.${nullsFirst ? 'nullsfirst' : 'nullslast'}`
     )
     return this
   }

diff --git a/test/__snapshots__/basic.test.ts.snap b/test/__snapshots__/basic.test.ts.snap
@@ -1,5 +1,38 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`allow ordering on JSON column 1`] = `
+Array [
+  Object {
+    "age_range": "[1,2)",
+    "catchphrase": "'cat' 'fat'",
+    "data": null,
+    "status": "ONLINE",
+    "username": "supabot",
+  },
+  Object {
+    "age_range": "[25,35)",
+    "catchphrase": "'bat' 'cat'",
+    "data": null,
+    "status": "OFFLINE",
+    "username": "kiwicopple",
+  },
+  Object {
+    "age_range": "[25,35)",
+    "catchphrase": "'bat' 'rat'",
+    "data": null,
+    "status": "ONLINE",
+    "username": "awailas",
+  },
+  Object {
+    "age_range": "[20,30)",
+    "catchphrase": "'fat' 'rat'",
+    "data": null,
+    "status": "ONLINE",
+    "username": "dragarcia",
+  },
+]
+`;
+
 exports[`basic insert, update, delete basic delete 1`] = `
 Object {
   "body": Array [

diff --git a/test/basic.test.ts b/test/basic.test.ts
@@ -126,3 +126,8 @@ test("don't mutate PostgrestClient.headers", async () => {
   const { error } = await postgrest.from('users').select()
   expect(error).toMatchSnapshot()
 })
+
+test("allow ordering on JSON column", async () => {
+  const { data } = await postgrest.from('users').select().order('data->something')
+  expect(data).toMatchSnapshot()
+})