aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorXavier Noria <fxn@hashref.com>2011-11-17 23:59:19 +0100
committerXavier Noria <fxn@hashref.com>2011-11-17 23:59:19 +0100
commit1079724fe643fe63e6d58a37274c2cf0ff172a8b (patch)
tree9dd3c4bc6207a2fa9a7f3288c84319a0bff34ca3
parenta437986f433a775c7c370ad3f0273938015699df (diff)
downloadrails-1079724fe643fe63e6d58a37274c2cf0ff172a8b.tar.gz
rails-1079724fe643fe63e6d58a37274c2cf0ff172a8b.tar.bz2
rails-1079724fe643fe63e6d58a37274c2cf0ff172a8b.zip
Revert "warn the user values are directly interpolated into _html translation strings"
Reason: After another round of discussion, it has been decided to let interpolation deal with unsafe strings as it should do. This reverts commit d57d8098fc269a26ea0051a9027a33af1a9a4b2b.
-rw-r--r--actionpack/lib/action_view/helpers/translation_helper.rb2
-rw-r--r--railties/guides/source/i18n.textile12
2 files changed, 0 insertions, 14 deletions
diff --git a/actionpack/lib/action_view/helpers/translation_helper.rb b/actionpack/lib/action_view/helpers/translation_helper.rb
index 0e6c3c5724..be64dc823e 100644
--- a/actionpack/lib/action_view/helpers/translation_helper.rb
+++ b/actionpack/lib/action_view/helpers/translation_helper.rb
@@ -43,8 +43,6 @@ module ActionView
# a safe HTML string that won't be escaped by other HTML helper methods. This
# naming convention helps to identify translations that include HTML tags so that
# you know what kind of output to expect when you call translate in a template.
- # Note however that rule extends to interpolated values, so you are responsible
- # for passing them already escaped in the call, if they need to be.
def translate(key, options = {})
options.merge!(:rescue_format => :html) unless options.key?(:rescue_format)
translation = I18n.translate(scope_key_by_partial(key), options)
diff --git a/railties/guides/source/i18n.textile b/railties/guides/source/i18n.textile
index 43afa6c9e2..2d4cc13571 100644
--- a/railties/guides/source/i18n.textile
+++ b/railties/guides/source/i18n.textile
@@ -634,18 +634,6 @@ en:
!images/i18n/demo_html_safe.png(i18n demo html safe)!
-Please note that values are interpolated directly into the translation.
-If they need to be escaped you need to pass them already escaped in the +t+ call.
-
-<erb>
-# config/locales/en.yml
-en:
- welcome_html: <b>Welcome %{name}!</b>
-
-<%# Note the call to h() to avoid injection %>
-<%= t('welcome_html', :name => h(user.name)) %>
-</erb>
-
h3. How to Store your Custom Translations
The Simple backend shipped with Active Support allows you to store translations in both plain Ruby and YAML format. [2]