aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record
diff options
context:
space:
mode:
authorRafael França <rafaelmfranca@gmail.com>2016-08-16 05:50:18 -0300
committerGitHub <noreply@github.com>2016-08-16 05:50:18 -0300
commitd60f3495be2df097c483e5aab227a8a636a55f4c (patch)
treea450a2a0668510cad168e94bd238f8f6cd5739a0 /activerecord/lib/active_record
parentcaef3b92149c2f73536110ec0ef582b43e48329c (diff)
parentf79d55a4cca155171c6d16c3c07834ba43d572c2 (diff)
downloadrails-d60f3495be2df097c483e5aab227a8a636a55f4c.tar.gz
rails-d60f3495be2df097c483e5aab227a8a636a55f4c.tar.bz2
rails-d60f3495be2df097c483e5aab227a8a636a55f4c.zip
Merge pull request #26074 from kamipo/where_by_attribute_with_array_or_range
`where` by `array|range` attribute with array or range value
Diffstat (limited to 'activerecord/lib/active_record')
-rw-r--r--activerecord/lib/active_record/relation/predicate_builder.rb18
1 files changed, 11 insertions, 7 deletions
diff --git a/activerecord/lib/active_record/relation/predicate_builder.rb b/activerecord/lib/active_record/relation/predicate_builder.rb
index 505a78cb78..9d6fc9f702 100644
--- a/activerecord/lib/active_record/relation/predicate_builder.rb
+++ b/activerecord/lib/active_record/relation/predicate_builder.rb
@@ -87,14 +87,14 @@ module ActiveRecord
binds = []
attributes.each do |column_name, value|
- case value
- when Hash
+ case
+ when value.is_a?(Hash)
attrs, bvs = associated_predicate_builder(column_name).create_binds_for_hash(value)
result[column_name] = attrs
binds += bvs
- when Relation
+ when value.is_a?(Relation)
binds += value.bound_attributes
- when Range
+ when value.is_a?(Range) && !table.type(column_name).respond_to?(:subtype)
first = value.begin
last = value.end
unless first.respond_to?(:infinite?) && first.infinite?
@@ -155,9 +155,13 @@ module ActiveRecord
end
def can_be_bound?(column_name, value)
- !value.nil? &&
- handler_for(value).is_a?(BasicObjectHandler) &&
- !table.associated_with?(column_name)
+ return if table.associated_with?(column_name)
+ case value
+ when Array, Range
+ table.type(column_name).respond_to?(:subtype)
+ else
+ !value.nil? && handler_for(value).is_a?(BasicObjectHandler)
+ end
end
def build_bind_param(column_name, value)