Merge branch 'master' into workflows

Author: kdinev

projects/igniteui-angular/src/lib/data-operations/filtering-strategy.spec.ts

@@ -49,6 +49,30 @@ describe('Unit testing FilteringStrategy', () => {
         const res = fs.matchRecord(rec, expressionTree);
         expect(res).toBeTruthy();
     });
+    it ('tests `findMatchByExpression` for working with filtering operands with missing condition', () => {
+        const rec = data[0];
+        const expressionTree = JSON.parse('{"filteringOperands":[{"fieldName":"Missing","condition":{"name":"notEmpty","isUnary":true,"iconName":"filter_not_empty"},"conditionName":"notEmpty","ignoreCase":true,"searchVal":null,"searchTree":null}],"operator":0,"returnFields":[],"type":1}');
+
+
+        const res = fs.matchRecord(rec, expressionTree);
+        expect(res).toBeFalsy();
+    });
+
+    it ('no error when condition is missing in the filtering expressions tree', () => {
+        const rec = data[0];
+        const expressionTree = new FilteringExpressionsTree(FilteringLogic.Or);
+        expressionTree.filteringOperands = [
+            {
+                conditionName: 'contains',
+                fieldName: 'string',
+                ignoreCase: false,
+                searchVal: 'ROW'
+            }
+        ];
+        const res = fs.matchRecord(rec, expressionTree);
+        expect(res).toBeFalsy();
+    });
+
     it ('tests `findMatch`', () => {
         const rec = data[0];
         const res = fs.findMatchByExpression(rec, {

projects/igniteui-angular/src/lib/data-operations/filtering-strategy.ts

@@ -39,9 +39,10 @@ export interface IgxFilterItem {
 export abstract class BaseFilteringStrategy implements IFilteringStrategy  {
     // protected
     public findMatchByExpression(rec: any, expr: IFilteringExpression, isDate?: boolean, isTime?: boolean, grid?: GridType): boolean {
-        const cond = expr.condition;
         const val = this.getFieldValue(rec, expr.fieldName, isDate, isTime, grid);
-        return cond.logic(val, expr.searchVal, expr.ignoreCase);
+        if (expr.condition?.logic) {
+            return expr.condition.logic(val, expr.searchVal, expr.ignoreCase);
+        }
     }
 
     // protected