Zero values with decimals not being treated as zero in views_handler_field_numeric

That wont' work. The value 'foo' will show up as empty because 'foo' == 0. That's why we use !==

Instead youll have to use is_numeric I believe. If it's numeric you can test != and if it's not numeric you have to test !==

I have the same problem for Drupal 6. What is the best way to submit a patch? Should I create another issue?

This no longer applies, but the problem still exists.

See #2450401: Multiple issues with is_value_empty().

Does this patch work even though failed testing? Need this solution. Thanks

How about this? We just ensure we check ($value !== 0 && $value !== '0')

Whoops that doesn't cover the float from the original issue just the integers.

Ok this should work better.

$is_zero = is_numeric($value) && !((float) $value);

Covers as 0:
0, 0.00, 0.000, '0', '0.00', '0.000'

Will ensure these don't count as 0:
'', '0.000001', 0.000001

And in the case where 'empty_zero' == TRUE, it will count '0.000' as a 0 value, and let it pass to set it to '' as I'd expect

This should really get some tests I suspect.

While testing I still got it a bit off, it was assuming NULL was allowed not be empty the way the conditional was written.

The 2 year old patch in #16 to views_handler_field_numeric.inc applied cleanly to the latest views 7.x-3.x-dev, but still needs review.

This is a near complete duplicate of #1782304: Can't use "rewrite if empty" on fields using views_handler_field_numeric, only that one has test coverage too.

Could everyone here please try out the patch over there to confirm it works for you? Thanks.