CodeMirror/budibase
1
0
Fork 0
mirror of synced 2024-09-19 18:59:06 +12:00
Code Issues Projects Releases Wiki Activity

Fix for cyclic relationships, getQueryableFields allowed relationships from other tables, which can't work.

Browse source
This commit is contained in:
mike12345567 2024-08-20 12:13:37 +01:00
parent 760be9d6af
commit 5435028e7d
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
packages/server/src/sdk/app/rows/queryUtils.ts
View file

@ -59,14 +59,15 @@ export const getQueryableFields = async (
const extractTableFields = async (
table: Table,
allowedFields: string[],
fromTables: string[]
fromTables: string[],
opts: { allowRelationships?: boolean } = { allowRelationships: true }
): Promise<string[]> => {
const result = []
for (const field of Object.keys(table.schema).filter(
f => allowedFields.includes(f) && table.schema[f].visible !== false
)) {
const subSchema = table.schema[field]
if (subSchema.type === FieldType.LINK) {
if (opts.allowRelationships && subSchema.type === FieldType.LINK) {
if (fromTables.includes(subSchema.tableId)) {
// avoid circular loops
continue
@ -76,7 +77,9 @@ export const getQueryableFields = async (
const relatedFields = await extractTableFields(
relatedTable,
Object.keys(relatedTable.schema),
[...fromTables, subSchema.tableId]
[...fromTables, subSchema.tableId],
// don't let it recurse back and forth between relationships
{ allowRelationships: false }
)
result.push(