Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fixed default scope of acts_as_list from "1" to "1 = 1", so itll work…

… in PostgreSQL (among other places) #427 [Alexey]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@380 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
commit a971dc5e4aa45f747d375c3703fadc5d1568c876 1 parent dbb5341
@dhh dhh authored
View
2  activerecord/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Fixed default scope of acts_as_list from "1" to "1 = 1", so it'll work in PostgreSQL (among other places) #427 [Alexey]
+
* Added Base#reload that reloads the attributes of an object from the database #422 [Andreas Schwarz]
* Added SQLite3 compatibility through the sqlite3-ruby adapter by Jamis Buck #381 [bitsweat]
View
2  activerecord/lib/active_record/acts/list.rb
@@ -31,7 +31,7 @@ module ClassMethods
# as the foreign key restriction. It's also possible to give it an entire string that is interpolated if you need a tighter scope than
# just a foreign key. Example: <tt>acts_as_list :scope => 'todo_list_id = #{todo_list_id} AND completed = 0'</tt>
def acts_as_list(options = {})
- configuration = { :column => "position", :scope => "1" }
+ configuration = { :column => "position", :scope => "1 = 1" }
configuration.update(options) if options.is_a?(Hash)
configuration[:scope] = "#{configuration[:scope]}_id".intern if configuration[:scope].is_a?(Symbol) && configuration[:scope].to_s !~ /_id$/
Please sign in to comment.
Something went wrong with that request. Please try again.