From 262b2ea8cda20999ddf8c4bf13b7a70453e996d2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Valim?= <jose.valim@gmail.com>
Date: Wed, 19 Jan 2011 23:42:10 +0100
Subject: [PATCH] Solve SystemStackError when changing locale inside
 ActionMailer [#5329 state:resolved]

---
 actionmailer/test/i18n_with_controller_test.rb  | 16 ++++++----------
 actionpack/lib/abstract_controller/rendering.rb |  9 +++++----
 actionpack/lib/action_view/lookup_context.rb    |  4 ++--
 actionpack/test/template/lookup_context_test.rb |  2 +-
 4 files changed, 14 insertions(+), 17 deletions(-)

diff --git a/actionmailer/test/i18n_with_controller_test.rb b/actionmailer/test/i18n_with_controller_test.rb
index 759c007c45c..7040ae6f8d1 100644
--- a/actionmailer/test/i18n_with_controller_test.rb
+++ b/actionmailer/test/i18n_with_controller_test.rb
@@ -2,13 +2,12 @@ require 'abstract_unit'
 require 'action_controller'
 
 class I18nTestMailer < ActionMailer::Base
-
   configure do |c|
-    c.assets_dir = '' # To get the tests to pass
+    c.assets_dir = ''
   end
 
   def mail_with_i18n_subject(recipient)
-    @recipient   = recipient
+    @recipient  = recipient
     I18n.locale = :de
     mail(:to => recipient, :subject => "#{I18n.t :email_subject} #{recipient}",
       :from => "system@loudthinking.com", :date => Time.local(2004, 12, 12))
@@ -17,13 +16,12 @@ end
 
 class TestController < ActionController::Base
   def send_mail
-    I18nTestMailer.mail_with_i18n_subject(@recipient).deliver
+    I18nTestMailer.mail_with_i18n_subject("test@localhost").deliver
     render :text => 'Mail sent'
   end
 end
 
 class ActionMailerI18nWithControllerTest < ActionDispatch::IntegrationTest
-
   Routes = ActionDispatch::Routing::RouteSet.new
   Routes.draw do
     match ':controller(/:action(/:id))'
@@ -35,12 +33,10 @@ class ActionMailerI18nWithControllerTest < ActionDispatch::IntegrationTest
 
   def setup
     I18n.backend.store_translations('de', :email_subject => '[Signed up] Welcome')
-    set_delivery_method :test
-    ActionMailer::Base.perform_deliveries = true
-    ActionMailer::Base.deliveries.clear
-    ActiveSupport::Deprecation.silenced = false
+  end
 
-    @recipient = 'test@localhost'
+  def teardown
+    I18n.locale = :en
   end
 
   def test_send_mail
diff --git a/actionpack/lib/abstract_controller/rendering.rb b/actionpack/lib/abstract_controller/rendering.rb
index 9b912ea988b..ec1160c31e0 100644
--- a/actionpack/lib/abstract_controller/rendering.rb
+++ b/actionpack/lib/abstract_controller/rendering.rb
@@ -13,14 +13,15 @@ module AbstractController
   # This is a class to fix I18n global state. Whenever you provide I18n.locale during a request,
   # it will trigger the lookup_context and consequently expire the cache.
   class I18nProxy < ::I18n::Config #:nodoc:
-    attr_reader :i18n_config, :lookup_context
+    attr_reader :original_config, :lookup_context
 
-    def initialize(i18n_config, lookup_context)
-      @i18n_config, @lookup_context = i18n_config, lookup_context
+    def initialize(original_config, lookup_context)
+      original_config = original_config.original_config if original_config.respond_to?(:original_config)
+      @original_config, @lookup_context = original_config, lookup_context
     end
 
     def locale
-      @i18n_config.locale
+      @original_config.locale
     end
 
     def locale=(value)
diff --git a/actionpack/lib/action_view/lookup_context.rb b/actionpack/lib/action_view/lookup_context.rb
index 1365048724d..e434f3b0593 100644
--- a/actionpack/lib/action_view/lookup_context.rb
+++ b/actionpack/lib/action_view/lookup_context.rb
@@ -186,11 +186,11 @@ module ActionView
       end
 
       # Overload locale= to also set the I18n.locale. If the current I18n.config object responds
-      # to i18n_config, it means that it's has a copy of the original I18n configuration and it's
+      # to original_config, it means that it's has a copy of the original I18n configuration and it's
       # acting as proxy, which we need to skip.
       def locale=(value)
         if value
-          config = I18n.config.respond_to?(:i18n_config) ? I18n.config.i18n_config : I18n.config
+          config = I18n.config.respond_to?(:original_config) ? I18n.config.original_config : I18n.config
           config.locale = value
         end
 
diff --git a/actionpack/test/template/lookup_context_test.rb b/actionpack/test/template/lookup_context_test.rb
index f7a684779c1..f3b13350008 100644
--- a/actionpack/test/template/lookup_context_test.rb
+++ b/actionpack/test/template/lookup_context_test.rb
@@ -73,7 +73,7 @@ class LookupContextTest < ActiveSupport::TestCase
       assert_equal :pt, I18n.locale
       assert_equal :pt, @lookup_context.locale
     ensure
-      I18n.config = I18n.config.i18n_config
+      I18n.config = I18n.config.original_config
     end
 
     assert_equal :pt, I18n.locale