orb icon indicating copy to clipboard operation
orb copied to clipboard

Published 20 hours ago • verified publisher icon nnajm

Prevent non-number in aggregate function

Open ValentinH opened this issue 9 years ago • 2 comments

I think aggregation on string should not be possible because it leads to weird/unpredictable results: capture d ecran 2015-03-25 a 20 33 01

I have started to work today on a patch that return a blank value when the input data is not a number (using typeof(val) === 'number' because strings like '012' are also unpredictable)

What do you think? Should I submit a PR?

ValentinH avatar Mar 25 '15 19:03 ValentinH

@ValentinH Any luck with this patch? It would definitely be useful.

ztrayner avatar Aug 11 '15 22:08 ztrayner

Created a patch that I will PR. Creates a new "none" aggregateFunc option to allow using text fields.

wilcraig avatar Oct 12 '15 16:10 wilcraig