MONGOID-5123 Fix methods incompatible with Ruby 3 (#5015)

Author: comandeo-mongo

lib/mongoid.rb

@@ -6,6 +6,7 @@
 require "forwardable"
 require "time"
 require "set"
+require "ruby2_keywords"
 
 require "active_support"
 require "active_support/core_ext"

lib/mongoid/association/embedded/embeds_many/proxy.rb

@@ -422,7 +422,7 @@ def integrate(document)
           # @param [ Proc ] block Optional block to pass.
           #
           # @return [ Criteria, Object ] A Criteria or return value from the target.
-          def method_missing(name, *args, &block)
+          ruby2_keywords def method_missing(name, *args, &block)
             return super if _target.respond_to?(name)
             klass.send(:with_scope, criteria) do
               criteria.public_send(name, *args, &block)

lib/mongoid/association/proxy.rb

@@ -133,12 +133,12 @@ def characterize_one(document)
       # @param [ String, Symbol ] name The name of the method.
       # @param [ Array ] args The arguments passed to the method.
       #
-      def method_missing(name, *args, &block)
+      ruby2_keywords def method_missing(name, *args, &block)
         _target.send(name, *args, &block)
       end
 
       # @api private
-      def respond_to_missing?(name, *args)
+      ruby2_keywords def respond_to_missing?(name, *args)
         _target.respond_to?(name, *args)
       end
 

lib/mongoid/association/referenced/has_many/enumerable.rb

@@ -532,7 +532,7 @@ def set_base(document)
             end
           end
 
-          def method_missing(name, *args, &block)
+          ruby2_keywords def method_missing(name, *args, &block)
             entries.send(name, *args, &block)
           end
 

lib/mongoid/association/referenced/has_many/proxy.rb

@@ -443,7 +443,7 @@ def cascade!(document)
           # @return [ Criteria, Object ] A Criteria or return value from the target.
           #
           # @since 2.0.0.beta.1
-          def method_missing(name, *args, &block)
+          ruby2_keywords def method_missing(name, *args, &block)
             if _target.respond_to?(name)
               _target.send(name, *args, &block)
             else

lib/mongoid/criteria.rb

@@ -567,7 +567,7 @@ def initialize_copy(other)
     # @return [ Object ] The result of the method call.
     #
     # @since 1.0.0
-    def method_missing(name, *args, &block)
+    ruby2_keywords def method_missing(name, *args, &block)
       if klass.respond_to?(name)
         klass.send(:with_scope, self) do
           klass.send(name, *args, &block)

lib/mongoid/interceptable.rb

@@ -125,7 +125,7 @@ def run_before_callbacks(*kinds)
     # @return [ Document ] The document
     #
     # @since 2.3.0
-    def run_callbacks(kind, *args, &block)
+    ruby2_keywords def run_callbacks(kind, *args, &block)
       cascadable_children(kind).each do |child|
         if child.run_callbacks(child_callback_type(kind, child), *args) == false
           return false

mongoid.gemspec

@@ -38,6 +38,18 @@ Gem::Specification.new do |s|
     s.add_dependency("activemodel", [">=6.0", "<6.2"])
   end
   s.add_dependency("mongo", ['>=2.10.5', '<3.0.0'])
+  # Using this gem is recommended for handling argument delegation issues,
+  # especially if support for 2.6 or prior is required.
+  # See https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/#delegation
+  #
+  # We have a bunch of complex delegation logic, including various method_missngs.
+  # If we try to fix them "right", it will add too much logic. We will have to
+  # handle different Ruby versions (including minor ones, Ruby 2.6 and 2.7
+  # behave differently), hash key types (strings vs symbols), ways of passing
+  # arguments (with curly braces vs without ones).
+  #
+  # Therefore, usage of this gem looks like a reasonable solution at the moment.
+  s.add_dependency("ruby2_keywords", "~> 0.0.5")
 
   s.add_development_dependency("bson", ['>=4.9.4', '<5.0.0'])
 

spec/mongoid/association/embedded/embeds_many/proxy_spec.rb

@@ -2449,13 +2449,26 @@ class TrackingIdValidationHistory
     end
 
     context "when providing a criteria class method" do
+      context "without keyword arguments" do
 
-      let(:addresses) do
-        person.addresses.california
+        let(:addresses) do
+          person.addresses.california
+        end
+
+        it "applies the criteria to the documents" do
+          expect(addresses).to eq([ address_one ])
+        end
       end
 
-      it "applies the criteria to the documents" do
-        expect(addresses).to eq([ address_one ])
+      context "with keyword arguments" do
+
+        let(:addresses) do
+          person.addresses.city_and_state(city: "Sacramento", state: "CA")
+        end
+
+        it "applies the criteria to the documents" do
+          expect(addresses).to eq([])
+        end
       end
     end
 

spec/mongoid/association/referenced/belongs_to/proxy_spec.rb

@@ -1349,4 +1349,21 @@ class C
       end
     end
   end
+
+  describe "#method_missing" do
+      let!(:person) do
+        Person.create
+      end
+
+      let!(:game) do
+        Game.create(person: person)
+      end
+
+    it 'handles keyword args' do
+      expect do
+        game.person.set_personal_data(ssn: '123', age: 25)
+      end.not_to raise_error
+    end
+
+  end
 end