Merge pull request #8750 from IgniteUI/fix-filtering-unboundcol-m

Always emit onFilteringDone with proper event arguments

Author: kdinev

projects/igniteui-angular/src/lib/grids/filtering/grid-filtering.service.ts

@@ -216,9 +216,9 @@ export class IgxFilteringService implements OnDestroy {
                 this.gridAPI.filter(field, value, expressionForColumn.condition, filteringIgnoreCase);
             }
         }
-
+        const eventArgs = this.grid.filteringExpressionsTree.find(field) as FilteringExpressionsTree;
         // Wait for the change detection to update filtered data through the pipes and then emit the event.
-        requestAnimationFrame(() => this.grid.onFilteringDone.emit(col.filteringExpressionsTree));
+        requestAnimationFrame(() => this.grid.onFilteringDone.emit(eventArgs));
     }
 
     /**

projects/igniteui-angular/src/lib/grids/grid/grid-filtering.spec.ts

@@ -895,6 +895,17 @@ describe('IgxGrid - Filtering actions #grid', () => {
         expect(grid.rowList.length).toEqual(2);
     }));
 
+    it('Should always emit onFilteringDone with proper eventArgs, even when column does not exist', fakeAsync(() => {
+        spyOn(grid.onFilteringDone, 'emit');
+        grid.filteringLogic = FilteringLogic.Or;
+        grid.filter('Nonexisting', 'ignite', IgxStringFilteringOperand.instance().condition('contains'), true);
+        tick(100);
+        fix.detectChanges();
+        expect(grid.rowList.length).toEqual(0);
+        const args = grid.filteringExpressionsTree.find('Nonexisting') as FilteringExpressionsTree;
+        expect(grid.onFilteringDone.emit).toHaveBeenCalledWith(args);
+    }));
+
     it('Should emit onFilteringDone when filtering globally', fakeAsync(() => {
         spyOn(grid.onFilteringDone, 'emit');