Grouppriority default treeAggregationFn overwrite bug

[MartijnWelker]

According to the docs the groupingColumnBuilder function should get one of the default aggregations when the groupPriority is undefined or less then 0, however, in the code it does otherwise:

If the groupPriority is undefined or less than 0, then we expect to be aggregating, and we look at the aggregation types to determine what sort of aggregation we can do. Values are in the constants file, but include SUM, COUNT, MAX, MIN

if ( typeof(col.grouping.groupPriority) !== 'undefined' && col.grouping.groupPriority > -1 ){
    col.treeAggregationFn = uiGridTreeBaseService.nativeAggregations()[uiGridGroupingConstants.aggregation.COUNT].aggregationFn;
    col.treeAggregationFinalizerFn = service.groupedFinalizerFn;
}

As you see in the above code, it actually checks for !== 'undefined' && > -1, which overwrites a possible custom set treeAggregationFn with a default one.

This above code is taken from the current release, line #18959

[troywoy]

@MartijnWelker, have you made any progress with this? I believe I'm running into a similar issue, in that when a columnDef has groupPriority defined, the customTreeAggregationFn is ignored.

In my scenario, I need two values in my customTreeAggregationFinalizerFn; the aggregation count from the front and an aggregation count from the database (which may exceed what was returned to the front end).

For example, Fruits (500/12389) would mean I'm display 500 of a possible 12389 Fruits. customTreeAggregationFinalizerFn isn't provided a row pointer so I have no access (unless I want to write some lame code) to get row.totalFruits. I was going to use customTreeAggregationFn to stick totalFruits onto the aggregation pointer.

[troywoy]

Here's my "hack" just in case:

customTreeAggregationFinalizerFn: function(aggregation) {
    // Get any row that has this group value
    var row = _.find($scope.gridOptions.data, { fruit: aggregation.groupVal });

    // Now update aggregation to also display database aggreation of rows in this group
    aggregation.rendered = aggregation.rendered.slice(0, -1) + '/' + row.totalFruis + ')';
}