Merge pull request #7641 from IgniteUI/pbozhinov/fix-7615-10

Fixes issue with hierarchical transaction commit emit count

Author: ChronosSF

projects/igniteui-angular/src/lib/services/transaction/igx-hierarchical-transaction.ts

@@ -55,7 +55,7 @@ export class IgxHierarchicalTransactionService<T extends HierarchicalTransaction
      * Applies all transactions over the provided data
      * @param data Data source to update
      * @param primaryKey Primary key of the hierarchical data
-     * @param childDataKey Kye of child data collection
+     * @param childDataKey Key of child data collection
      * @param id Optional record id to commit transactions for
      */
     public commit(data: any[], primaryKey?: any, childDataKey?: any, id?: any): void {
@@ -65,10 +65,10 @@ export class IgxHierarchicalTransactionService<T extends HierarchicalTransaction
                 transactions = transactions.filter(t => t.id === id);
             }
             DataUtil.mergeHierarchicalTransactions(data, transactions, childDataKey, primaryKey, true);
+            this.clear(id);
         } else {
             super.commit(data, id);
         }
-        this.clear(id);
     }
 
     //  TODO: remove this method. Force cloning to strip child arrays when needed instead

projects/igniteui-angular/src/lib/services/transaction/igx-transaction.spec.ts

@@ -945,5 +945,26 @@ describe('IgxTransaction', () => {
             expect(transaction.canUndo).toBeFalsy();
             expect(transaction.getAggregatedChanges(false).length).toBe(0);
         });
+
+        it('Should emit onStateUpdate once when commiting a hierarchical transaction', () => {
+            const data = SampleTestData.employeeTreeData();
+            const transaction = new IgxHierarchicalTransactionService();
+            spyOn(transaction.onStateUpdate, 'emit').and.callThrough();
+            expect(transaction).toBeDefined();
+
+            const updateTransaction: HierarchicalTransaction = {
+                id: 475,
+                type: TransactionType.UPDATE,
+                newValue: {
+                    Age: 60
+                },
+                path: [data[0].ID]
+            };
+            transaction.add(updateTransaction, data[0].Employees[0]);
+            expect(transaction.onStateUpdate.emit).toHaveBeenCalledTimes(1);
+
+            transaction.commit(data, 'ID');
+            expect(transaction.onStateUpdate.emit).toHaveBeenCalledTimes(2);
+        });
     });
 });