diff options
author | Tristan Dunn <tristanzdunn@gmail.com> | 2009-08-10 00:41:36 +0100 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2009-08-10 00:41:36 +0100 |
commit | 0472839d684d836e1e7ecd39722c313410d76d5b (patch) | |
tree | cbb7ac14634a388116cc43600ba2c163f0b0532a /activerecord/test | |
parent | 00d6c766608f90ce64f11feb98786cbc7f71b89d (diff) | |
download | rails-0472839d684d836e1e7ecd39722c313410d76d5b.tar.gz rails-0472839d684d836e1e7ecd39722c313410d76d5b.tar.bz2 rails-0472839d684d836e1e7ecd39722c313410d76d5b.zip |
Prevent overwriting of table name in merging SQL conditions [#2949 state:resolved]
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/named_scope_test.rb | 6 | ||||
-rw-r--r-- | activerecord/test/models/comment.rb | 6 |
2 files changed, 11 insertions, 1 deletions
diff --git a/activerecord/test/cases/named_scope_test.rb b/activerecord/test/cases/named_scope_test.rb index 2a729f0678..bb2aba9d92 100644 --- a/activerecord/test/cases/named_scope_test.rb +++ b/activerecord/test/cases/named_scope_test.rb @@ -368,6 +368,12 @@ class NamedScopeTest < ActiveRecord::TestCase end end end + + def test_table_names_for_chaining_scopes_with_and_without_table_name_included + assert_nothing_raised do + Comment.for_first_post.for_first_author.all + end + end end class DynamicScopeMatchTest < ActiveRecord::TestCase diff --git a/activerecord/test/models/comment.rb b/activerecord/test/models/comment.rb index f7f07c103f..399dea9f12 100644 --- a/activerecord/test/models/comment.rb +++ b/activerecord/test/models/comment.rb @@ -1,6 +1,10 @@ class Comment < ActiveRecord::Base named_scope :containing_the_letter_e, :conditions => "comments.body LIKE '%e%'" - + named_scope :for_first_post, :conditions => { :post_id => 1 } + named_scope :for_first_author, + :joins => :post, + :conditions => { "posts.author_id" => 1 } + belongs_to :post, :counter_cache => true def self.what_are_you |