Upgrading from 10.4.5 to 10.4.6 removes views format plural on Content ID field

This patch just adds back the 'field' => 'numeric', to the "'count_distinct' => [" array solving the issue I'm facing.

The other keys are kept as they are, but potentially, they need to get back the "'field' => 'numeric'" as well.

The commit that removed the field numeric is this one: Issue #2735997 by ramil g, joelpittet, ranjith_kumar_k_u, asad_ahmed,... and belongs to the following issue: Decimal separator and decimals settings ignored when aggregating decimal fields 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work

It looks like 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work will be reverted so perhaps this issue should be paused.

Hi Cilefen,

Now I see they say "Revert MR up" on comment #76. Let's see if a new version is released soon.

Thanks

Thanks for the initial patch!

I applied the patch as shared, but unfortunately, it didn’t resolve the issue on my end. After further investigation, I noticed that adding 'field' => 'numeric' not only to the 'count_distinct' method but also to the other aggregate methods helped fix the issue for me.

I've updated the patch accordingly and attached it here for reference.

I can confirm that downgrading to Drupal Core 10.4.5 fixes the issue.
This breaks commerce_reports, for example.

Does the revert in 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work correct this? If so, maybe we can close this 🐛 [regression] COUNT aggregation no longer outputs count value for some fields Active ?

Thanks @cilefen - I just downgraded and have very busy days currently, would be great if someone else could test it.

But I guess YES and thought the same...

Marking duplicate of the original issue, which is now reverted.

Hi,
#5 works in 10.4.6, thanks a lot.

I found this post after getting this error on commerce reports like @anybody said.

I don't see how the 2735997 is a duplicate since it's in D11 and not for 10.4.*. I think we should keep it open till a patch is in 10.4.*.

@adsyy I think you are incorrect. 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work , in addition to being released and reverted in 11.x, was released in 10.4.6 → and reverted in 10.4.x 🐛 Decimal separator and decimals settings ignored when aggregating decimal fields Needs work . The revert hasn't yet been released. If I am wrong, please update with more information.

@cilefen you're right, just not released yet, my bad.