Merge pull request #5744 from IgniteUI/ddincheva/fixIssue5551-master

Should not throw error when you filter data column

Author: zdrawku

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

@@ -390,6 +390,7 @@ export class IgxDateFilteringOperand extends IgxFilteringOperand {
     }
 
     protected findValueInSet(target: any, searchVal: Set<any>) {
+        if (!target) { return false; }
         return searchVal.has(new Date(target.getFullYear(), target.getMonth(), target.getDate()).toISOString());
     }
 }

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

@@ -4843,6 +4843,34 @@ describe('IgxGrid - Filtering actions - Excel style filtering #grid', () => {
                 new Set(['Ignite UI for Angular', 'Ignite UI for JavaScript', 'NetAdvantage']));
         }));
 
+        it('Should not throw error when selecting more than two values and column dataType is date.', fakeAsync(() => {
+            fix.detectChanges();
+
+            const headers: DebugElement[] = fix.debugElement.queryAll(By.directive(IgxGridHeaderGroupComponent));
+            const headerResArea = headers[4].children[0].nativeElement;
+
+            const filterIcon = headerResArea.querySelector('.igx-excel-filter__icon');
+            filterIcon.click();
+            fix.detectChanges();
+
+            const excelMenu = grid.nativeElement.querySelector('.igx-excel-filter__menu');
+            const checkbox = excelMenu.querySelectorAll('.igx-checkbox__input');
+            const applyButton = excelMenu.querySelector('.igx-button--raised');
+
+            checkbox[0].click(); // Select All
+            tick();
+            fix.detectChanges();
+
+            checkbox[2].click();
+            checkbox[3].click();
+            checkbox[4].click();
+            checkbox[6].click();
+            tick();
+            fix.detectChanges();
+
+            expect(() => { applyButton.click(); }).not.toThrowError();
+        }));
+
         it('Should generate "in" and "empty" conditions when selecting more than two values including (Blanks).', fakeAsync(() => {
             fix.detectChanges();