@ag-grid-enterprise/row-grouping - npm Package Compare versions

Comparing version 30.1.0 to 30.2.0

dist/cjs/es5/rowGrouping/aggregationStage.d.ts

@@ -6,8 +6,2 @@ import { BeanStub, IRowNodeStage, RowNode, Column, StageExecuteParams, IAggFunc } from "@ag-grid-community/core";
     private aggFuncService;
-    private gridApi;
-    private columnApi;
-    private filteredOnly;
-    private alwaysAggregateAtRootLevel;
-    private groupIncludeTotalFooter;
-    private init;
     execute(params: StageExecuteParams): any;
@@ -14,0 +8,0 @@ private createAggDetails;

dist/cjs/es5/rowGrouping/aggregationStage.js

@@ -31,9 +31,2 @@ "use strict";
     }
-    AggregationStage.prototype.init = function () {
-        var _this = this;
-        this.alwaysAggregateAtRootLevel = this.gridOptionsService.is('alwaysAggregateAtRootLevel');
-        this.addManagedPropertyListener('alwaysAggregateAtRootLevel', function (propChange) { return _this.alwaysAggregateAtRootLevel = propChange.currentValue; });
-        this.groupIncludeTotalFooter = this.gridOptionsService.is('groupIncludeTotalFooter');
-        this.addManagedPropertyListener('groupIncludeTotalFooter', function (propChange) { return _this.groupIncludeTotalFooter = propChange.currentValue; });
-    };
     // it's possible to recompute the aggregate without doing the other parts
@@ -61,5 +54,9 @@ // + api.refreshClientSideRowModel('aggregate')
         var aggDetails = {
+            alwaysAggregateAtRootLevel: this.gridOptionsService.is('alwaysAggregateAtRootLevel'),
+            groupIncludeTotalFooter: this.gridOptionsService.is('groupIncludeTotalFooter'),
             changedPath: params.changedPath,
             valueColumns: measureColumns,
-            pivotColumns: pivotColumns
+            pivotColumns: pivotColumns,
+            filteredOnly: !this.isSuppressAggFilteredOnly(),
+            userAggFunc: this.gridOptionsService.getCallback('getGroupRowAgg'),
         };
@@ -74,4 +71,2 @@ return aggDetails;
         var _this = this;
