Permalink
Browse files

Handle raise flag in translate when both main and default translation…

… is missing. Fixes #19967
  • Loading branch information...
imanel committed May 4, 2015
1 parent 7166937 commit 9c8542bb12fac316253addf24e4fa2af34d33d7d
View
@@ -1,3 +1,10 @@
* `translate` should handle `raise` flag correctly in case of both main and default
translation is missing.
Fixes #19967
*Bernard Potocki*
* Load the `default_form_builder` from the controller on initialization, which overrides
the global config if it is present.
@@ -62,10 +62,10 @@ def translate(key, options = {})
# Note: `raise_error` refers to us re-raising the error in this method. I18n is forced to raise by default.
if options[:raise] == false || (options.key?(:rescue_format) && options[:rescue_format].nil?)
raise_error = false
options[:raise] = false
i18n_raise = false
else
raise_error = options[:raise] || options[:rescue_format] || ActionView::Base.raise_on_missing_translations
options[:raise] = true
i18n_raise = true
end
if html_safe_translation_key?(key)
@@ -75,11 +75,11 @@ def translate(key, options = {})
html_safe_options[name] = ERB::Util.html_escape(value.to_s)
end
end
translation = I18n.translate(scope_key_by_partial(key), html_safe_options)
translation = I18n.translate(scope_key_by_partial(key), html_safe_options.merge(raise: i18n_raise))
translation.respond_to?(:html_safe) ? translation.html_safe : translation
else
I18n.translate(scope_key_by_partial(key), options)
I18n.translate(scope_key_by_partial(key), options.merge(raise: i18n_raise))
end
rescue I18n::MissingTranslationData => e
if remaining_defaults.present?
@@ -157,6 +157,19 @@ def test_translate_with_default_named_html
assert_equal true, translation.html_safe?
end
def test_translate_with_missing_default
translation = translate(:'translations.missing', :default => :'translations.missing_html')
expected = '<span class="translation_missing" title="translation missing: en.translations.missing_html">Missing Html</span>'
assert_equal expected, translation
assert_equal true, translation.html_safe?
end
def test_translate_with_missing_default_and_raise_option
assert_raise(I18n::MissingTranslationData) do
translate(:'translations.missing', :default => :'translations.missing_html', :raise => true)
end
end
def test_translate_with_two_defaults_named_html
translation = translate(:'translations.missing', :default => [:'translations.missing_html', :'translations.hello_html'])
assert_equal '<a>Hello World</a>', translation

0 comments on commit 9c8542b

Please sign in to comment.