MONGOID-5330 Operator methods overwrite previous conditions (#5263)

* MONGOID-5330 add failing test

* MONGOID-5330 pend test

* MONGOID-5330 make gt use and

* MONGOID-5330 change hard coded value

* MONGOID-5330 fix tests

* MONGOID-5330 add feature flag

* MONGOID-5330 use the feature flag

* MONGOID-5330 add testing and fix eq

* MONGOID-5330 change op map

* MONGOID-5330 fix elemMatch

* Update lib/mongoid/criteria/queryable/mergeable.rb

* Update lib/mongoid/criteria/queryable/mergeable.rb

* Update lib/mongoid/criteria/queryable/mergeable.rb

* MONGOID-5330 make this feature flag in 7.5

* MONGOID-5330 remove extra spacing

* MONGOID-5330 update docs to update default

* MONGOID-5330 update tests after switching defaults

* MONGOID-5330 update docs with note about strings vs symbols

* MONGOID-5330 update test

* MONGOID-5330 change feature flag to overwrite

Co-authored-by: Oleg Pudeyev <[email protected]>

Author: neilshweky

source/reference/configuration.txt

@@ -395,6 +395,12 @@ for details on driver options.
       # (default: false)
       #object_id_as_json_oid: true
 
+      # When chaining the same operators that use the same field, setting this
+      # feature flag to false will cause those operators to be combined using an
+      # and. Setting this feature flag to true will cause the later chained
+      # operators to overwrite the earlier ones. (default: false)
+      #overwrite_chained_operators: false
+
       # Preload all models in development, needed when models use
       # inheritance. (default: false)
       preload_models: false

source/release-notes/mongoid-7.5.txt

@@ -0,0 +1,82 @@
+***********
+Mongoid 7.5
+***********
+
+.. default-domain:: mongodb
+
+.. contents:: On this page
+   :local:
+   :backlinks: none
+   :depth: 2
+   :class: singlecol
+
+This page describes significant changes and improvements in Mongoid 7.5.
+The complete list of releases is available `on GitHub
+<https://github.com/mongodb/mongoid/releases>`_ and `in JIRA
+<https://jira.mongodb.org/projects/MONGOID?selectedItem=com.atlassian.jira.jira-projects-plugin:release-page>`_;
+please consult GitHub releases for detailed release notes and JIRA for
+the complete list of issues fixed in each release, including bug fixes.
+
+
+``Document#to_a`` deprecated
+````````````````````````````
+
+The ``Document#to_a`` method is deprecated in Mongoid 7.5.
+
+
+Combine Chained Operators Using ``and`` Instead of ``override``
+```````````````````````````````````````````````````````````````
+
+Mongoid 7.5 with the ``Mongoid.overwrite_chained_operators`` option set to ``false``
+will combine conditions that use the same operator and field using an ``and``.
+For example, in the following query:
+
+.. code-block:: ruby
+
+  Band.ne(name: "The Rolling Stones").ne(name: "The Beatles")
+
+Mongoid 7.5 with the ``Mongoid.overwrite_chained_operators`` option set to ``false``
+will generate the following criteria:
+
+.. code-block:: ruby
+
+  #<Mongoid::Criteria
+    selector: {"name"=>{"$ne"=>"The Rolling Stones"}, "$and"=>[{"name"=>{"$ne"=>"The Beatles"}}]}
+    options:  {}
+    class:    Band
+    embedded: false>
+
+In Mongoid 7.4 and earlier, and in 7.5 with the ``Mongoid.overwrite_chained_operators``
+option set to ``true``, the following criteria would be generated instead which
+overwrites the first condition:
+
+.. code-block:: ruby
+
+  #<Mongoid::Criteria
+    selector: {"name"=>{"$ne"=>"The Beatles"}}
+    options:  {}
+    class:    Band
+    embedded: false>
+
+The following functions are affected by this change:
+
+- ``eq``
+- ``elem_match``
+- ``gt``
+- ``gte``
+- ``lt``
+- ``lte``
+- ``mod``
+- ``ne``
+- ``near``
+- ``near_sphere``
+
+.. note::
+
+   In Mongoid 7.5 with the ``Mongoid.overwrite_chained_operators`` option set to
+   ``false``, nested keys in the generated selector will always be strings,
+   whereas in Mongoid 7.4 and earlier, and in 7.5 with the
+   ``Mongoid.overwrite_chained_operators`` option set to ``true``, nested keys in
+   the selector can be strings or symbols, depending on what was passed to the
+   operator.
+

source/release-notes/mongoid-8.0.txt

@@ -31,6 +31,7 @@ Default Option Values Changed
 **Breaking change:** The following options have had their default values
 changed in Mongoid 8.0:
 
+- ``:overwrite_chained_operators`` => ``false``
 - ``:broken_aggregables`` => ``false``
 - ``:broken_alias_handling`` => ``false``
 - ``:broken_and`` => ``false``
@@ -161,20 +162,20 @@ Mongoid 8.0 behavior:
 
 .. code-block:: ruby
 
-  Band.any_of({name: 'Rolling Stone'}, {founded: 1990}).
+  Band.any_of({name: 'The Rolling Stones'}, {founded: 1990}).
     any_of({members: 2}, {last_tour: 1995})
   # =>
   # #<Mongoid::Criteria
-  #   selector: {"$or"=>[{"name"=>"Rolling Stone"}, {"founded"=>1990}],
+  #   selector: {"$or"=>[{"name"=>"The Rolling Stones"}, {"founded"=>1990}],
   #     "$and"=>[{"$or"=>[{"members"=>2}, {"last_tour"=>1995}]}]}
   #   options:  {}
   #   class:    Band
   #   embedded: false>
 
-  Band.any_of({name: 'Rolling Stone'}, {founded: 1990}).any_of({members: 2})
+  Band.any_of({name: 'The Rolling Stones'}, {founded: 1990}).any_of({members: 2})
   # =>
   # #<Mongoid::Criteria
-  #   selector: {"$or"=>[{"name"=>"Rolling Stone"}, {"founded"=>1990}], "members"=>2}
+  #   selector: {"$or"=>[{"name"=>"The Rolling Stones"}, {"founded"=>1990}], "members"=>2}
   #   options:  {}
   #   class:    Band
   #   embedded: false>
@@ -183,20 +184,20 @@ Mongoid 7 behavior:
 
 .. code-block:: ruby
 
-  Band.any_of({name: 'Rolling Stone'}, {founded: 1990}).
+  Band.any_of({name: 'The Rolling Stones'}, {founded: 1990}).
     any_of({members: 2}, {last_tour: 1995})
   # =>
   # #<Mongoid::Criteria
-  #   selector: {"$or"=>[{"name"=>"Rolling Stone"}, {"founded"=>1990},
+  #   selector: {"$or"=>[{"name"=>"The Rolling Stones"}, {"founded"=>1990},
   #     {"members"=>2}, {"last_tour"=>1995}]}
   #   options:  {}
   #   class:    Band
   #   embedded: false>
 
-  Band.any_of({name: 'Rolling Stone'}, {founded: 1990}).any_of({members: 2})
+  Band.any_of({name: 'The Rolling Stones'}, {founded: 1990}).any_of({members: 2})
   # =>
   # #<Mongoid::Criteria
-  #   selector: {"$or"=>[{"name"=>"Rolling Stone"}, {"founded"=>1990}], "members"=>2}
+  #   selector: {"$or"=>[{"name"=>"The Rolling Stones"}, {"founded"=>1990}], "members"=>2}
   #   options:  {}
   #   class:    Band
   #   embedded: false>