mirror of
https://github.com/github/rails.git
synced 2026-04-26 03:00:59 -04:00
fix activerecord query_method regression with offset into Fixnum
add test to show offset query_methods on mysql & mysql2 change test to cover public API
This commit is contained in:
@@ -270,7 +270,7 @@ module ActiveRecord
|
||||
arel.having(*@having_values.uniq.reject{|h| h.blank?}) unless @having_values.empty?
|
||||
|
||||
arel.take(connection.sanitize_limit(@limit_value)) if @limit_value
|
||||
arel.skip(@offset_value) if @offset_value
|
||||
arel.skip(@offset_value.to_i) if @offset_value
|
||||
|
||||
arel.group(*@group_values.uniq.reject{|g| g.blank?}) unless @group_values.empty?
|
||||
|
||||
|
||||
@@ -1204,6 +1204,10 @@ class FinderTest < ActiveRecord::TestCase
|
||||
end
|
||||
end
|
||||
|
||||
def test_finder_with_offset_string
|
||||
assert_nothing_raised(ActiveRecord::StatementInvalid) { Topic.find(:all, :offset => "3") }
|
||||
end
|
||||
|
||||
protected
|
||||
def bind(statement, *vars)
|
||||
if vars.first.is_a?(Hash)
|
||||
|
||||
Reference in New Issue
Block a user