Fix for issue #193

Fix for issue #193: IgHierarchicalGridComponent retains values from
previous binding when data source lengths are same

Author: MayaKirova

src/iggrid/iggridbase.ts

@@ -87,12 +87,10 @@ export class IgGridBase<Model> extends IgControlBase<Model> implements AfterCont
 			}
 			//check for changes in collection
 			this._changes = this._differ.diff(this._config.dataSource);
-			if (this._config.dataSource.length !== this._dataSource.length) {
+			if (this._changes) {
 				this._dataSource = jQuery.extend(true, [], this._config.dataSource);
-				if (this._changes) {
-					this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
-					this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]))
-				}
+				this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
+				this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]));
 			}
 			//check for changes in values
 			if (!this.equalsDiff(this._config.dataSource, this._dataSource, diff)) {

src/ighierarchicalgrid/ighierarchicalgrid.component.ts

@@ -37,12 +37,10 @@ export class IgHierarchicalGridComponent extends IgGridBase<IgHierarchicalGrid>
 
 			//check for changes in collection
 			this._changes = this._differ.diff(this._config.dataSource);
-			if (this._config.dataSource.length !== this._dataSource.length) {
+			if (this._changes) {
 				this._dataSource = jQuery.extend(true, [], this._config.dataSource);
-				if (this._changes) {
-					this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
-					this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]))
-				}
+				this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
+				this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]));
 			}
 			//check for changes in data source values
 			if (!this.equalsDiff(this._config.dataSource, this._dataSource, diff)) {

src/igtreegrid/igtreegrid.component.ts

@@ -39,12 +39,10 @@ export class IgTreeGridComponent extends IgGridBase<IgTreeGrid> {
 
 			//check for changes in collection
 			this._changes = this._differ.diff(this._config.dataSource);
-			if (this._config.dataSource.length !== this._dataSource.length) {
-				this._dataSource = jQuery.extend(true, [], this._config.dataSource);
-				if (this._changes) {
-					this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
-					this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]))
-				}
+			if (this._changes) {
+				this._dataSource = jQuery.extend(true, [], this._config.dataSource);				
+				this._changes.forEachAddedItem(r => this.addRow(r.item, r.currentIndex));
+				this._changes.forEachRemovedItem(r => this.deleteRow(r.item[pkKey]));
 			}
 			//check for changes in values
 			if (!this.equalsDiff(this._config.dataSource, this._dataSource, diff)) {