fix: eager load belongs to many association (#1906)

2023-05-22 16:09:37 +08:00 · 2023-05-22 16:09:37 +08:00 · 5288cbb1cf
commit 5288cbb1cf
parent e27cff15c9
2 changed files with 7 additions and 1 deletions
--- a/packages/core/database/src/eager-loading/eager-loading-tree.ts
+++ b/packages/core/database/src/eager-loading/eager-loading-tree.ts
@ -242,7 +242,7 @@ export class EagerLoadingTree {

          for (const instance of node.instances) {
            const parentInstance = node.parent.instances.find(
-              (parentInstance) => parentInstance.get(sourceKey) == instance.get(oneFromTarget.as).get(foreignKey),
+              (parentInstance) => parentInstance.get(sourceKey) == instance[oneFromTarget.as].get(foreignKey),
            );

            if (parentInstance) {
--- a/packages/plugins/acl/src/tests/role-check.test.ts
+++ b/packages/plugins/acl/src/tests/role-check.test.ts
@ -24,6 +24,12 @@ describe('role check action', () => {
      },
    });

+    await role.createMenuUiSchema({
+      values: {
+        name: 'test',
+      },
+    });
+
    const user = await db.getRepository('users').create({
      values: {
        roles: ['test'],