aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2014-02-14 14:20:07 -0800
committerAaron Patterson <aaron.patterson@gmail.com>2014-02-14 14:20:07 -0800
commit8e6ef92fd9ede38aa435afeaef6370ff65191a0b (patch)
tree03dc053a579d5401a12498e9ea0716881b08bb0c /activerecord
parent3b675f05ffb11cdbfd3f7d416906ff16941e8367 (diff)
downloadrails-8e6ef92fd9ede38aa435afeaef6370ff65191a0b.tar.gz
rails-8e6ef92fd9ede38aa435afeaef6370ff65191a0b.tar.bz2
rails-8e6ef92fd9ede38aa435afeaef6370ff65191a0b.zip
pass the association and connection to the scope method
Diffstat (limited to 'activerecord')
-rw-r--r--activerecord/lib/active_record/associations/association.rb2
-rw-r--r--activerecord/lib/active_record/associations/association_scope.rb10
-rw-r--r--activerecord/test/cases/associations/association_scope_test.rb6
3 files changed, 7 insertions, 11 deletions
diff --git a/activerecord/lib/active_record/associations/association.rb b/activerecord/lib/active_record/associations/association.rb
index 67ea489b22..2dba33898c 100644
--- a/activerecord/lib/active_record/associations/association.rb
+++ b/activerecord/lib/active_record/associations/association.rb
@@ -94,7 +94,7 @@ module ActiveRecord
# actually gets built.
def association_scope
if klass
- @association_scope ||= AssociationScope.new(self).scope
+ @association_scope ||= AssociationScope.new.scope(self, klass.connection)
end
end
diff --git a/activerecord/lib/active_record/associations/association_scope.rb b/activerecord/lib/active_record/associations/association_scope.rb
index 1f094d42b3..cd2911e0a4 100644
--- a/activerecord/lib/active_record/associations/association_scope.rb
+++ b/activerecord/lib/active_record/associations/association_scope.rb
@@ -1,18 +1,12 @@
module ActiveRecord
module Associations
class AssociationScope #:nodoc:
- attr_reader :association
-
- def initialize(association)
- @association = association
- end
-
- def scope
+ def scope(association, connection)
klass = association.klass
reflection = association.reflection
scope = klass.unscoped
owner = association.owner
- alias_tracker = AliasTracker.new klass.connection
+ alias_tracker = AliasTracker.new connection
scope.extending! Array(reflection.options[:extend])
add_constraints(scope, owner, klass, reflection, alias_tracker)
diff --git a/activerecord/test/cases/associations/association_scope_test.rb b/activerecord/test/cases/associations/association_scope_test.rb
index d38648202e..f9793277e5 100644
--- a/activerecord/test/cases/associations/association_scope_test.rb
+++ b/activerecord/test/cases/associations/association_scope_test.rb
@@ -6,8 +6,10 @@ module ActiveRecord
module Associations
class AssociationScopeTest < ActiveRecord::TestCase
test 'does not duplicate conditions' do
- association_scope = AssociationScope.new(Author.new.association(:welcome_posts))
- wheres = association_scope.scope.where_values.map(&:right)
+ association_scope = AssociationScope.new
+ scope = association_scope.scope(Author.new.association(:welcome_posts),
+ Author.connection)
+ wheres = scope.where_values.map(&:right)
assert_equal wheres.uniq, wheres
end
end