-        // update prop, in case changed since last time
-        this.filteredOnly = !this.isSuppressAggFilteredOnly();
         var callback = function (rowNode) {
@@ -91,5 +86,5 @@ var hasNoChildren = !rowNode.hasChildren();
             // if total footer is displayed, the value is in use
-            if (isRootNode && !_this.groupIncludeTotalFooter) {
+            if (isRootNode && !aggDetails.groupIncludeTotalFooter) {
                 var notPivoting = !_this.columnModel.isPivotMode();
-                if (!_this.alwaysAggregateAtRootLevel && notPivoting) {
+                if (!aggDetails.alwaysAggregateAtRootLevel && notPivoting) {
                     return;
@@ -105,7 +100,5 @@ }
         var pivotColumnsMissing = aggDetails.pivotColumns.length === 0;
-        var userFunc = this.gridOptionsService.getCallback('getGroupRowAgg');
         var aggResult;
-        if (userFunc) {
-            var params = { nodes: rowNode.childrenAfterFilter };
-            aggResult = userFunc(params);
+        if (aggDetails.userAggFunc) {
+            aggResult = aggDetails.userAggFunc({ nodes: rowNode.childrenAfterFilter });
         }
@@ -176,3 +169,3 @@ else if (measureColumnsMissing) {
             : null;
-        var values2d = this.getValuesNormal(rowNode, changedValueColumns);
+        var values2d = this.getValuesNormal(rowNode, changedValueColumns, aggDetails.filteredOnly);
         var oldValues = rowNode.aggData;
@@ -211,3 +204,3 @@ changedValueColumns.forEach(function (valueColumn, index) {
     };
-    AggregationStage.prototype.getValuesNormal = function (rowNode, valueColumns) {
+    AggregationStage.prototype.getValuesNormal = function (rowNode, valueColumns, filteredOnly) {
         // create 2d array, of all values for all valueColumns
@@ -217,3 +210,3 @@ var values = [];
         var valueColumnCount = valueColumns.length;
-        var nodeList = this.filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
+        var nodeList = filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
         var rowCount = nodeList.length;
@@ -248,4 +241,4 @@ for (var i = 0; i < rowCount; i++) {
             data: rowNode ? rowNode.data : undefined,
-            api: this.gridApi,
-            columnApi: this.columnApi,
+            api: this.gridOptionsService.api,
+            columnApi: this.gridOptionsService.columnApi,
             context: this.gridOptionsService.context,
@@ -264,11 +257,2 @@ }; // the "as any" is needed to allow the deprecation warning messages
     ], AggregationStage.prototype, "aggFuncService", void 0);
-    __decorate([
-        core_1.Autowired('gridApi')
-    ], AggregationStage.prototype, "gridApi", void 0);
-    __decorate([
-        core_1.Autowired('columnApi')
-    ], AggregationStage.prototype, "columnApi", void 0);
-    __decorate([
-        core_1.PostConstruct
-    ], AggregationStage.prototype, "init", null);
     AggregationStage = __decorate([
@@ -275,0 +259,0 @@ core_1.Bean('aggregationStage')

dist/cjs/es5/rowGrouping/columnDropZones/dropZoneColumnComp.js

@@ -130,3 +130,3 @@ "use strict";
             eElement: this.eDragHandle,
-            defaultIconName: core_1.DragAndDropService.ICON_HIDE,
+            getDefaultIconName: function () { return core_1.DragAndDropService.ICON_HIDE; },
             getDragItem: function () { return _this.createDragItem(); },
@@ -133,0 +133,0 @@ dragItemName: this.displayName,

dist/cjs/es5/rowGrouping/filterAggregatesStage.js

@@ -111,3 +111,3 @@ "use strict";
         }
-        if (this.gridOptionsService.isTreeData()) {
+        if (this.gridOptionsService.is('treeData')) {
             this.setAllChildrenCountTreeData(rowNode);
@@ -114,0 +114,0 @@ }

dist/cjs/es5/rowGrouping/groupStage.d.ts

@@ -8,8 +8,5 @@ import { BeanStub, IRowNodeStage, StageExecuteParams } from "@ag-grid-community/core";
     private selectionService;
-    private usingTreeData;
-    private getDataPath;
     private groupIdSequence;
     private oldGroupingDetails;
     private oldGroupDisplayColIds;
-    private postConstruct;
     execute(params: StageExecuteParams): void;
@@ -16,0 +13,0 @@ private positionLeafsAndGroups;

dist/cjs/es5/rowGrouping/groupStage.js

@@ -58,8 +58,2 @@ "use strict";
     }
-    GroupStage.prototype.postConstruct = function () {
-        this.usingTreeData = this.gridOptionsService.isTreeData();
-        if (this.usingTreeData) {
-            this.getDataPath = this.gridOptionsService.get('getDataPath');
-        }
-    };
     GroupStage.prototype.execute = function (params) {
@@ -74,11 +68,10 @@ var details = this.createGroupingDetails(params);
         }
-        this.positionLeafsAndGroups(params.changedPath);
-        this.orderGroups(details.rootNode);
+        if (!details.usingTreeData) {
+            // we don't do group sorting for tree data
+            this.positionLeafsAndGroups(params.changedPath);
+            this.orderGroups(details);
+        }
         this.selectableService.updateSelectableAfterGrouping(details.rootNode);
     };
     GroupStage.prototype.positionLeafsAndGroups = function (changedPath) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
         changedPath.forEachChangedNodeDepthFirst(function (group) {
@@ -112,7 +105,7 @@ if (group.childrenAfterGroup) {
         var rowNode = params.rowNode, changedPath = params.changedPath, rowNodeTransactions = params.rowNodeTransactions, rowNodeOrder = params.rowNodeOrder;
-        var groupedCols = this.usingTreeData ? null : this.columnModel.getRowGroupColumns();
+        var usingTreeData = this.gridOptionsService.is('treeData');
+        var groupedCols = usingTreeData ? null : this.columnModel.getRowGroupColumns();
         var details = {
             // someone complained that the parent attribute was causing some change detection
-            // to break is some angular add-on - which i never used. taking the parent out breaks
-            // a cyclic dependency, hence this flag got introduced.
+            // to break in an angular add-on.  Taking the parent out breaks a cyclic dependency, hence this flag got introduced.
             includeParents: !this.gridOptionsService.is('suppressParentsInRowNodes'),
@@ -123,7 +116,13 @@ expandByDefault: this.gridOptionsService.getNum('groupDefaultExpanded'),
             pivotMode: this.columnModel.isPivotMode(),
-            groupedColCount: this.usingTreeData || !groupedCols ? 0 : groupedCols.length,
+            groupedColCount: usingTreeData || !groupedCols ? 0 : groupedCols.length,
             rowNodeOrder: rowNodeOrder,
             transactions: rowNodeTransactions,
             // if no transaction, then it's shotgun, changed path would be 'not active' at this point anyway
-            changedPath: changedPath
+            changedPath: changedPath,
+            groupAllowUnbalanced: this.gridOptionsService.is('groupAllowUnbalanced'),
+            isGroupOpenByDefault: this.gridOptionsService.getCallback('isGroupOpenByDefault'),
+            initialGroupOrderComparator: this.gridOptionsService.getCallback('initialGroupOrderComparator'),
+            createGroupFooter: this.gridOptionsService.isGroupIncludeFooterTrueOrCallback(),
+            usingTreeData: usingTreeData,
+            getDataPath: usingTreeData ? this.gridOptionsService.get('getDataPath') : undefined
         };
@@ -140,3 +139,3 @@ return details;
             // modified to work, however for now I don't have the brain energy to work it all out).
-            var batchRemover = !_this.usingTreeData ? new batchRemover_1.BatchRemover() : undefined;
+            var batchRemover = !details.usingTreeData ? new batchRemover_1.BatchRemover() : undefined;
             // the order here of [add, remove, update] needs to be the same as in ClientSideNodeManager,
@@ -178,10 +177,6 @@ // as the order is important when a record with the same id is added and removed in the same
     };
-    GroupStage.prototype.orderGroups = function (rootNode) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
-        var comparator = this.gridOptionsService.getCallback('initialGroupOrderComparator');
+    GroupStage.prototype.orderGroups = function (details) {
+        var comparator = details.initialGroupOrderComparator;
         if (core_1._.exists(comparator)) {
-            recursiveSort(rootNode);
+            recursiveSort(details.rootNode);
         }
@@ -202,3 +197,3 @@ function recursiveSort(rowNode) {
         // but when doing grid grouping, the node is not part of the path so we start with the parent.
-        var pointer = this.usingTreeData ? node : node.parent;
+        var pointer = details.usingTreeData ? node : node.parent;
         while (pointer && pointer !== details.rootNode) {
@@ -256,3 +251,3 @@ res.push({
         this.removeNodesFromParents(leafRowNodes, details, batchRemover);
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.postRemoveCreateFillerNodes(leafRowNodes, details);
@@ -400,3 +395,3 @@ // When not TreeData, then removeEmptyGroups is called just before the BatchRemover is flushed.
             rowNodes.forEach(function (rowNode) {
-                var isLeafNode = !_this.usingTreeData && !rowNode.group;
+                var isLeafNode = !details.usingTreeData && !rowNode.group;
                 if (isLeafNode) {
@@ -410,3 +405,3 @@ return;
                 };
-                _this.setGroupData(rowNode, groupInfo);
+                _this.setGroupData(rowNode, groupInfo, details);
                 recurse(rowNode.childrenAfterGroup);
@@ -428,3 +423,3 @@ });
         // isn't allowed with treeData, so the grid never actually use .leafGroup when doing treeData.
-        rootNode.leafGroup = this.usingTreeData ? false : groupedCols.length === 0;
+        rootNode.leafGroup = details.usingTreeData ? false : groupedCols.length === 0;
         // we are doing everything from scratch, so reset childrenAfterGroup and childrenMapped from the rootNode
@@ -449,3 +444,3 @@ rootNode.childrenAfterGroup = [];
             // and the group cols have changed.
-            noFurtherProcessingNeeded = this.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
+            noFurtherProcessingNeeded = details.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
             // if the group display cols have changed, then we need to update rowNode.groupData
@@ -476,3 +471,3 @@ // (regardless of tree data or row grouping)
         }
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.swapGroupWithUserNode(parentGroup, childNode, isMove);
@@ -544,3 +539,3 @@ }
         groupNode.rowGroupColumn = groupInfo.rowGroupColumn;
-        this.setGroupData(groupNode, groupInfo);
+        this.setGroupData(groupNode, groupInfo, details);
         // we put 'row-group-' before the group id, so it doesn't clash with standard row id's. we also use 't-' and 'b-'
@@ -551,3 +546,3 @@ // for top pinned and bottom pinned rows.
         groupNode.level = level;
-        groupNode.leafGroup = this.usingTreeData ? false : level === (details.groupedColCount - 1);
+        groupNode.leafGroup = details.usingTreeData ? false : level === (details.groupedColCount - 1);
         groupNode.allLeafChildren = [];
@@ -557,3 +552,3 @@ // why is this done here? we are not updating the children count as we go,
         groupNode.setAllChildrenCount(0);
-        groupNode.rowGroupIndex = this.usingTreeData ? null : level;
+        groupNode.rowGroupIndex = details.usingTreeData ? null : level;
         groupNode.childrenAfterGroup = [];
@@ -564,9 +559,5 @@ groupNode.childrenMapped = {};
         this.setExpandedInitialValue(details, groupNode);
-        if (this.gridOptionsService.is('groupIncludeFooter')) {
-            groupNode.createFooter();
-        }
         return groupNode;
     };
-    GroupStage.prototype.setGroupData = function (groupNode, groupInfo) {
-        var _this = this;
+    GroupStage.prototype.setGroupData = function (groupNode, groupInfo, details) {
         groupNode.groupData = {};
@@ -577,3 +568,3 @@ var groupDisplayCols = this.columnModel.getGroupDisplayColumns();
             // if rowGroupColumn is present, then it's grid row grouping and we only include if configuration says so
-            var displayGroupForCol = _this.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
+            var displayGroupForCol = details.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
             if (displayGroupForCol) {
@@ -599,3 +590,3 @@ groupNode.groupData[col.getColId()] = groupInfo.key;
         // use callback if exists
-        var userCallback = this.gridOptionsService.getCallback('isGroupOpenByDefault');
+        var userCallback = details.isGroupOpenByDefault;
         if (userCallback) {
@@ -622,9 +613,9 @@ var params = {
     GroupStage.prototype.getGroupInfo = function (rowNode, details) {
-        if (this.usingTreeData) {
-            return this.getGroupInfoFromCallback(rowNode);
+        if (details.usingTreeData) {
+            return this.getGroupInfoFromCallback(rowNode, details);
         }
         return this.getGroupInfoFromGroupColumns(rowNode, details);
     };
-    GroupStage.prototype.getGroupInfoFromCallback = function (rowNode) {
-        var keys = this.getDataPath ? this.getDataPath(rowNode.data) : null;
+    GroupStage.prototype.getGroupInfoFromCallback = function (rowNode, details) {
+        var keys = details.getDataPath ? details.getDataPath(rowNode.data) : null;
         if (keys === null || keys === undefined || keys.length === 0) {
@@ -645,3 +636,3 @@ core_1._.doOnce(function () { return console.warn("AG Grid: getDataPath() should not return an empty path for data", rowNode.data); }, 'groupStage.getGroupInfoFromCallback');
             // this keeps the tree balanced and hence can be represented as a group.
-            var createGroupForEmpty = details.pivotMode || !_this.gridOptionsService.is('groupAllowUnbalanced');
+            var createGroupForEmpty = details.pivotMode || !details.groupAllowUnbalanced;
             if (createGroupForEmpty && !keyExists) {
@@ -677,5 +668,2 @@ key = '';
     ], GroupStage.prototype, "selectionService", void 0);
-    __decorate([
-        core_1.PostConstruct
-    ], GroupStage.prototype, "postConstruct", null);
     GroupStage = __decorate([
@@ -682,0 +670,0 @@ core_1.Bean('groupStage')

dist/cjs/es5/rowGrouping/pivotColDefService.js

@@ -66,7 +66,8 @@ "use strict";
         var _this = this;
-        var _a, _b;
-        this.fieldSeparator = (_a = this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_';
-        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', function (propChange) { return _this.fieldSeparator = propChange.currentValue; });
-        this.pivotDefaultExpanded = (_b = this.gos.getNum('pivotDefaultExpanded')) !== null && _b !== void 0 ? _b : 0;
-        this.addManagedPropertyListener('pivotDefaultExpanded', function (propChange) { return _this.pivotDefaultExpanded = propChange.currentValue; });
+        var getFieldSeparator = function () { var _a; return (_a = _this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_'; };
+        this.fieldSeparator = getFieldSeparator();
+        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', function () { _this.fieldSeparator = getFieldSeparator(); });
+        var getPivotDefaultExpanded = function () { var _a; return (_a = _this.gos.getNum('pivotDefaultExpanded')) !== null && _a !== void 0 ? _a : 0; };
+        this.pivotDefaultExpanded = getPivotDefaultExpanded();
+        this.addManagedPropertyListener('pivotDefaultExpanded', function () { _this.pivotDefaultExpanded = getPivotDefaultExpanded(); });
     };
@@ -73,0 +74,0 @@ PivotColDefService.prototype.createPivotColumnDefs = function (uniqueValues) {

dist/cjs/es5/version.d.ts

@@ -1,1 +0,1 @@
-export declare const VERSION = "30.1.0";
+export declare const VERSION = "30.2.0";

dist/cjs/es5/version.js

@@ -5,2 +5,2 @@ "use strict";
 // DO NOT UPDATE MANUALLY: Generated from script during build time
-exports.VERSION = '30.1.0';
+exports.VERSION = '30.2.0';

dist/cjs/es6/rowGrouping/aggregationStage.d.ts

@@ -6,8 +6,2 @@ import { BeanStub, IRowNodeStage, RowNode, Column, StageExecuteParams, IAggFunc } from "@ag-grid-community/core";
     private aggFuncService;
-    private gridApi;
-    private columnApi;
-    private filteredOnly;
-    private alwaysAggregateAtRootLevel;
-    private groupIncludeTotalFooter;
-    private init;
     execute(params: StageExecuteParams): any;
@@ -14,0 +8,0 @@ private createAggDetails;

dist/cjs/es6/rowGrouping/aggregationStage.js

@@ -12,8 +12,2 @@ "use strict";
 let AggregationStage = class AggregationStage extends core_1.BeanStub {
-    init() {
-        this.alwaysAggregateAtRootLevel = this.gridOptionsService.is('alwaysAggregateAtRootLevel');
-        this.addManagedPropertyListener('alwaysAggregateAtRootLevel', (propChange) => this.alwaysAggregateAtRootLevel = propChange.currentValue);
-        this.groupIncludeTotalFooter = this.gridOptionsService.is('groupIncludeTotalFooter');
-        this.addManagedPropertyListener('groupIncludeTotalFooter', (propChange) => this.groupIncludeTotalFooter = propChange.currentValue);
-    }
     // it's possible to recompute the aggregate without doing the other parts
@@ -41,5 +35,9 @@ // + api.refreshClientSideRowModel('aggregate')
         const aggDetails = {
+            alwaysAggregateAtRootLevel: this.gridOptionsService.is('alwaysAggregateAtRootLevel'),
+            groupIncludeTotalFooter: this.gridOptionsService.is('groupIncludeTotalFooter'),
             changedPath: params.changedPath,
             valueColumns: measureColumns,
-            pivotColumns: pivotColumns
+            pivotColumns: pivotColumns,
+            filteredOnly: !this.isSuppressAggFilteredOnly(),
+            userAggFunc: this.gridOptionsService.getCallback('getGroupRowAgg'),
         };
@@ -53,4 +51,2 @@ return aggDetails;
     recursivelyCreateAggData(aggDetails) {
-        // update prop, in case changed since last time
-        this.filteredOnly = !this.isSuppressAggFilteredOnly();
         const callback = (rowNode) => {
@@ -70,5 +66,5 @@ const hasNoChildren = !rowNode.hasChildren();
             // if total footer is displayed, the value is in use
-            if (isRootNode && !this.groupIncludeTotalFooter) {
+            if (isRootNode && !aggDetails.groupIncludeTotalFooter) {
                 const notPivoting = !this.columnModel.isPivotMode();
-                if (!this.alwaysAggregateAtRootLevel && notPivoting) {
+                if (!aggDetails.alwaysAggregateAtRootLevel && notPivoting) {
                     return;
@@ -84,7 +80,5 @@ }
         const pivotColumnsMissing = aggDetails.pivotColumns.length === 0;
-        const userFunc = this.gridOptionsService.getCallback('getGroupRowAgg');
         let aggResult;
-        if (userFunc) {
-            const params = { nodes: rowNode.childrenAfterFilter };
-            aggResult = userFunc(params);
+        if (aggDetails.userAggFunc) {
+            aggResult = aggDetails.userAggFunc({ nodes: rowNode.childrenAfterFilter });
         }
@@ -153,3 +147,3 @@ else if (measureColumnsMissing) {
             : null;
-        const values2d = this.getValuesNormal(rowNode, changedValueColumns);
+        const values2d = this.getValuesNormal(rowNode, changedValueColumns, aggDetails.filteredOnly);
         const oldValues = rowNode.aggData;
@@ -187,3 +181,3 @@ changedValueColumns.forEach((valueColumn, index) => {
     }
-    getValuesNormal(rowNode, valueColumns) {
+    getValuesNormal(rowNode, valueColumns, filteredOnly) {
         // create 2d array, of all values for all valueColumns
@@ -193,3 +187,3 @@ const values = [];
         const valueColumnCount = valueColumns.length;
-        const nodeList = this.filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
+        const nodeList = filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
         const rowCount = nodeList.length;
@@ -224,4 +218,4 @@ for (let i = 0; i < rowCount; i++) {
             data: rowNode ? rowNode.data : undefined,
-            api: this.gridApi,
-            columnApi: this.columnApi,
+            api: this.gridOptionsService.api,
+            columnApi: this.gridOptionsService.columnApi,
             context: this.gridOptionsService.context,
@@ -241,11 +235,2 @@ }; // the "as any" is needed to allow the deprecation warning messages
 ], AggregationStage.prototype, "aggFuncService", void 0);
-__decorate([
-    core_1.Autowired('gridApi')
-], AggregationStage.prototype, "gridApi", void 0);
-__decorate([
-    core_1.Autowired('columnApi')
-], AggregationStage.prototype, "columnApi", void 0);
-__decorate([
-    core_1.PostConstruct
-], AggregationStage.prototype, "init", null);
 AggregationStage = __decorate([
@@ -252,0 +237,0 @@ core_1.Bean('aggregationStage')

dist/cjs/es6/rowGrouping/columnDropZones/dropZoneColumnComp.js

@@ -109,3 +109,3 @@ "use strict";
             eElement: this.eDragHandle,
-            defaultIconName: core_1.DragAndDropService.ICON_HIDE,
+            getDefaultIconName: () => core_1.DragAndDropService.ICON_HIDE,
             getDragItem: () => this.createDragItem(),
@@ -112,0 +112,0 @@ dragItemName: this.displayName,

dist/cjs/es6/rowGrouping/filterAggregatesStage.js

@@ -90,3 +90,3 @@ "use strict";
         }
-        if (this.gridOptionsService.isTreeData()) {
+        if (this.gridOptionsService.is('treeData')) {
             this.setAllChildrenCountTreeData(rowNode);
@@ -93,0 +93,0 @@ }

dist/cjs/es6/rowGrouping/groupStage.d.ts

@@ -8,8 +8,5 @@ import { BeanStub, IRowNodeStage, StageExecuteParams } from "@ag-grid-community/core";
     private selectionService;
-    private usingTreeData;
-    private getDataPath;
     private groupIdSequence;
     private oldGroupingDetails;
     private oldGroupDisplayColIds;
-    private postConstruct;
     execute(params: StageExecuteParams): void;
@@ -16,0 +13,0 @@ private positionLeafsAndGroups;

dist/cjs/es6/rowGrouping/groupStage.js

@@ -20,8 +20,2 @@ "use strict";
     }
-    postConstruct() {
-        this.usingTreeData = this.gridOptionsService.isTreeData();
-        if (this.usingTreeData) {
-            this.getDataPath = this.gridOptionsService.get('getDataPath');
-        }
-    }
     execute(params) {
@@ -36,11 +30,10 @@ const details = this.createGroupingDetails(params);
         }
-        this.positionLeafsAndGroups(params.changedPath);
-        this.orderGroups(details.rootNode);
+        if (!details.usingTreeData) {
+            // we don't do group sorting for tree data
+            this.positionLeafsAndGroups(params.changedPath);
+            this.orderGroups(details);
+        }
         this.selectableService.updateSelectableAfterGrouping(details.rootNode);
     }
     positionLeafsAndGroups(changedPath) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
         changedPath.forEachChangedNodeDepthFirst(group => {
@@ -74,7 +67,7 @@ if (group.childrenAfterGroup) {
         const { rowNode, changedPath, rowNodeTransactions, rowNodeOrder } = params;
-        const groupedCols = this.usingTreeData ? null : this.columnModel.getRowGroupColumns();
+        const usingTreeData = this.gridOptionsService.is('treeData');
+        const groupedCols = usingTreeData ? null : this.columnModel.getRowGroupColumns();
         const details = {
             // someone complained that the parent attribute was causing some change detection
-            // to break is some angular add-on - which i never used. taking the parent out breaks
-            // a cyclic dependency, hence this flag got introduced.
+            // to break in an angular add-on.  Taking the parent out breaks a cyclic dependency, hence this flag got introduced.
             includeParents: !this.gridOptionsService.is('suppressParentsInRowNodes'),
@@ -85,7 +78,13 @@ expandByDefault: this.gridOptionsService.getNum('groupDefaultExpanded'),
             pivotMode: this.columnModel.isPivotMode(),
-            groupedColCount: this.usingTreeData || !groupedCols ? 0 : groupedCols.length,
+            groupedColCount: usingTreeData || !groupedCols ? 0 : groupedCols.length,
             rowNodeOrder: rowNodeOrder,
             transactions: rowNodeTransactions,
             // if no transaction, then it's shotgun, changed path would be 'not active' at this point anyway
-            changedPath: changedPath
+            changedPath: changedPath,
+            groupAllowUnbalanced: this.gridOptionsService.is('groupAllowUnbalanced'),
+            isGroupOpenByDefault: this.gridOptionsService.getCallback('isGroupOpenByDefault'),
+            initialGroupOrderComparator: this.gridOptionsService.getCallback('initialGroupOrderComparator'),
+            createGroupFooter: this.gridOptionsService.isGroupIncludeFooterTrueOrCallback(),
+            usingTreeData: usingTreeData,
+            getDataPath: usingTreeData ? this.gridOptionsService.get('getDataPath') : undefined
         };
@@ -101,3 +100,3 @@ return details;
             // modified to work, however for now I don't have the brain energy to work it all out).
-            const batchRemover = !this.usingTreeData ? new batchRemover_1.BatchRemover() : undefined;
+            const batchRemover = !details.usingTreeData ? new batchRemover_1.BatchRemover() : undefined;
             // the order here of [add, remove, update] needs to be the same as in ClientSideNodeManager,
@@ -139,10 +138,6 @@ // as the order is important when a record with the same id is added and removed in the same
     }
-    orderGroups(rootNode) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
-        const comparator = this.gridOptionsService.getCallback('initialGroupOrderComparator');
+    orderGroups(details) {
+        const comparator = details.initialGroupOrderComparator;
         if (core_1._.exists(comparator)) {
-            recursiveSort(rootNode);
+            recursiveSort(details.rootNode);
         }
@@ -163,3 +158,3 @@ function recursiveSort(rowNode) {
         // but when doing grid grouping, the node is not part of the path so we start with the parent.
-        let pointer = this.usingTreeData ? node : node.parent;
+        let pointer = details.usingTreeData ? node : node.parent;
         while (pointer && pointer !== details.rootNode) {
@@ -216,3 +211,3 @@ res.push({
         this.removeNodesFromParents(leafRowNodes, details, batchRemover);
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.postRemoveCreateFillerNodes(leafRowNodes, details);
@@ -353,3 +348,3 @@ // When not TreeData, then removeEmptyGroups is called just before the BatchRemover is flushed.
             rowNodes.forEach(rowNode => {
-                const isLeafNode = !this.usingTreeData && !rowNode.group;
+                const isLeafNode = !details.usingTreeData && !rowNode.group;
                 if (isLeafNode) {
@@ -363,3 +358,3 @@ return;
                 };
-                this.setGroupData(rowNode, groupInfo);
+                this.setGroupData(rowNode, groupInfo, details);
                 recurse(rowNode.childrenAfterGroup);
@@ -381,3 +376,3 @@ });
         // isn't allowed with treeData, so the grid never actually use .leafGroup when doing treeData.
-        rootNode.leafGroup = this.usingTreeData ? false : groupedCols.length === 0;
+        rootNode.leafGroup = details.usingTreeData ? false : groupedCols.length === 0;
         // we are doing everything from scratch, so reset childrenAfterGroup and childrenMapped from the rootNode
@@ -402,3 +397,3 @@ rootNode.childrenAfterGroup = [];
             // and the group cols have changed.
-            noFurtherProcessingNeeded = this.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
+            noFurtherProcessingNeeded = details.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
             // if the group display cols have changed, then we need to update rowNode.groupData
@@ -428,3 +423,3 @@ // (regardless of tree data or row grouping)
         }
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.swapGroupWithUserNode(parentGroup, childNode, isMove);
@@ -495,3 +490,3 @@ }
         groupNode.rowGroupColumn = groupInfo.rowGroupColumn;
-        this.setGroupData(groupNode, groupInfo);
+        this.setGroupData(groupNode, groupInfo, details);
         // we put 'row-group-' before the group id, so it doesn't clash with standard row id's. we also use 't-' and 'b-'
@@ -502,3 +497,3 @@ // for top pinned and bottom pinned rows.
         groupNode.level = level;
-        groupNode.leafGroup = this.usingTreeData ? false : level === (details.groupedColCount - 1);
+        groupNode.leafGroup = details.usingTreeData ? false : level === (details.groupedColCount - 1);
         groupNode.allLeafChildren = [];
@@ -508,3 +503,3 @@ // why is this done here? we are not updating the children count as we go,
         groupNode.setAllChildrenCount(0);
-        groupNode.rowGroupIndex = this.usingTreeData ? null : level;
+        groupNode.rowGroupIndex = details.usingTreeData ? null : level;
         groupNode.childrenAfterGroup = [];
@@ -515,8 +510,5 @@ groupNode.childrenMapped = {};
         this.setExpandedInitialValue(details, groupNode);
-        if (this.gridOptionsService.is('groupIncludeFooter')) {
-            groupNode.createFooter();
-        }
         return groupNode;
     }
-    setGroupData(groupNode, groupInfo) {
+    setGroupData(groupNode, groupInfo, details) {
         groupNode.groupData = {};
@@ -527,3 +519,3 @@ const groupDisplayCols = this.columnModel.getGroupDisplayColumns();
             // if rowGroupColumn is present, then it's grid row grouping and we only include if configuration says so
-            const displayGroupForCol = this.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
+            const displayGroupForCol = details.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
             if (displayGroupForCol) {
@@ -549,3 +541,3 @@ groupNode.groupData[col.getColId()] = groupInfo.key;
         // use callback if exists
-        const userCallback = this.gridOptionsService.getCallback('isGroupOpenByDefault');
+        const userCallback = details.isGroupOpenByDefault;
         if (userCallback) {
@@ -572,9 +564,9 @@ const params = {
     getGroupInfo(rowNode, details) {
-        if (this.usingTreeData) {
-            return this.getGroupInfoFromCallback(rowNode);
+        if (details.usingTreeData) {
+            return this.getGroupInfoFromCallback(rowNode, details);
         }
         return this.getGroupInfoFromGroupColumns(rowNode, details);
     }
-    getGroupInfoFromCallback(rowNode) {
-        const keys = this.getDataPath ? this.getDataPath(rowNode.data) : null;
+    getGroupInfoFromCallback(rowNode, details) {
+        const keys = details.getDataPath ? details.getDataPath(rowNode.data) : null;
         if (keys === null || keys === undefined || keys.length === 0) {
@@ -594,3 +586,3 @@ core_1._.doOnce(() => console.warn(`AG Grid: getDataPath() should not return an empty path for data`, rowNode.data), 'groupStage.getGroupInfoFromCallback');
             // this keeps the tree balanced and hence can be represented as a group.
-            const createGroupForEmpty = details.pivotMode || !this.gridOptionsService.is('groupAllowUnbalanced');
+            const createGroupForEmpty = details.pivotMode || !details.groupAllowUnbalanced;
             if (createGroupForEmpty && !keyExists) {
@@ -627,5 +619,2 @@ key = '';
 ], GroupStage.prototype, "selectionService", void 0);
-__decorate([
-    core_1.PostConstruct
-], GroupStage.prototype, "postConstruct", null);
 GroupStage = __decorate([
@@ -632,0 +621,0 @@ core_1.Bean('groupStage')

dist/cjs/es6/rowGrouping/pivotColDefService.js

@@ -14,7 +14,8 @@ "use strict";
     init() {
-        var _a, _b;
-        this.fieldSeparator = (_a = this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_';
-        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', (propChange) => this.fieldSeparator = propChange.currentValue);
-        this.pivotDefaultExpanded = (_b = this.gos.getNum('pivotDefaultExpanded')) !== null && _b !== void 0 ? _b : 0;
-        this.addManagedPropertyListener('pivotDefaultExpanded', (propChange) => this.pivotDefaultExpanded = propChange.currentValue);
+        const getFieldSeparator = () => { var _a; return (_a = this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_'; };
+        this.fieldSeparator = getFieldSeparator();
+        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', () => { this.fieldSeparator = getFieldSeparator(); });
+        const getPivotDefaultExpanded = () => { var _a; return (_a = this.gos.getNum('pivotDefaultExpanded')) !== null && _a !== void 0 ? _a : 0; };
+        this.pivotDefaultExpanded = getPivotDefaultExpanded();
+        this.addManagedPropertyListener('pivotDefaultExpanded', () => { this.pivotDefaultExpanded = getPivotDefaultExpanded(); });
     }
@@ -21,0 +22,0 @@ createPivotColumnDefs(uniqueValues) {

dist/cjs/es6/version.d.ts

@@ -1,1 +0,1 @@
-export declare const VERSION = "30.1.0";
+export declare const VERSION = "30.2.0";

dist/cjs/es6/version.js

@@ -5,2 +5,2 @@ "use strict";
 // DO NOT UPDATE MANUALLY: Generated from script during build time
-exports.VERSION = '30.1.0';
+exports.VERSION = '30.2.0';

dist/esm/es5/rowGrouping/aggregationStage.d.ts

@@ -6,8 +6,2 @@ import { BeanStub, IRowNodeStage, RowNode, Column, StageExecuteParams, IAggFunc } from "@ag-grid-community/core";
     private aggFuncService;
-    private gridApi;
-    private columnApi;
-    private filteredOnly;
-    private alwaysAggregateAtRootLevel;
-    private groupIncludeTotalFooter;
-    private init;
     execute(params: StageExecuteParams): any;
@@ -14,0 +8,0 @@ private createAggDetails;

dist/esm/es5/rowGrouping/aggregationStage.js

@@ -22,3 +22,3 @@ var __extends = (this && this.__extends) || (function () {
 };
-import { Bean, BeanStub, Autowired, _, PostConstruct, } from "@ag-grid-community/core";
+import { Bean, BeanStub, Autowired, _, } from "@ag-grid-community/core";
 var AggregationStage = /** @class */ (function (_super) {
@@ -29,9 +29,2 @@ __extends(AggregationStage, _super);
     }
-    AggregationStage.prototype.init = function () {
-        var _this = this;
-        this.alwaysAggregateAtRootLevel = this.gridOptionsService.is('alwaysAggregateAtRootLevel');
-        this.addManagedPropertyListener('alwaysAggregateAtRootLevel', function (propChange) { return _this.alwaysAggregateAtRootLevel = propChange.currentValue; });
-        this.groupIncludeTotalFooter = this.gridOptionsService.is('groupIncludeTotalFooter');
-        this.addManagedPropertyListener('groupIncludeTotalFooter', function (propChange) { return _this.groupIncludeTotalFooter = propChange.currentValue; });
-    };
     // it's possible to recompute the aggregate without doing the other parts
@@ -59,5 +52,9 @@ // + api.refreshClientSideRowModel('aggregate')
         var aggDetails = {
+            alwaysAggregateAtRootLevel: this.gridOptionsService.is('alwaysAggregateAtRootLevel'),
+            groupIncludeTotalFooter: this.gridOptionsService.is('groupIncludeTotalFooter'),
             changedPath: params.changedPath,
             valueColumns: measureColumns,
-            pivotColumns: pivotColumns
+            pivotColumns: pivotColumns,
+            filteredOnly: !this.isSuppressAggFilteredOnly(),
+            userAggFunc: this.gridOptionsService.getCallback('getGroupRowAgg'),
         };
@@ -72,4 +69,2 @@ return aggDetails;
         var _this = this;
-        // update prop, in case changed since last time
-        this.filteredOnly = !this.isSuppressAggFilteredOnly();
         var callback = function (rowNode) {
@@ -89,5 +84,5 @@ var hasNoChildren = !rowNode.hasChildren();
             // if total footer is displayed, the value is in use
-            if (isRootNode && !_this.groupIncludeTotalFooter) {
+            if (isRootNode && !aggDetails.groupIncludeTotalFooter) {
                 var notPivoting = !_this.columnModel.isPivotMode();
-                if (!_this.alwaysAggregateAtRootLevel && notPivoting) {
+                if (!aggDetails.alwaysAggregateAtRootLevel && notPivoting) {
                     return;
@@ -103,7 +98,5 @@ }
         var pivotColumnsMissing = aggDetails.pivotColumns.length === 0;
-        var userFunc = this.gridOptionsService.getCallback('getGroupRowAgg');
         var aggResult;
-        if (userFunc) {
-            var params = { nodes: rowNode.childrenAfterFilter };
-            aggResult = userFunc(params);
+        if (aggDetails.userAggFunc) {
+            aggResult = aggDetails.userAggFunc({ nodes: rowNode.childrenAfterFilter });
         }
@@ -174,3 +167,3 @@ else if (measureColumnsMissing) {
             : null;
-        var values2d = this.getValuesNormal(rowNode, changedValueColumns);
+        var values2d = this.getValuesNormal(rowNode, changedValueColumns, aggDetails.filteredOnly);
         var oldValues = rowNode.aggData;
@@ -209,3 +202,3 @@ changedValueColumns.forEach(function (valueColumn, index) {
     };
-    AggregationStage.prototype.getValuesNormal = function (rowNode, valueColumns) {
+    AggregationStage.prototype.getValuesNormal = function (rowNode, valueColumns, filteredOnly) {
         // create 2d array, of all values for all valueColumns
@@ -215,3 +208,3 @@ var values = [];
         var valueColumnCount = valueColumns.length;
-        var nodeList = this.filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
+        var nodeList = filteredOnly ? rowNode.childrenAfterFilter : rowNode.childrenAfterGroup;
         var rowCount = nodeList.length;
@@ -246,4 +239,4 @@ for (var i = 0; i < rowCount; i++) {
             data: rowNode ? rowNode.data : undefined,
-            api: this.gridApi,
-            columnApi: this.columnApi,
+            api: this.gridOptionsService.api,
+            columnApi: this.gridOptionsService.columnApi,
             context: this.gridOptionsService.context,
@@ -262,11 +255,2 @@ }; // the "as any" is needed to allow the deprecation warning messages
     ], AggregationStage.prototype, "aggFuncService", void 0);
-    __decorate([
-        Autowired('gridApi')
-    ], AggregationStage.prototype, "gridApi", void 0);
-    __decorate([
-        Autowired('columnApi')
-    ], AggregationStage.prototype, "columnApi", void 0);
-    __decorate([
-        PostConstruct
-    ], AggregationStage.prototype, "init", null);
     AggregationStage = __decorate([
@@ -273,0 +257,0 @@ Bean('aggregationStage')

dist/esm/es5/rowGrouping/columnDropZones/dropZoneColumnComp.js

@@ -127,3 +127,3 @@ var __extends = (this && this.__extends) || (function () {
             eElement: this.eDragHandle,
-            defaultIconName: DragAndDropService.ICON_HIDE,
+            getDefaultIconName: function () { return DragAndDropService.ICON_HIDE; },
             getDragItem: function () { return _this.createDragItem(); },
@@ -130,0 +130,0 @@ dragItemName: this.displayName,

dist/esm/es5/rowGrouping/filterAggregatesStage.js

@@ -108,3 +108,3 @@ var __extends = (this && this.__extends) || (function () {
         }
-        if (this.gridOptionsService.isTreeData()) {
+        if (this.gridOptionsService.is('treeData')) {
             this.setAllChildrenCountTreeData(rowNode);
@@ -111,0 +111,0 @@ }

dist/esm/es5/rowGrouping/groupStage.d.ts

@@ -8,8 +8,5 @@ import { BeanStub, IRowNodeStage, StageExecuteParams } from "@ag-grid-community/core";
     private selectionService;
-    private usingTreeData;
-    private getDataPath;
     private groupIdSequence;
     private oldGroupingDetails;
     private oldGroupDisplayColIds;
-    private postConstruct;
     execute(params: StageExecuteParams): void;
@@ -16,0 +13,0 @@ private positionLeafsAndGroups;

dist/esm/es5/rowGrouping/groupStage.js

@@ -43,3 +43,3 @@ var __extends = (this && this.__extends) || (function () {
 };
-import { _, Autowired, Bean, BeanStub, NumberSequence, PostConstruct, RowNode } from "@ag-grid-community/core";
+import { _, Autowired, Bean, BeanStub, NumberSequence, RowNode } from "@ag-grid-community/core";
 import { BatchRemover } from "./batchRemover";
@@ -56,8 +56,2 @@ var GroupStage = /** @class */ (function (_super) {
     }
-    GroupStage.prototype.postConstruct = function () {
-        this.usingTreeData = this.gridOptionsService.isTreeData();
-        if (this.usingTreeData) {
-            this.getDataPath = this.gridOptionsService.get('getDataPath');
-        }
-    };
     GroupStage.prototype.execute = function (params) {
@@ -72,11 +66,10 @@ var details = this.createGroupingDetails(params);
         }
-        this.positionLeafsAndGroups(params.changedPath);
-        this.orderGroups(details.rootNode);
+        if (!details.usingTreeData) {
+            // we don't do group sorting for tree data
+            this.positionLeafsAndGroups(params.changedPath);
+            this.orderGroups(details);
+        }
         this.selectableService.updateSelectableAfterGrouping(details.rootNode);
     };
     GroupStage.prototype.positionLeafsAndGroups = function (changedPath) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
         changedPath.forEachChangedNodeDepthFirst(function (group) {
@@ -110,7 +103,7 @@ if (group.childrenAfterGroup) {
         var rowNode = params.rowNode, changedPath = params.changedPath, rowNodeTransactions = params.rowNodeTransactions, rowNodeOrder = params.rowNodeOrder;
-        var groupedCols = this.usingTreeData ? null : this.columnModel.getRowGroupColumns();
+        var usingTreeData = this.gridOptionsService.is('treeData');
+        var groupedCols = usingTreeData ? null : this.columnModel.getRowGroupColumns();
         var details = {
             // someone complained that the parent attribute was causing some change detection
-            // to break is some angular add-on - which i never used. taking the parent out breaks
-            // a cyclic dependency, hence this flag got introduced.
+            // to break in an angular add-on.  Taking the parent out breaks a cyclic dependency, hence this flag got introduced.
             includeParents: !this.gridOptionsService.is('suppressParentsInRowNodes'),
@@ -121,7 +114,13 @@ expandByDefault: this.gridOptionsService.getNum('groupDefaultExpanded'),
             pivotMode: this.columnModel.isPivotMode(),
-            groupedColCount: this.usingTreeData || !groupedCols ? 0 : groupedCols.length,
+            groupedColCount: usingTreeData || !groupedCols ? 0 : groupedCols.length,
             rowNodeOrder: rowNodeOrder,
             transactions: rowNodeTransactions,
             // if no transaction, then it's shotgun, changed path would be 'not active' at this point anyway
-            changedPath: changedPath
+            changedPath: changedPath,
+            groupAllowUnbalanced: this.gridOptionsService.is('groupAllowUnbalanced'),
+            isGroupOpenByDefault: this.gridOptionsService.getCallback('isGroupOpenByDefault'),
+            initialGroupOrderComparator: this.gridOptionsService.getCallback('initialGroupOrderComparator'),
+            createGroupFooter: this.gridOptionsService.isGroupIncludeFooterTrueOrCallback(),
+            usingTreeData: usingTreeData,
+            getDataPath: usingTreeData ? this.gridOptionsService.get('getDataPath') : undefined
         };
@@ -138,3 +137,3 @@ return details;
             // modified to work, however for now I don't have the brain energy to work it all out).
-            var batchRemover = !_this.usingTreeData ? new BatchRemover() : undefined;
+            var batchRemover = !details.usingTreeData ? new BatchRemover() : undefined;
             // the order here of [add, remove, update] needs to be the same as in ClientSideNodeManager,
@@ -176,10 +175,6 @@ // as the order is important when a record with the same id is added and removed in the same
     };
-    GroupStage.prototype.orderGroups = function (rootNode) {
-        // we don't do group sorting for tree data
-        if (this.usingTreeData) {
-            return;
-        }
-        var comparator = this.gridOptionsService.getCallback('initialGroupOrderComparator');
+    GroupStage.prototype.orderGroups = function (details) {
+        var comparator = details.initialGroupOrderComparator;
         if (_.exists(comparator)) {
-            recursiveSort(rootNode);
+            recursiveSort(details.rootNode);
         }
@@ -200,3 +195,3 @@ function recursiveSort(rowNode) {
         // but when doing grid grouping, the node is not part of the path so we start with the parent.
-        var pointer = this.usingTreeData ? node : node.parent;
+        var pointer = details.usingTreeData ? node : node.parent;
         while (pointer && pointer !== details.rootNode) {
@@ -254,3 +249,3 @@ res.push({
         this.removeNodesFromParents(leafRowNodes, details, batchRemover);
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.postRemoveCreateFillerNodes(leafRowNodes, details);
@@ -398,3 +393,3 @@ // When not TreeData, then removeEmptyGroups is called just before the BatchRemover is flushed.
             rowNodes.forEach(function (rowNode) {
-                var isLeafNode = !_this.usingTreeData && !rowNode.group;
+                var isLeafNode = !details.usingTreeData && !rowNode.group;
                 if (isLeafNode) {
@@ -408,3 +403,3 @@ return;
                 };
-                _this.setGroupData(rowNode, groupInfo);
+                _this.setGroupData(rowNode, groupInfo, details);
                 recurse(rowNode.childrenAfterGroup);
@@ -426,3 +421,3 @@ });
         // isn't allowed with treeData, so the grid never actually use .leafGroup when doing treeData.
-        rootNode.leafGroup = this.usingTreeData ? false : groupedCols.length === 0;
+        rootNode.leafGroup = details.usingTreeData ? false : groupedCols.length === 0;
         // we are doing everything from scratch, so reset childrenAfterGroup and childrenMapped from the rootNode
@@ -447,3 +442,3 @@ rootNode.childrenAfterGroup = [];
             // and the group cols have changed.
-            noFurtherProcessingNeeded = this.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
+            noFurtherProcessingNeeded = details.usingTreeData || this.areGroupColsEqual(details, this.oldGroupingDetails);
             // if the group display cols have changed, then we need to update rowNode.groupData
@@ -474,3 +469,3 @@ // (regardless of tree data or row grouping)
         }
-        if (this.usingTreeData) {
+        if (details.usingTreeData) {
             this.swapGroupWithUserNode(parentGroup, childNode, isMove);
@@ -542,3 +537,3 @@ }
         groupNode.rowGroupColumn = groupInfo.rowGroupColumn;
-        this.setGroupData(groupNode, groupInfo);
+        this.setGroupData(groupNode, groupInfo, details);
         // we put 'row-group-' before the group id, so it doesn't clash with standard row id's. we also use 't-' and 'b-'
@@ -549,3 +544,3 @@ // for top pinned and bottom pinned rows.
         groupNode.level = level;
-        groupNode.leafGroup = this.usingTreeData ? false : level === (details.groupedColCount - 1);
+        groupNode.leafGroup = details.usingTreeData ? false : level === (details.groupedColCount - 1);
         groupNode.allLeafChildren = [];
@@ -555,3 +550,3 @@ // why is this done here? we are not updating the children count as we go,
         groupNode.setAllChildrenCount(0);
-        groupNode.rowGroupIndex = this.usingTreeData ? null : level;
+        groupNode.rowGroupIndex = details.usingTreeData ? null : level;
         groupNode.childrenAfterGroup = [];
@@ -562,9 +557,5 @@ groupNode.childrenMapped = {};
         this.setExpandedInitialValue(details, groupNode);
-        if (this.gridOptionsService.is('groupIncludeFooter')) {
-            groupNode.createFooter();
-        }
         return groupNode;
     };
-    GroupStage.prototype.setGroupData = function (groupNode, groupInfo) {
-        var _this = this;
+    GroupStage.prototype.setGroupData = function (groupNode, groupInfo, details) {
         groupNode.groupData = {};
@@ -575,3 +566,3 @@ var groupDisplayCols = this.columnModel.getGroupDisplayColumns();
             // if rowGroupColumn is present, then it's grid row grouping and we only include if configuration says so
-            var displayGroupForCol = _this.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
+            var displayGroupForCol = details.usingTreeData || (groupNode.rowGroupColumn ? col.isRowGroupDisplayed(groupNode.rowGroupColumn.getId()) : false);
             if (displayGroupForCol) {
@@ -597,3 +588,3 @@ groupNode.groupData[col.getColId()] = groupInfo.key;
         // use callback if exists
-        var userCallback = this.gridOptionsService.getCallback('isGroupOpenByDefault');
+        var userCallback = details.isGroupOpenByDefault;
         if (userCallback) {
@@ -620,9 +611,9 @@ var params = {
     GroupStage.prototype.getGroupInfo = function (rowNode, details) {
-        if (this.usingTreeData) {
-            return this.getGroupInfoFromCallback(rowNode);
+        if (details.usingTreeData) {
+            return this.getGroupInfoFromCallback(rowNode, details);
         }
         return this.getGroupInfoFromGroupColumns(rowNode, details);
     };
-    GroupStage.prototype.getGroupInfoFromCallback = function (rowNode) {
-        var keys = this.getDataPath ? this.getDataPath(rowNode.data) : null;
+    GroupStage.prototype.getGroupInfoFromCallback = function (rowNode, details) {
+        var keys = details.getDataPath ? details.getDataPath(rowNode.data) : null;
         if (keys === null || keys === undefined || keys.length === 0) {
@@ -643,3 +634,3 @@ _.doOnce(function () { return console.warn("AG Grid: getDataPath() should not return an empty path for data", rowNode.data); }, 'groupStage.getGroupInfoFromCallback');
             // this keeps the tree balanced and hence can be represented as a group.
-            var createGroupForEmpty = details.pivotMode || !_this.gridOptionsService.is('groupAllowUnbalanced');
+            var createGroupForEmpty = details.pivotMode || !details.groupAllowUnbalanced;
             if (createGroupForEmpty && !keyExists) {
@@ -675,5 +666,2 @@ key = '';
     ], GroupStage.prototype, "selectionService", void 0);
-    __decorate([
-        PostConstruct
-    ], GroupStage.prototype, "postConstruct", null);
     GroupStage = __decorate([
@@ -680,0 +668,0 @@ Bean('groupStage')

dist/esm/es5/rowGrouping/pivotColDefService.js

@@ -63,7 +63,8 @@ var __extends = (this && this.__extends) || (function () {
         var _this = this;
-        var _a, _b;
-        this.fieldSeparator = (_a = this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_';
-        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', function (propChange) { return _this.fieldSeparator = propChange.currentValue; });
-        this.pivotDefaultExpanded = (_b = this.gos.getNum('pivotDefaultExpanded')) !== null && _b !== void 0 ? _b : 0;
-        this.addManagedPropertyListener('pivotDefaultExpanded', function (propChange) { return _this.pivotDefaultExpanded = propChange.currentValue; });
+        var getFieldSeparator = function () { var _a; return (_a = _this.gos.get('serverSidePivotResultFieldSeparator')) !== null && _a !== void 0 ? _a : '_'; };
+        this.fieldSeparator = getFieldSeparator();
+        this.addManagedPropertyListener('serverSidePivotResultFieldSeparator', function () { _this.fieldSeparator = getFieldSeparator(); });
+        var getPivotDefaultExpanded = function () { var _a; return (_a = _this.gos.getNum('pivotDefaultExpanded')) !== null && _a !== void 0 ? _a : 0; };
+        this.pivotDefaultExpanded = getPivotDefaultExpanded();
+        this.addManagedPropertyListener('pivotDefaultExpanded', function () { _this.pivotDefaultExpanded = getPivotDefaultExpanded(); });
     };
@@ -70,0 +71,0 @@ PivotColDefService.prototype.createPivotColumnDefs = function (uniqueValues) {

dist/esm/es5/version.d.ts

@@ -1,1 +0,1 @@
-export declare const VERSION = "30.1.0";
+export declare const VERSION = "30.2.0";

dist/esm/es5/version.js

 // DO NOT UPDATE MANUALLY: Generated from script during build time
-export var VERSION = '30.1.0';
+export var VERSION = '30.2.0';

dist/esm/es6/rowGrouping/aggregationStage.d.ts

@@ -6,8 +6,2 @@ import { BeanStub, IRowNodeStage, RowNode, Column, StageExecuteParams, IAggFunc } from "@ag-grid-community/core";
     private aggFuncService;
-    private gridApi;
-    private columnApi;
-    private filteredOnly;
-    private alwaysAggregateAtRootLevel;
-    private groupIncludeTotalFooter;
-    private init;
     execute(params: StageExecuteParams): any;
@@ -14,0 +8,0 @@ private createAggDetails;

dist/esm/es6/rowGrouping/groupStage.d.ts

@@ -8,8 +8,5 @@ import { BeanStub, IRowNodeStage, StageExecuteParams } from "@ag-grid-community/core";
     private selectionService;
-    private usingTreeData;
-    private getDataPath;
     private groupIdSequence;
     private oldGroupingDetails;
     private oldGroupDisplayColIds;
-    private postConstruct;
     execute(params: StageExecuteParams): void;
@@ -16,0 +13,0 @@ private positionLeafsAndGroups;

dist/esm/es6/version.d.ts

@@ -1,1 +0,1 @@
-export declare const VERSION = "30.1.0";
+export declare const VERSION = "30.2.0";

package.json

 {
   "name": "@ag-grid-enterprise/row-grouping",
-  "version": "30.1.0",
+  "version": "30.2.0",
   "description": "Advanced Data Grid / Data Table supporting Javascript / Typescript / React / Angular / Vue",
@@ -50,4 +50,4 @@ "main": "./dist/esm/es6/main.mjs",
   "dependencies": {
-    "@ag-grid-community/core": "~30.1.0",
-    "@ag-grid-enterprise/core": "~30.1.0"
+    "@ag-grid-community/core": "~30.2.0",
+    "@ag-grid-enterprise/core": "~30.2.0"
   },
@@ -54,0 +54,0 @@ "devDependencies": {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

4302302

increased by1.17%

Lines of code

72559

increased by0.96%

Dependency changes

• + Added@ag-grid-community/core@30.2.1(transitive)

• + Added@ag-grid-enterprise/core@30.2.1(transitive)

• - Removed@ag-grid-community/core@30.1.0(transitive)

• - Removed@ag-grid-enterprise/core@30.1.0(transitive)

• Updated@ag-grid-community/core@~30.2.0

• Updated@ag-grid-enterprise/core@~30.2.0