admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix one to many relation dynamic query (#2430)

Browse Source
This commit is contained in:
Weiko
2023-11-10 18:20:36 +01:00
committed by GitHub
parent 41f658b1ed
commit 032e516a46
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
server/src/tenant/query-builder/factories/composite-field-alias.factory.ts
View File

@ -64,11 +64,14 @@ export class CompositeFieldAliasFactory {
);
}
const targetTableName = relationMetadata.toObjectMetadata.targetTableName;
const relationDirection = deduceRelationDirection(
fieldMetadata.objectMetadataId,
relationMetadata,
);
const targetTableName =
relationDirection == RelationDirection.TO
? relationMetadata.fromObjectMetadata.targetTableName
: relationMetadata.toObjectMetadata.targetTableName;
// If it's a relation destination is of kind MANY, we need to add the collection suffix and extract the args
if (
@ -78,9 +81,8 @@ export class CompositeFieldAliasFactory {
const args = getFieldArgumentsByKey(info, fieldKey);
const argsString = this.argsStringFactory.create(
args,
relationMetadata.toObjectMetadata.fields,
relationMetadata.toObjectMetadata.fields ?? [],
);
return `
${fieldKey}: ${targetTableName}Collection${
argsString ? `(${argsString})` : ''
@ -88,7 +90,7 @@ export class CompositeFieldAliasFactory {
${this.fieldsStringFactory.createFieldsStringRecursive(
info,
fieldValue,
relationMetadata.toObjectMetadata.fields,
relationMetadata.toObjectMetadata.fields ?? [],
)}
}
`;
@ -100,7 +102,7 @@ export class CompositeFieldAliasFactory {
${this.fieldsStringFactory.createFieldsStringRecursive(
info,
fieldValue,
relationMetadata.toObjectMetadata.fields,
relationMetadata.toObjectMetadata.fields ?? [],
)}
}
`;