From d8bd19f6abd1f8cb344ff5c77971a357c4f5b3ec Mon Sep 17 00:00:00 2001
From: Andrew White <andyw@pixeltrix.co.uk>
Date: Tue, 24 May 2011 07:21:32 +0100
Subject: [PATCH] Ignore :includes on through associations

---
 .../lib/active_record/associations/through_association.rb   | 2 +-
 activerecord/test/cases/associations/eager_test.rb          | 6 ++++++
 activerecord/test/models/author.rb                          | 3 +++
 activerecord/test/models/post.rb                            | 6 ++++++
 4 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/activerecord/lib/active_record/associations/through_association.rb b/activerecord/lib/active_record/associations/through_association.rb
index 53c5c3cedf..81172179e0 100644
--- a/activerecord/lib/active_record/associations/through_association.rb
+++ b/activerecord/lib/active_record/associations/through_association.rb
@@ -16,7 +16,7 @@ def target_scope
           chain[1..-1].each do |reflection|
             scope = scope.merge(
               reflection.klass.scoped.with_default_scope.
-                except(:select, :create_with)
+                except(:select, :create_with, :includes)
             )
           end
           scope
diff --git a/activerecord/test/cases/associations/eager_test.rb b/activerecord/test/cases/associations/eager_test.rb
index 3e92a77830..ea186be9dd 100644
--- a/activerecord/test/cases/associations/eager_test.rb
+++ b/activerecord/test/cases/associations/eager_test.rb
@@ -448,6 +448,12 @@ def test_eager_with_has_many_through_with_conditions_join_model_with_include
     assert_equal post_tags, eager_post_tags
   end
 
+  def test_eager_with_has_many_through_join_model_ignores_default_includes
+    assert_nothing_raised do
+      authors(:david).comments_on_posts_with_default_include.to_a
+    end
+  end
+
   def test_eager_with_has_many_and_limit
     posts = Post.find(:all, :order => 'posts.id asc', :include => [ :author, :comments ], :limit => 2)
     assert_equal 2, posts.size
diff --git a/activerecord/test/models/author.rb b/activerecord/test/models/author.rb
index e0cbc44265..23db5650d4 100644
--- a/activerecord/test/models/author.rb
+++ b/activerecord/test/models/author.rb
@@ -138,6 +138,9 @@ def testing_proxy_target
   has_many :misc_post_first_blue_tags_2, :through => :posts, :source => :first_blue_tags_2,
            :conditions => { :posts => { :title => ['misc post by bob', 'misc post by mary'] } }
 
+  has_many :posts_with_default_include, :class_name => 'PostWithDefaultInclude'
+  has_many :comments_on_posts_with_default_include, :through => :posts_with_default_include, :source => :comments
+
   scope :relation_include_posts, includes(:posts)
   scope :relation_include_tags, includes(:tags)
 
diff --git a/activerecord/test/models/post.rb b/activerecord/test/models/post.rb
index 80296032bb..f2ab7b053e 100644
--- a/activerecord/test/models/post.rb
+++ b/activerecord/test/models/post.rb
@@ -162,3 +162,9 @@ class FirstPost < ActiveRecord::Base
   has_many :comments, :foreign_key => :post_id
   has_one  :comment,  :foreign_key => :post_id
 end
+
+class PostWithDefaultInclude < ActiveRecord::Base
+  self.table_name = 'posts'
+  default_scope includes(:comments)
+  has_many :comments, :foreign_key => :post_id
+end
\ No newline at end of file
-- 
GitLab