Merge pull request #7413 from IgniteUI/dmdimitrov/issue7343-9.0.x

fix(grid): end grid edit on advanced filtering clear/apply - 9.0.x

Author: kdinev

projects/igniteui-angular/src/lib/grids/filtering/advanced-filtering/advanced-filtering-dialog.component.ts

@@ -1062,6 +1062,7 @@ export class IgxAdvancedFilteringDialogComponent implements AfterViewInit, OnDes
      * @hidden @internal
      */
     public onClearButtonClick() {
+        this.grid.endEdit(false);
         this.grid.advancedFilteringExpressionsTree = null;
     }
 
@@ -1078,6 +1079,7 @@ export class IgxAdvancedFilteringDialogComponent implements AfterViewInit, OnDes
      * @hidden @internal
      */
     public applyChanges() {
+        this.grid.endEdit(false);
         this.exitOperandEdit();
         this.grid.advancedFilteringExpressionsTree = this.createExpressionsTreeFromGroupItem(this.rootGroup);
     }

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

@@ -264,6 +264,44 @@ describe('IgxGrid - Cell Editing #grid', () => {
             expect(editCell.nativeElement.value).toBe('test');
             expect(firstCell.editMode).toBeTruthy();
         }));
+
+        it('should end cell editing when clearing or applying advanced filter', () => {
+            const cell = grid.getCellByColumn(0, 'fullName');
+
+            // Enter cell edit mode
+            UIInteractions.simulateDoubleClickAndSelectEvent(cell);
+            fixture.detectChanges();
+            expect(cell.editMode).toBe(true);
+
+            // Open Advanced Filtering dialog.
+            grid.openAdvancedFilteringDialog();
+            fixture.detectChanges();
+
+            // Clear the filters.
+            GridFunctions.clickAdvancedFilteringClearFilterButton(fixture);
+            fixture.detectChanges();
+
+            expect(cell.editMode).toBe(false);
+
+            // Close the dialog.
+            GridFunctions.clickAdvancedFilteringCancelButton(fixture);
+            fixture.detectChanges();
+
+            // Enter cell edit mode
+            UIInteractions.simulateDoubleClickAndSelectEvent(cell);
+            fixture.detectChanges();
+            expect(cell.editMode).toBe(true);
+
+            // Open Advanced Filtering dialog.
+            grid.openAdvancedFilteringDialog();
+            fixture.detectChanges();
+
+            // Apply the filters.
+            GridFunctions.clickAdvancedFilteringApplyButton(fixture);
+            fixture.detectChanges();
+
+            expect(cell.editMode).toBe(false);
+        });
     });
 
     describe('Scroll, pin and blur', () => {

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

@@ -340,6 +340,46 @@ describe('IgxGrid - Row Editing #grid', () => {
             cell.setEditMode(false);
             tick(16);
         }));
+
+        it('should end row editing when clearing or applying advanced filter', () => {
+            fix.detectChanges();
+            const row = grid.getRowByIndex(2);
+            const cell = grid.getCellByColumn(2, 'ProductName');
+
+            // Enter row edit mode
+            UIInteractions.simulateDoubleClickAndSelectEvent(cell);
+            fix.detectChanges();
+            expect(row.inEditMode).toBe(true);
+
+            // Open Advanced Filtering dialog.
+            grid.openAdvancedFilteringDialog();
+            fix.detectChanges();
+
+            // Clear the filters.
+            GridFunctions.clickAdvancedFilteringClearFilterButton(fix);
+            fix.detectChanges();
+
+            expect(row.inEditMode).toBe(false);
+
+            // Close the dialog.
+            GridFunctions.clickAdvancedFilteringCancelButton(fix);
+            fix.detectChanges();
+
+            // Enter row edit mode
+            UIInteractions.simulateDoubleClickAndSelectEvent(cell);
+            fix.detectChanges();
+            expect(row.inEditMode).toBe(true);
+
+            // Open Advanced Filtering dialog.
+            grid.openAdvancedFilteringDialog();
+            fix.detectChanges();
+
+            // Apply the filters.
+            GridFunctions.clickAdvancedFilteringApplyButton(fix);
+            fix.detectChanges();
+
+            expect(row.inEditMode).toBe(false);
+        });
     });
 
     describe('Navigation - Keyboard', () => {