diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2009-07-29 16:46:16 -0700 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2009-07-29 16:54:54 -0700 |
commit | d83b1828577c268de56e1b3942e16002c9efdd57 (patch) | |
tree | 7d3038fd5b21e63ce03418d23a52c609cd5def2f /activerecord/test | |
parent | e566fc0577f336be35145b815f3fd0a554eab564 (diff) | |
download | rails-d83b1828577c268de56e1b3942e16002c9efdd57.tar.gz rails-d83b1828577c268de56e1b3942e16002c9efdd57.tar.bz2 rails-d83b1828577c268de56e1b3942e16002c9efdd57.zip |
Revert "Methods invoked within named scope Procs should respect the scope stack. [#1267 state:resolved]"
This reverts commit 6a13376525f34a00e013fc3a6022838329dfe856.
Conflicts:
activerecord/test/cases/named_scope_test.rb
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/named_scope_test.rb | 4 | ||||
-rw-r--r-- | activerecord/test/models/topic.rb | 2 |
2 files changed, 0 insertions, 6 deletions
diff --git a/activerecord/test/cases/named_scope_test.rb b/activerecord/test/cases/named_scope_test.rb index 92fe48cb5a..f4fdc9a39d 100644 --- a/activerecord/test/cases/named_scope_test.rb +++ b/activerecord/test/cases/named_scope_test.rb @@ -354,10 +354,6 @@ class NamedScopeTest < ActiveRecord::TestCase assert_equal [posts(:sti_comments)], Post.with_special_comments.with_post(4).all.uniq end - def test_methods_invoked_within_scopes_should_respect_scope - assert_equal [], Topic.approved.by_rejected_ids.proxy_options[:conditions][:id] - end - def test_named_scopes_batch_finders assert_equal 3, Topic.approved.count diff --git a/activerecord/test/models/topic.rb b/activerecord/test/models/topic.rb index 201d96dcd7..9594dc300a 100644 --- a/activerecord/test/models/topic.rb +++ b/activerecord/test/models/topic.rb @@ -35,8 +35,6 @@ class Topic < ActiveRecord::Base end named_scope :named_extension, :extend => NamedExtension named_scope :multiple_extensions, :extend => [MultipleExtensionTwo, MultipleExtensionOne] - - named_scope :by_rejected_ids, lambda {{ :conditions => { :id => all(:conditions => {:approved => false}).map(&:id) } }} has_many :replies, :dependent => :destroy, :foreign_key => "parent_id" has_many :replies_with_primary_key, :class_name => "Reply", :dependent => :destroy, :primary_key => "title", :foreign_key => "parent_title" |