- Issue created by @acbramley
- Merge request !11761Issue #3517853: Regression count aggregation with numeric fields → (Open) created by acbramley
- 🇦🇺Australia acbramley
It seems like string based fields work fine (tested the output of COUNT on the name and that works without the fixed) so perhaps it's a numeric field issue?
- 🇦🇺Australia acbramley
That breaks testFieldAggregationSettings again, gonna have to dig further into this
- 🇦🇺Australia acbramley
Test is correctly failing now:
├ Failed asserting that two strings are equal. ┊ ---·Expected ┊ +++·Actual ┊ @@ @@ ┊ -'4' ┊ +'Thu,·1·Jan·1970·-·10:00\n
I'm debugging how views renders this stuff 🤯
When we get to StylePluginBase::elementPreRenderRow we have the correct count value in
$data['#row']
. What's happening is that value of4
is being passed to the timestamp formatter which results in the Jan 1 1970 date (unix epoch)Before 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work , the field handler was being overridden to numeric, therefore $field in that preRender was the NumericField plugin, not EntityField.
- 🇦🇺Australia acbramley
Confirmed this isn't just an issue with COUNT either, SUM etc have the same behaviour. I'm starting to think we do need to revert here and come up with a proper fix. The field handlers should be overridden because we don't want to go through the EntityField plugin when outputting aggregation data.
- 🇦🇺Australia acbramley
I also confirmed this issue with other field types such as Link