Skip to content

Commit bdd2868

Browse files
committed
SVY-20931 AG Grid row grouping fails when valuelist columns have duplicate display values. Grid groups by display name instead of real value, causing node expansion failures and rendering errors even when real values are unique.
1 parent 7865cd8 commit bdd2868

1 file changed

Lines changed: 16 additions & 6 deletions

File tree

‎aggrid/groupingtable/groupingtable.js‎

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3270,10 +3270,14 @@ angular.module('aggridGroupingtable', ['webSocketModule', 'servoy']).directive('
32703270
if (groupKeys[i] == NULL_VALUE) {
32713271
groupKeys[i] = null; // reset to real null, so we use the right value for grouping
32723272
}
3273-
else {
3274-
var vl = getValuelistEx(params.parentNode.data, rowGroupCols[i]['id'], false);
3275-
if(vl) {
3276-
filterPromisesFction(vl, i);
3273+
if (groupKeys[i] !== null) {
3274+
if(params?.parentNode?.data['_unresolved_' + rowGroupCols[i]['field']] !== undefined) {
3275+
groupKeys[i] = params?.parentNode?.data['_unresolved_' + rowGroupCols[i]['field']];
3276+
} else {
3277+
var vl = getValuelistEx(params.parentNode.data, rowGroupCols[i]['id'], false);
3278+
if(vl) {
3279+
filterPromisesFction(vl, i);
3280+
}
32773281
}
32783282
}
32793283
}
@@ -4043,8 +4047,14 @@ angular.module('aggridGroupingtable', ['webSocketModule', 'servoy']).directive('
40434047
var header = columns[i];
40444048
var field = header.id == 'svycount' ? header.id : getColumnID(header, i);
40454049

4046-
var value = header.dataprovider ? header.dataprovider[index] : null;
4047-
r[field] = value;
4050+
if(header.dataprovider) {
4051+
r[field] = header.dataprovider[index];
4052+
if(header.dataproviderUnresolved && (header.dataprovider[index] !== header.dataproviderUnresolved[index])) {
4053+
r["_unresolved_" + field] = header.dataproviderUnresolved[index];
4054+
}
4055+
} else {
4056+
r[field] = null;
4057+
}
40484058
}
40494059

40504060
return r;

0 commit comments

Comments
 (0)