@contember/engine-content-api 1.3.11 → 1.3.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contember/engine-content-api",
-  "version": "1.3.11",
+  "version": "1.3.12",
   "license": "Apache-2.0",
   "main": "dist/src/index.js",
   "typings": "dist/src/index.d.ts",
@@ -13,20 +13,20 @@
     "test": "vitest --dir ./tests/cases --no-threads"
   },
   "dependencies": {
-    "@contember/database": "1.3.11",
-    "@contember/dic": "1.3.11",
-    "@contember/graphql-utils": "1.3.11",
-    "@contember/logger": "1.3.11",
-    "@contember/schema": "1.3.11",
-    "@contember/schema-definition": "1.3.11",
-    "@contember/schema-utils": "1.3.11",
+    "@contember/database": "1.3.12",
+    "@contember/dic": "1.3.12",
+    "@contember/graphql-utils": "1.3.12",
+    "@contember/logger": "1.3.12",
+    "@contember/schema": "1.3.12",
+    "@contember/schema-definition": "1.3.12",
+    "@contember/schema-utils": "1.3.12",
     "fast-deep-equal": "^3.1.3",
     "graphql-tag": "^2.12.5"
   },
   "devDependencies": {
-    "@contember/database-tester": "1.3.11",
-    "@contember/engine-api-tester": "1.3.11",
-    "@contember/schema-definition": "1.3.11",
+    "@contember/database-tester": "1.3.12",
+    "@contember/engine-api-tester": "1.3.12",
+    "@contember/schema-definition": "1.3.12",
     "@graphql-codegen/cli": "^2.6.2",
     "@graphql-codegen/typescript": "^2.5.1",
     "@graphql-codegen/typescript-operations": "^2.4.2",

package/src/mapper/Mapper.ts

@@ -60,7 +60,8 @@ export class Mapper<ConnectionType extends Connection.ConnectionLike = Connectio
 			.from(entity.tableName, 'root_')
 			.select(['root_', columnName])
 		const expandedWhere = this.uniqueWhereExpander.expand(entity, where)
-		const builtQb = this.whereBuilder.build(qb, entity, this.pathFactory.create([]), expandedWhere)
+		const withPredicates = this.predicatesInjector.inject(entity, expandedWhere)
+		const builtQb = this.whereBuilder.build(qb, entity, this.pathFactory.create([]), withPredicates)
 		const result = await builtQb.getResult(this.db)
 
 		return result[0] !== undefined ? result[0][columnName] : undefined

package/tests/cases/integration/graphQlRequests/update/acl/aclUpdate.test.ts

@@ -1,6 +1,6 @@
 import { test } from 'vitest'
 import { execute, failedTransaction, sqlTransaction } from '../../../../../src/test'
-import { SchemaBuilder } from '@contember/schema-definition'
+import { c, createSchema, SchemaBuilder } from '@contember/schema-definition'
 import {  Model } from '@contember/schema'
 import { GQL, SQL } from '../../../../../src/tags'
 import { testUuid } from '../../../../../src/testUuid'
@@ -255,3 +255,70 @@ test('update m:n', async () => {
 	})
 })
 
+
+
+namespace ConnectWithAclOnRead {
+	export const editor = c.createRole('editor')
+
+	@c.Allow(editor, {
+		read: ['id'],
+		update: ['id', 'category'],
+	})
+	export class Article {
+		category = c.manyHasOne(Category)
+	}
+
+	@c.Allow(editor, {
+		when: { isPublic: { eq: true } },
+		read: ['id', 'slug'],
+	})
+	export class Category {
+		isPublic = c.boolColumn().notNull()
+		slug = c.stringColumn().notNull().unique()
+	}
+
+}
+
+test('connect category', async () => {
+	const schema = createSchema(ConnectWithAclOnRead)
+
+	await execute({
+		schema: schema.model,
+		query: GQL`mutation {
+        updateArticle(
+            by: {id: "${testUuid(1)}"},
+            data: {category: {connect: {slug: "abcd"}}}
+          ) {
+          ok
+        }
+      }`,
+		permissions: schema.acl.roles.editor.entities,
+		executes: [
+			...sqlTransaction([
+				{
+					sql: SQL`select "root_"."id" from "public"."article" as "root_" where "root_"."id" = ?`,
+					parameters: [testUuid(1)],
+					response: { rows: [{ id: testUuid(1) }] },
+				},
+				{
+					sql: SQL`select "root_"."id"  from "public"."category" as "root_"  where "root_"."slug" = ? and "root_"."is_public" = ?`,
+					parameters: ['abcd', true],
+					response: { rows: [{ id: testUuid(2) }] },
+				},
+				{
+					sql: SQL`with "newData_" as (select ? :: uuid as "category_id", "root_"."category_id" as "category_id_old__", "root_"."id" from "public"."article" as "root_"  where "root_"."id" = ?) 
+update "public"."article" set  "category_id" =  "newData_"."category_id" from "newData_"  where "article"."id" = "newData_"."id"  returning "category_id_old__"`,
+					parameters: [testUuid(2), testUuid(1)],
+					response: { rows: [{ category_id_old__: null }] },
+				},
+			]),
+		],
+		return: {
+			data: {
+				updateArticle: {
+					ok: true,
+				},
+			},
+		},
+	})
+})