mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Ensure number helpers can handle HTML safe strings - closes #1597.
This commit is contained in:
parent
90c4885766
commit
0bcbcdfdad
2 changed files with 8 additions and 1 deletions
|
@ -211,7 +211,7 @@ module ActionView
|
|||
defaults = I18n.translate(:'number.format', :locale => options[:locale], :default => {})
|
||||
options = options.reverse_merge(defaults)
|
||||
|
||||
parts = number.to_s.split('.')
|
||||
parts = number.to_s.to_str.split('.')
|
||||
parts[0].gsub!(/(\d)(?=(\d\d\d)+(?!\d))/, "\\1#{options[:delimiter]}")
|
||||
parts.join(options[:separator]).html_safe
|
||||
|
||||
|
|
|
@ -283,33 +283,40 @@ class NumberHelperTest < ActionView::TestCase
|
|||
assert number_to_human(1).html_safe?
|
||||
assert !number_to_human("<script></script>").html_safe?
|
||||
assert number_to_human("asdf".html_safe).html_safe?
|
||||
assert number_to_human("1".html_safe).html_safe?
|
||||
|
||||
assert number_to_human_size(1).html_safe?
|
||||
assert number_to_human_size(1000000).html_safe?
|
||||
assert !number_to_human_size("<script></script>").html_safe?
|
||||
assert number_to_human_size("asdf".html_safe).html_safe?
|
||||
assert number_to_human_size("1".html_safe).html_safe?
|
||||
|
||||
assert number_with_precision(1, :strip_insignificant_zeros => false).html_safe?
|
||||
assert number_with_precision(1, :strip_insignificant_zeros => true).html_safe?
|
||||
assert !number_with_precision("<script></script>").html_safe?
|
||||
assert number_with_precision("asdf".html_safe).html_safe?
|
||||
assert number_with_precision("1".html_safe).html_safe?
|
||||
|
||||
assert number_to_currency(1).html_safe?
|
||||
assert !number_to_currency("<script></script>").html_safe?
|
||||
assert number_to_currency("asdf".html_safe).html_safe?
|
||||
assert number_to_currency("1".html_safe).html_safe?
|
||||
|
||||
assert number_to_percentage(1).html_safe?
|
||||
assert !number_to_percentage("<script></script>").html_safe?
|
||||
assert number_to_percentage("asdf".html_safe).html_safe?
|
||||
assert number_to_percentage("1".html_safe).html_safe?
|
||||
|
||||
assert number_to_phone(1).html_safe?
|
||||
assert_equal "<script></script>", number_to_phone("<script></script>")
|
||||
assert number_to_phone("<script></script>").html_safe?
|
||||
assert number_to_phone("asdf".html_safe).html_safe?
|
||||
assert number_to_phone("1".html_safe).html_safe?
|
||||
|
||||
assert number_with_delimiter(1).html_safe?
|
||||
assert !number_with_delimiter("<script></script>").html_safe?
|
||||
assert number_with_delimiter("asdf".html_safe).html_safe?
|
||||
assert number_with_delimiter("1".html_safe).html_safe?
|
||||
end
|
||||
|
||||
def test_number_helpers_should_raise_error_if_invalid_when_specified
|
||||
|
|
Loading…
Reference in a new issue