kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

number_to_percentage and precision: 0 work with NAN and INFINITY.

Browse source 
Closes #19227.
This commit is contained in:
Yves Senn 2015-03-06 09:04:51 +01:00
parent 1c1bced7b1
commit 081a3963ea
4 changed files with 16 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
actionview/test/template/number_helper_test.rb
View file

@ -35,6 +35,10 @@ class NumberHelperTest < ActionView::TestCase
assert_equal "98a%", number_to_percentage("98a")
assert_equal "NaN%", number_to_percentage(Float::NAN)
assert_equal "Inf%", number_to_percentage(Float::INFINITY)
assert_equal "NaN%", number_to_percentage(Float::NAN, precision: 0)
assert_equal "Inf%", number_to_percentage(Float::INFINITY, precision: 0)
assert_equal "NaN%", number_to_percentage(Float::NAN, precision: 1)
assert_equal "Inf%", number_to_percentage(Float::INFINITY, precision: 1)
end
def test_number_with_delimiter

7
activesupport/CHANGELOG.md
View file

@ -1,3 +1,10 @@
* `number_to_percentage` does not crash with `Float::NAN` or `Float::INFINITY`
as input when `precision: 0` is used.
Fixes #19227.
*Yves Senn*
* Take DST into account when locating TimeZone from Numeric.
When given a specific offset, use the first result found where the

2
activesupport/lib/active_support/number_helper/number_to_rounded_converter.rb
View file

@ -23,7 +23,7 @@ module ActiveSupport
precision = 0 if precision < 0 # don't let it be negative
else
rounded_number = number.round(precision)
rounded_number = rounded_number.to_i if precision == 0
rounded_number = rounded_number.to_i if precision == 0 && rounded_number.finite?
rounded_number = rounded_number.abs if rounded_number.zero? # prevent showing negative zeros
end

4
activesupport/test/number_helper_test.rb
View file

@ -83,6 +83,10 @@ module ActiveSupport
assert_equal("98a%", number_helper.number_to_percentage("98a"))
assert_equal("NaN%", number_helper.number_to_percentage(Float::NAN))
assert_equal("Inf%", number_helper.number_to_percentage(Float::INFINITY))
assert_equal("NaN%", number_helper.number_to_percentage(Float::NAN, precision: 0))
assert_equal("Inf%", number_helper.number_to_percentage(Float::INFINITY, precision: 0))
assert_equal("NaN%", number_helper.number_to_percentage(Float::NAN, precision: 1))
assert_equal("Inf%", number_helper.number_to_percentage(Float::INFINITY, precision: 1))
assert_equal("1000%", number_helper.number_to_percentage(1000, precision: nil))
assert_equal("1000%", number_helper.number_to_percentage(1000, precision: nil))
assert_equal("1000.1%", number_helper.number_to_percentage(1000.1, precision: nil))