[5.1] Fixed aggregate queries

[GrahamCampbell]

No description provided.

[JoostK]

This may now become int|float

[GrahamCampbell]

Ah, yeh, oops, Meant to put that :)

[taylorotwell]

Can you describe whats going on here?

[JoostK]

I like that you try to do something with my earlier comments in #14781 but really doubt if we should go this way. Has there been any practical issues with aggregate how it always behaved?

[GrahamCampbell]

Yeh, it'll break if you have strict types turned on and you were expecting a float instead of a string without these changes.

[JoostK]

Ah, in a PHP 7 context that makes sense. With formattedAggregate having been changed to numericalAggregate this is then okay with me.

[segadora]

I might have posted a comment in the wrong place.

I have issues since this change with the min and max method. I am trying to get the maximum value if a date time field like 2016-08-24 00:00:00, but the max() method only returns 2016. Is this the expected behavior and if so, how to get the min and max values of a date time field?

[GrahamCampbell]

Basically reverted this.