Merge pull request #7176 from IgniteUI/mkirova/fix-rowPin-persistance

fix(igxGrid): Ensure previous row pin state is cleared before applyin…

Author: ChronosSF

projects/igniteui-angular/src/lib/grids/state.directive.spec.ts

@@ -414,6 +414,15 @@ describe('IgxGridState - input properties #grid', () => {
         expect(grid.pinnedRows[1].rowID).toBe(3);
         gridState = state.getState(true, 'rowPinning');
         expect(gridState).toBe(rowPinState);
+
+        grid.getRowByIndex(3).pin();
+
+        state.setState(rowPinStateObject);
+        fix.detectChanges();
+
+        expect(grid.pinnedRows.length).toBe(2);
+        expect(grid.pinnedRows[0].rowID).toBe(1);
+        expect(grid.pinnedRows[1].rowID).toBe(3);
     });
 
     it('setState should correctly restore grid cell selection state from string', () => {

projects/igniteui-angular/src/lib/grids/state.directive.ts

@@ -478,6 +478,8 @@ export class IgxGridStateDirective {
     }
 
     private restoreRowPinning(state: any[]) {
+        // clear current state.
+        this.grid.pinnedRows.forEach(row => row.unpin());
         state.forEach(rowID => this.grid.pinRow(rowID));
     }