From 640f6c201d59cd7fc16943154372d4ef754f33f4 Mon Sep 17 00:00:00 2001
From: kennyj <kennyj@gmail.com>
Date: Sat, 3 Mar 2012 20:32:46 +0900
Subject: [PATCH 1/2] Fix GH #4674. Reset column information and sequence name
 when setting table_name.

---
 activerecord/lib/active_record/model_schema.rb | 15 +++++++++++----
 activerecord/test/cases/base_test.rb           | 13 +++++++++++++
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/activerecord/lib/active_record/model_schema.rb b/activerecord/lib/active_record/model_schema.rb
index 60bac2f802..8710884bfd 100644
--- a/activerecord/lib/active_record/model_schema.rb
+++ b/activerecord/lib/active_record/model_schema.rb
@@ -114,10 +114,17 @@ def table_name
       # You can also just define your own <tt>self.table_name</tt> method; see
       # the documentation for ActiveRecord::Base#table_name.
       def table_name=(value)
-        @table_name        = value && value.to_s
-        @quoted_table_name = nil
-        @arel_table        = nil
-        @relation          = Relation.new(self, arel_table)
+        value = value && value.to_s
+        if defined?(@table_name)
+          return if value == @table_name
+
+          reset_column_information
+        end
+        @table_name          = value
+        @quoted_table_name   = nil
+        @arel_table          = nil
+        @sequence_name       = nil
+        @relation            = Relation.new(self, arel_table)
       end
 
       # Returns a quoted version of the table name, used to construct SQL statements.
diff --git a/activerecord/test/cases/base_test.rb b/activerecord/test/cases/base_test.rb
index 698c3d0cb1..d810704418 100644
--- a/activerecord/test/cases/base_test.rb
+++ b/activerecord/test/cases/base_test.rb
@@ -1479,6 +1479,19 @@ def test_switching_between_table_name
     end
   end
 
+  def test_clear_cash_when_setting_table_name
+    Joke.table_name = "cold_jokes"
+    before_columns = Joke.columns
+    before_seq     = Joke.sequence_name
+
+    Joke.table_name = "funny_jokes"
+    after_columns = Joke.columns
+    after_seq     = Joke.sequence_name
+
+    assert_not_equal before_columns, after_columns
+    assert_not_equal before_seq, after_seq unless before_seq.blank? && after_seq.blank?
+  end
+
   def test_set_table_name_symbol_converted_to_string
     Joke.table_name = :cold_jokes
     assert_equal 'cold_jokes', Joke.table_name

From 200d994870e0b2e59c31ab9d268def19abc563cf Mon Sep 17 00:00:00 2001
From: kennyj <kennyj@gmail.com>
Date: Sun, 4 Mar 2012 22:18:57 +0900
Subject: [PATCH 2/2] Don't clear sequence name when we explicitly assign it.

---
 activerecord/lib/active_record/model_schema.rb |  6 ++++--
 activerecord/test/cases/base_test.rb           | 11 +++++++++++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/activerecord/lib/active_record/model_schema.rb b/activerecord/lib/active_record/model_schema.rb
index 8710884bfd..73c9e22aa3 100644
--- a/activerecord/lib/active_record/model_schema.rb
+++ b/activerecord/lib/active_record/model_schema.rb
@@ -123,7 +123,7 @@ def table_name=(value)
         @table_name          = value
         @quoted_table_name   = nil
         @arel_table          = nil
-        @sequence_name       = nil
+        @sequence_name       = nil unless defined?(@explicitly_sequence_name) && @explicitly_sequence_name
         @relation            = Relation.new(self, arel_table)
       end
 
@@ -170,7 +170,8 @@ def sequence_name
       end
 
       def reset_sequence_name #:nodoc:
-        self.sequence_name = connection.default_sequence_name(table_name, primary_key)
+        @sequence_name = connection.default_sequence_name(table_name, primary_key)
+        @explicitly_sequence_name = false
       end
 
       # Sets the name of the sequence to use when generating ids to the given
@@ -189,6 +190,7 @@ def reset_sequence_name #:nodoc:
       #   end
       def sequence_name=(value)
         @sequence_name = value.to_s
+        @explicitly_sequence_name = true
       end
 
       # Indicates whether the table associated with this class exists
diff --git a/activerecord/test/cases/base_test.rb b/activerecord/test/cases/base_test.rb
index d810704418..444ea474b1 100644
--- a/activerecord/test/cases/base_test.rb
+++ b/activerecord/test/cases/base_test.rb
@@ -1492,6 +1492,17 @@ def test_clear_cash_when_setting_table_name
     assert_not_equal before_seq, after_seq unless before_seq.blank? && after_seq.blank?
   end
 
+  def test_dont_clear_sequence_name_when_setting_explicitly
+    Joke.sequence_name = "black_jokes_seq"
+    Joke.table_name    = "cold_jokes"
+    before_seq         = Joke.sequence_name
+
+    Joke.table_name    = "funny_jokes"
+    after_seq          = Joke.sequence_name
+
+    assert_equal before_seq, after_seq unless before_seq.blank? && after_seq.blank?
+  end
+
   def test_set_table_name_symbol_converted_to_string
     Joke.table_name = :cold_jokes
     assert_equal 'cold_jokes', Joke.table_name