Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Merge pull request #6089 from frodsan/delete_all_limit

delete_all raise an error if a limit is provided - fixes #4979
  • Loading branch information...
commit 7d2df5fa62da393e429393ccd161d237117932fd 2 parents b2a24a1 + 2f68125
Aaron Patterson tenderlove authored
2  activerecord/lib/active_record/relation.rb
View
@@ -390,6 +390,8 @@ def destroy(id)
# If you need to destroy dependent associations or call your <tt>before_*</tt> or
# +after_destroy+ callbacks, use the +destroy_all+ method instead.
def delete_all(conditions = nil)
+ raise ActiveRecordError.new("delete_all doesn't support limit scope") if self.limit_value
+
if conditions
where(conditions).delete_all
else
4 activerecord/test/cases/relations_test.rb
View
@@ -688,6 +688,10 @@ def test_delete_all_loaded
assert davids.loaded?
end
+ def test_delete_all_limit_error
+ assert_raises(ActiveRecord::ActiveRecordError) { Author.limit(10).delete_all }
+ end
+
def test_select_argument_error
assert_raises(ArgumentError) { Developer.select }
end
Please sign in to comment.
Something went wrong with that request. Please try again.