Merge pull request #70 from klerick/nestjs-json-api-67

klerick · web-flow · commit 36a03df341a3 · 2023-11-20T14:23:49.000+06:00
fix(json-api-nestjs): escape conditional
diff --git a/libs/json-api-nestjs/src/lib/mixin/service/typeorm/methods/get-all/get-all.ts b/libs/json-api-nestjs/src/lib/mixin/service/typeorm/methods/get-all/get-all.ts
@@ -165,7 +165,7 @@ export async function getAll<T>(
     .innerJoin(
       `(${builder.getQuery()})`,
       'subQuery',
-      `"subQuery"."${subQueryIdAlias}" = ${countAlias}.${primaryColumn}`
+      `${builder.escape('subQuery')}.${builder.escape(subQueryIdAlias)} = ${countAlias}.${primaryColumn}`
     )
     .setParameters(builder.getParameters())
     .getRawMany<T>();

Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ export async function getAll<T>(`
`165`	`165`	`.innerJoin(`
`166`	`166`	`(${builder.getQuery()})`,
`167`	`167`	`'subQuery',`
`168`		- `"subQuery"."${subQueryIdAlias}" = ${countAlias}.${primaryColumn}`
	`168`	+ `${builder.escape('subQuery')}.${builder.escape(subQueryIdAlias)} = ${countAlias}.${primaryColumn}`
`169`	`169`	`)`
`170`	`170`	`.setParameters(builder.getParameters())`
`171`	`171`	`.getRawMany<T>();`