diff --git a/packages/nocodb-sdk/src/lib/formulaHelpers.ts b/packages/nocodb-sdk/src/lib/formulaHelpers.ts index 89479a121c..b3cc06d364 100644 --- a/packages/nocodb-sdk/src/lib/formulaHelpers.ts +++ b/packages/nocodb-sdk/src/lib/formulaHelpers.ts @@ -1594,7 +1594,11 @@ async function extractColumnIdentifierType({ res.dataType = FormulaDataTypes.STRING; break; case UITypes.Checkbox: - res.dataType = FormulaDataTypes.NUMERIC; + if (col.dt === 'boolean' || col.dt === 'bool') { + res.dataType = FormulaDataTypes.BOOLEAN; + } else { + res.dataType = FormulaDataTypes.NUMERIC; + } break; case UITypes.ID: case UITypes.ForeignKey: diff --git a/packages/nocodb/src/db/formulav2/formulaQueryBuilderv2.ts b/packages/nocodb/src/db/formulav2/formulaQueryBuilderv2.ts index 3f2d429da6..39d3923bd3 100644 --- a/packages/nocodb/src/db/formulav2/formulaQueryBuilderv2.ts +++ b/packages/nocodb/src/db/formulav2/formulaQueryBuilderv2.ts @@ -905,6 +905,23 @@ async function _formulaQueryBuilder( // if operator is == or !=, then handle comparison with BLANK which should accept NULL and empty string if (pt.operator === '==' || pt.operator === '!=') { + for (const operand of ['left', 'right']) { + if ( + pt[operand].dataType === FormulaDataTypes.BOOLEAN && + pt[operand === 'left' ? 'right' : 'left'].dataType === + FormulaDataTypes.NUMERIC + ) { + pt[operand === 'left' ? 'right' : 'left'] = { + type: 'CallExpression', + arguments: [pt[operand === 'left' ? 'right' : 'left']], + callee: { + type: 'Identifier', + name: 'BOOLEAN', + }, + dataType: FormulaDataTypes.BOOLEAN, + }; + } + } if (pt.left.callee?.name !== pt.right.callee?.name) { // if left/right is BLANK, accept both NULL and empty string for (const operand of ['left', 'right']) { @@ -1168,7 +1185,7 @@ export default async function formulaQueryBuilderv2( const formula = await column.getColOptions(); // clean the previous formula error if the formula works this time if (formula.error) { - await FormulaColumn.update(formula.id, { + await FormulaColumn.update(column.id, { error: null, }); } @@ -1178,9 +1195,8 @@ export default async function formulaQueryBuilderv2( console.error(e); if (column) { - const formula = await column.getColOptions(); // add formula error to show in UI - await FormulaColumn.update(formula.id, { + await FormulaColumn.update(column.id, { error: e.message, }); // update cache to reflect the error in UI diff --git a/packages/nocodb/src/db/functionMappings/commonFns.ts b/packages/nocodb/src/db/functionMappings/commonFns.ts index 376086d65c..3a1c267075 100644 --- a/packages/nocodb/src/db/functionMappings/commonFns.ts +++ b/packages/nocodb/src/db/functionMappings/commonFns.ts @@ -184,6 +184,9 @@ export default { STRING(args: MapFnArgs) { return args.fn(args.pt?.arguments?.[0]); }, + BOOLEAN(args: MapFnArgs) { + return args.fn(args.pt?.arguments?.[0]); + }, AND: async (args: MapFnArgs) => { return { builder: args.knex.raw( diff --git a/packages/nocodb/src/db/functionMappings/pg.ts b/packages/nocodb/src/db/functionMappings/pg.ts index f6fa76b3da..5b36f24282 100644 --- a/packages/nocodb/src/db/functionMappings/pg.ts +++ b/packages/nocodb/src/db/functionMappings/pg.ts @@ -348,6 +348,15 @@ END ${colAlias}`, ), }; }, + BOOLEAN: async (args: MapFnArgs) => { + return { + builder: args.knex.raw( + `(${(await args.fn(args.pt.arguments[0])).builder})::boolean${ + args.colAlias + }`, + ), + }; + }, }; export default pg; diff --git a/packages/nocodb/src/models/FormulaColumn.ts b/packages/nocodb/src/models/FormulaColumn.ts index c9bd744f73..f073f69cd8 100644 --- a/packages/nocodb/src/models/FormulaColumn.ts +++ b/packages/nocodb/src/models/FormulaColumn.ts @@ -62,7 +62,7 @@ export default class FormulaColumn { id: string; static async update( - id: string, + columnId: string, formula: Partial & { parsed_tree?: any }, ncMeta = Noco.ncMeta, ) { @@ -75,7 +75,7 @@ export default class FormulaColumn { ]); // get existing cache - const key = `${CacheScope.COL_FORMULA}:${id}`; + const key = `${CacheScope.COL_FORMULA}:${columnId}`; let o = await NocoCache.get(key, CacheGetType.TYPE_OBJECT); if (o) { o = { ...o, ...updateObj }; @@ -85,7 +85,9 @@ export default class FormulaColumn { if ('parsed_tree' in updateObj) updateObj.parsed_tree = stringifyMetaProp(updateObj, 'parsed_tree'); // set meta - await ncMeta.metaUpdate(null, null, MetaTable.COL_FORMULA, updateObj, id); + await ncMeta.metaUpdate(null, null, MetaTable.COL_FORMULA, updateObj, { + fk_column_id: columnId, + }); } public getParsedTree() {