mirror of
https://github.com/github/rails.git
synced 2026-04-26 03:00:59 -04:00
Replace Base#safe_to_array with Array.wrap
This commit is contained in:
@@ -1645,7 +1645,7 @@ module ActiveRecord #:nodoc:
|
||||
|
||||
# Merges includes so that the result is a valid +include+
|
||||
def merge_includes(first, second)
|
||||
(safe_to_array(first) + safe_to_array(second)).uniq
|
||||
(Array.wrap(first) + Array.wrap(second)).uniq
|
||||
end
|
||||
|
||||
def merge_joins(*joins)
|
||||
@@ -1657,7 +1657,7 @@ module ActiveRecord #:nodoc:
|
||||
end
|
||||
joins.flatten.map{|j| j.strip}.uniq
|
||||
else
|
||||
joins.collect{|j| safe_to_array(j)}.flatten.uniq
|
||||
joins.collect{|j| Array.wrap(j)}.flatten.uniq
|
||||
end
|
||||
end
|
||||
|
||||
@@ -1674,18 +1674,6 @@ module ActiveRecord #:nodoc:
|
||||
}.join(" ")
|
||||
end
|
||||
|
||||
# Object#to_a is deprecated, though it does have the desired behavior
|
||||
def safe_to_array(o)
|
||||
case o
|
||||
when NilClass
|
||||
[]
|
||||
when Array
|
||||
o
|
||||
else
|
||||
[o]
|
||||
end
|
||||
end
|
||||
|
||||
def array_of_strings?(o)
|
||||
o.is_a?(Array) && o.all?{|obj| obj.is_a?(String)}
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user