Skip to content
Browse files

Infer currency negative format from positive one.

When a locale file sets the format of the positive
currency value as '%n %u', the default negative
currency format should be '-%n %u'.
  • Loading branch information...
1 parent 78372b6 commit 6724c8c8879dc41a4cdffc507353d8a114ca8028 @kuroda kuroda committed Jan 5, 2012
View
1 actionpack/lib/action_view/helpers/number_helper.rb
@@ -114,6 +114,7 @@ def number_to_currency(number, options = {})
defaults = I18n.translate(:'number.format', :locale => options[:locale], :default => {})
currency = I18n.translate(:'number.currency.format', :locale => options[:locale], :default => {})
+ currency[:negative_format] ||= "-" + currency[:format] if currency[:format]
defaults = DEFAULT_CURRENCY_VALUES.merge(defaults).merge!(currency)
defaults[:negative_format] = "-" + options[:format] if options[:format]
View
7 actionpack/test/template/number_helper_i18n_test.rb
@@ -53,6 +53,13 @@ def test_number_to_currency_with_clean_i18n_settings
assert_equal("-$10.00", number_to_currency(-10))
end
end
+
+ def test_number_to_currency_without_currency_negative_format
+ clean_i18n do
+ I18n.backend.store_translations 'ts', :number => { :currency => { :format => { :unit => '@', :format => '%n %u' } } }
+ assert_equal("-10.00 @", number_to_currency(-10, :locale => 'ts'))
+ end
+ end
def test_number_with_i18n_precision
#Delimiter was set to ""

0 comments on commit 6724c8c

Please sign in to comment.
Something went wrong with that request. Please try again.