Added assumption that a Symbol-based scope should end in _id unless it does so already and that you can pass vanilla string-based scopes as a parameter

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@183 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

2 files changed, 17 insertions, 4 deletions

diff --git a/activerecord/test/fixtures/mixin.rb b/activerecord/test/fixtures/mixin.rb
index 6da85e55e7..0c3bf5000a 100644
--- a/activerecord/test/fixtures/mixin.rb
+++ b/activerecord/test/fixtures/mixin.rb
@@ -9,5 +9,13 @@ class ListMixin < ActiveRecord::Base
   def self.table_name
     "mixins"
   end
-    
+end
+
+
+class ListWithStringScopeMixin < ActiveRecord::Base
+  acts_as_list :column => "pos", :scope => 'parent_id = #{parent_id}'
+
+  def self.table_name
+    "mixins"
+  end
 end
\ No newline at end of file
diff --git a/activerecord/test/mixin_test.rb b/activerecord/test/mixin_test.rb
index 107d6d2f0a..58e7ff1807 100644
--- a/activerecord/test/mixin_test.rb
+++ b/activerecord/test/mixin_test.rb
@@ -28,7 +28,7 @@ class ListTest < Test::Unit::TestCase
     
   end
 
-  def test_insert          
+  def test_insert
     new = ListMixin.create("parent_id"=>5)
     assert_equal 1, new.pos
     assert new.first?
@@ -48,8 +48,7 @@ class ListTest < Test::Unit::TestCase
     assert_equal 1, new.pos
     assert new.first?
     assert new.last?
-  end  
-  
+  end    
   
   def test_delete_middle
     first = ListMixin.create("parent_id"=>5)
@@ -68,6 +67,12 @@ class ListTest < Test::Unit::TestCase
     
   end  
 
+  def test_with_string_based_scope
+    new = ListWithStringScopeMixin.create("parent_id"=>5)
+    assert_equal 1, new.pos
+    assert new.first?
+    assert new.last?
+  end 
 end
 
 class TreeTest < Test::Unit::TestCase