Ensure there won't be any regression with where(nil) calls

Consider this scenario:

    if params[:foo]
      conditions = { foo: true }
    end

    foos = Foo.where(conditions).order(:id)

When params[:foo] is nil, this would call:

    foos = Foo.where(nil).order(:id)

In this scenario, we want Foo.where(conditions) to be the same as calling
Foo.all, otherwise we'd get a "NoMethodError order for WhereChain".

Related to #8332.

1 files changed, 4 insertions, 4 deletions

diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb
index d197bfe946..256a7e839f 100644
--- a/activerecord/lib/active_record/relation/query_methods.rb
+++ b/activerecord/lib/active_record/relation/query_methods.rb
@@ -452,8 +452,8 @@ module ActiveRecord
     #
     # If the condition is any other blank-ish object than nil, then where is a # no-op and returns
     # the current relation.
-    def where(opts = nil, *rest)
-      if opts.nil?
+    def where(opts = :chain, *rest)
+      if opts == :chain
         WhereChain.new(spawn)
       elsif opts.blank?
         self
@@ -464,8 +464,8 @@ module ActiveRecord
 
     # #where! is identical to #where, except that instead of returning a new relation, it adds
     # the condition to the existing relation.
-    def where!(opts = nil, *rest) # :nodoc:
-      if opts.nil?
+    def where!(opts = :chain, *rest) # :nodoc:
+      if opts == :chain
         WhereChain.new(self)
       else
         references!(PredicateBuilder.references(opts)) if Hash === opts