Using this.fields instead of this.completeFields in getGridSummaryData

[senordhuff]

Hello,

I notice that your logic uses this.completeFields by default in the getGridSummaryData function. Since we have hundreds of fields in our main grid, we'd prefer to use this.fields which represents only the visible fields. After testing a patch, it seems to work fine and will recalculate whenever any hidden fields are exposed. Is there any danger in us defaulting to this.fields to avoid the extra unneeded calculations of non-visible fields?

[Isomorphic]

This will probably be fine, but has not been extensively tested.
There may potentially be some edge cases - for example fields that pick up their values from another, hidden field [formula fields, perhaps or fields using the displayField attribute]. Obviously you should test the use cases you care about within your application before "going live" with such a change in place.

Regards
Isomorphic Software

[senordhuff]

Thanks, I've tested it further with user formulas and it seems good so far to use this.fields instead of this.completeFields. We'll certainly let you know if we have any problems. I also noticed that getGroupSummaryData uses this.fields exclusively and not this.completeFields. Are you able to tell me why there is a discrepancy there between getGridSummaryData vs getGroupSummaryData? It seems like they should both refer to the same set of fields at all times.