0
@@ -1,5 +1,8 @@
0
 *SVN*
0
 
0
+* Pass a range in :conditions to use the SQL BETWEEN operator.  #6974 [dcmanges]
0
+    Student.find(:all, :conditions => { :grade => 9..12 })
0
+
0
 * Don't create instance writer methods for class attributes. [Rick]
0
 
0
 * When dealing with SQLite3, use the table_info pragma helper, so that the bindings can do some translation for when sqlite3 breaks incompatibly between point releases. [Jamis Buck]

0
@@ -84,7 +84,7 @@ module ActiveRecord #:nodoc:
0
   # Conditions can either be specified as a string, array, or hash representing the WHERE-part of an SQL statement.
0
   # The array form is to be used when the condition input is tainted and requires sanitization. The string form can
0
   # be used for statements that don't involve tainted data. The hash form works much like the array form, except
0
-  # only equality is possible. Examples:
0
+  # only equality and range is possible. Examples:
0
   #
0
   #   class User < ActiveRecord::Base
0
   #     def self.authenticate_unsafely(user_name, password)
0
@@ -120,6 +120,9 @@ module ActiveRecord #:nodoc:
0
   #   Student.find(:all, :conditions => { :first_name => "Harvey", :status => 1 })
0
   #   Student.find(:all, :conditions => params[:student])
0
   #
0
+  # A range may be used in the hash to use the SQL BETWEEN operator:
0
+  #
0
+  #   Student.find(:all, :conditions => { :grade => 9..12 })
0
   #
0
   # == Overwriting default accessors
0
   #
0
@@ -1264,6 +1267,7 @@ module ActiveRecord #:nodoc:
0
           case argument
0
             when nil   then "IS ?"
0
             when Array then "IN (?)"
0
+            when Range then "BETWEEN ? AND ?"
0
             else            "= ?"
0
           end
0
         end
0
@@ -1392,12 +1396,14 @@ module ActiveRecord #:nodoc:
0
         #     # => "name='foo''bar' and group_id= 4"
0
         #   { :status => nil, :group_id => [1,2,3] }
0
         #     # => "status IS NULL and group_id IN (1,2,3)"
0
+        #   { :age => 13..18 }
0
+        #     # => "age BETWEEN 13 AND 18"
0
         def sanitize_sql_hash(attrs)
0
           conditions = attrs.map do |attr, value|
0
             "#{table_name}.#{connection.quote_column_name(attr)} #{attribute_condition(value)}"
0
           end.join(' AND ')
0
 
0
-          replace_bind_variables(conditions, attrs.values)
0
+          replace_bind_variables(conditions, expand_range_bind_variables(attrs.values))
0
         end
0
 
0
         # Accepts an array of conditions.  The array has each value
0
@@ -1433,6 +1439,13 @@ module ActiveRecord #:nodoc:
0
           end
0
         end
0
 
0
+        def expand_range_bind_variables(bind_vars) #:nodoc:
0
+          bind_vars.each_with_index do |var, index|
0
+            bind_vars[index, 1] = [var.first, var.last] if var.is_a?(Range)
0
+          end
0
+          bind_vars
0
+        end
0
+
0
         def quote_bound_value(value) #:nodoc:
0
           if value.respond_to?(:map) && !value.is_a?(String)
0
             if value.respond_to?(:empty?) && value.empty?

0
@@ -136,6 +136,16 @@ class FinderTest < Test::Unit::TestCase
0
     assert_raises(ActiveRecord::RecordNotFound) { Topic.find(1, :conditions => { :approved => true }) }
0
   end
0
   
0
+  def test_find_on_hash_conditions_with_range
0
+    assert_equal [1,2], Topic.find(:all, :conditions => { :id => 1..2 }).map(&:id).sort
0
+    assert_raises(ActiveRecord::RecordNotFound) { Topic.find(1, :conditions => { :id => 2..3 }) }
0
+  end
0
+  
0
+  def test_find_on_hash_conditions_with_multiple_ranges
0
+    assert_equal [1,2,3], Comment.find(:all, :conditions => { :id => 1..3, :post_id => 1..2 }).map(&:id).sort
0
+    assert_equal [1], Comment.find(:all, :conditions => { :id => 1..1, :post_id => 1..10 }).map(&:id).sort
0
+  end
0
+  
0
   def test_find_on_multiple_hash_conditions
0
     assert Topic.find(1, :conditions => { :author_name => "David", :title => "The First Topic", :replies_count => 1, :approved => false })
0
     assert_raises(ActiveRecord::RecordNotFound) { Topic.find(1, :conditions => { :author_name => "David", :title => "The First Topic", :replies_count => 1, :approved => true }) }