Fix nested json array transform - fixes porsager#506

porsager · porsager · commit 5dea9530bda1 · 2022-10-15T23:29:17.000+02:00
diff --git a/src/types.js b/src/types.js
@@ -331,7 +331,7 @@ function createJsonTransform(fn) {
   return function jsonTransform(x, column) {
     return column.type === 114 || column.type === 3802
       ? Array.isArray(x)
-        ? x.map(jsonTransform)
+        ? x.map(x => jsonTransform(x, column))
         : Object.entries(x).reduce((acc, [k, v]) => Object.assign(acc, { [fn(k)]: v }), {})
       : x
   }
diff --git a/tests/index.js b/tests/index.js
@@ -603,6 +603,14 @@ t('column toKebab', async() => {
   return ['hello-world', Object.keys((await sql`select * from test`)[0])[0], await sql`drop table test`]
 })
 
+t('Transform nested json in arrays', async() => {
+  const sql = postgres({
+    ...options,
+    transform: postgres.camel
+  })
+  return ['aBcD', (await sql`select '[{"a_b":1},{"c_d":2}]'::jsonb as x`)[0].x.map(Object.keys).join('')]
+})
+
 t('unsafe', async() => {
   await sql`create table test (x int)`
   return [1, (await sql.unsafe('insert into test values ($1) returning *', [1]))[0].x, await sql`drop table test`]

Original file line number	Diff line number	Diff line change
`@@ -331,7 +331,7 @@ function createJsonTransform(fn) {`
`331`	`331`	`return function jsonTransform(x, column) {`
`332`	`332`	`return column.type === 114 \|\| column.type === 3802`
`333`	`333`	`? Array.isArray(x)`
`334`		`- ? x.map(jsonTransform)`
	`334`	`+ ? x.map(x => jsonTransform(x, column))`
`335`	`335`	`: Object.entries(x).reduce((acc, [k, v]) => Object.assign(acc, { [fn(k)]: v }), {})`
`336`	`336`	`: x`
`337`	`337`	`}`