update uses of subExpression with and

actualbudget · Nov 3, 2024 · e875016 · e875016
1 parent 6984f03
commit e875016
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/packages/loot-core/src/server/accounts/rules.ts b/packages/loot-core/src/server/accounts/rules.ts
@@ -204,7 +204,7 @@ const CONDITION_TYPES = {
     ],
     nullable: true,
     parse(op, value, fieldName) {
-      if (op === 'oneOf' || op === 'notOneOf' || op === 'subExpression') {
+      if (op === 'oneOf' || op === 'notOneOf' || op === 'and') {
         assert(
           Array.isArray(value),
           'no-empty-array',

diff --git a/packages/loot-core/src/server/accounts/transaction-rules.test.ts b/packages/loot-core/src/server/accounts/transaction-rules.test.ts
@@ -510,7 +510,7 @@ describe('Transaction rules', () => {
     // todo: isapprox
   });
 
-  test('subexpression builds $and condition', async () => {
+  test('and sub expression builds $and condition', async () => {
     const conds = [{ field: 'category', op: 'is', value: null }];
     debugger;
     const { filters } = conditionsToAQL(conds);

diff --git a/packages/loot-core/src/server/accounts/transaction-rules.ts b/packages/loot-core/src/server/accounts/transaction-rules.ts
@@ -295,7 +295,7 @@ function conditionSpecialCases(cond: Condition): Condition {
   //special cases that require multiple conditions
   if (cond.op === 'is' && cond.field === 'category' && cond.value === null) {
     return new Condition(
-      'subExpression',
+      'and',
       cond.field,
       [
         cond,
@@ -310,7 +310,7 @@ function conditionSpecialCases(cond: Condition): Condition {
     cond.value === null
   ) {
     return new Condition(
-      'subExpression',
+      'and',
       cond.field,
       [cond, new Condition('is', 'parent', false, null)],
       {},
@@ -541,7 +541,7 @@ export function conditionsToAQL(conditions, { recurDateBounds = 100 } = {}) {
         return apply(field, '$eq', true);
       case 'false':
         return apply(field, '$eq', false);
-      case 'subExpression':
+      case 'and':
         debugger;
         return {
           $and: getValue(value).map(subExpr => mapConditionToActualQL(subExpr)),

diff --git a/packages/loot-core/src/shared/rules.ts b/packages/loot-core/src/shared/rules.ts
@@ -20,7 +20,7 @@ const TYPE_INFO = {
       'doesNotContain',
       'notOneOf',
       'hasTags',
-      'subExpression',
+      'and',
     ],
     nullable: true,
   },