Merge pull request #3441 from IgniteUI/dpetev/tree-grid-child-batch-d…

…elete #3439 `generateRowPath` proper path order, cleanup extra calls
IgniteUI · Dec 13, 2018 · a9754d9 · a9754d9
2 parents 3c142f0 + 8349d66
commit a9754d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/projects/igniteui-angular/src/lib/grids/tree-grid/tree-grid-api.service.ts b/projects/igniteui-angular/src/lib/grids/tree-grid/tree-grid-api.service.ts
@@ -149,7 +149,7 @@ export class IgxTreeGridAPIService extends GridBaseAPIService<IgxTreeGridCompone
         rowCurrentValue: any,
         rowNewValue: {[x: string]: any}) {
         if (grid.transactions.enabled) {
-            const path = grid.generateRowPath(rowID).reverse();
+            const path = grid.generateRowPath(rowID);
             const transaction: HierarchicalTransaction = {
                 id: rowID,
                 type: TransactionType.UPDATE,

diff --git a/projects/igniteui-angular/src/lib/grids/tree-grid/tree-grid.component.ts b/projects/igniteui-angular/src/lib/grids/tree-grid/tree-grid.component.ts
@@ -361,9 +361,8 @@ export class IgxTreeGridComponent extends IgxGridBaseComponent {
                 if (this.transactions.enabled) {
                     const rowId = this.primaryKey ? data[this.primaryKey] : data;
                     const path: any[] = [];
-                    path.push(parentRowID);
                     path.push(...this.generateRowPath(parentRowID));
-                    path.reverse();
+                    path.push(parentRowID);
                     this.transactions.add({
                         id: rowId,
                         path: path,
@@ -467,7 +466,7 @@ export class IgxTreeGridComponent extends IgxGridBaseComponent {
             record = record.parent;
         }
 
-        return path;
+        return path.reverse();
     }
 
     /**