mongoid 9.0.11 → 9.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6976a4808058209fda8554306590eeb5fd44759d8c228d02bca544acbe5c200
-  data.tar.gz: e0dcd20719682c2a425d9ed17c1459376b6956311f9ec439c74843a4cefb8502
+  metadata.gz: '0182a5e78d9fecbf490a537bd4dcf5aba640683c566b400ff7c70b189201372f'
+  data.tar.gz: d194627c4d001e4b420652d52a8612e6f4c4209948259f3d1f4b33de4257920b
 SHA512:
-  metadata.gz: 64ee6ee13a326ed9fae0e6b6dc08fa28ba0608a75342993b873023e4f87076160f6a8997e3edd53a273da4716c92d5aabc6b49f0931a489d2074d92f8de912d7
-  data.tar.gz: 491ecfb05581a08609bc41aedd39600f143a15f681405400c813402be95e11ed24d4be7363ac8079da9bc022d7d5b22ad3c53e534d41ee66a10977751a44b954
+  metadata.gz: b0e2447ca53e3ac80aee5a4a29dc9fe45110ded5f35a6e25569b9aca9e34a1f48060ef17a2dc9257f4d4d5ec3d07d725679a1f83e6850fa24edb32099b471ff2
+  data.tar.gz: b41c4d87a55e8ba10c90b1998f36280021793c6c4816d54259642a29a23f39201afb5c272cf473545f0b14a47ea609e6be8b8b35e68ee5e39076c1f2ceaa64f1

data/CHANGELOG.md

@@ -105,7 +105,7 @@ For instructions on upgrading to newer versions, visit
 
 * \#4005 Fixed inclusion of mongoid with Rails components that don't have the Rails environment.
 
-* \#3993 Fixes issue where `dup`/`clone` fails for embedded documents that use store_as without using Mongoid::Atributes::Dynamic
+* \#3993 Fixes issue where `dup`/`clone` fails for embedded documents that use store_as without using Mongoid::Attributes::Dynamic
 
 * \#3991 Fixed embedded documents not flagging as changed after calling #changed? and modifying the
 child elements.
@@ -798,7 +798,7 @@ child elements.
         Band.where(name: "Placebo").unset(:members, :origin)
 
 * \#2669 Passing a block to `Criteria#new` now properly sends the
-  block through to the model's contructor. (Arthur Neves)
+  block through to the model's constructor. (Arthur Neves)
 
 * \#2667 `exists?` no longer hits the database in cases where we have
   the necessary information in memory.
@@ -1200,7 +1200,7 @@ child elements.
 
 * \#2571 Remove blank error message from locales. (Jordan Elver)
 
-* \#2568 Fix uniqueness validation for lacalized fields when a scope is also
+* \#2568 Fix uniqueness validation for localized fields when a scope is also
   provided.
 
 * \#2552 Ensure `InvalidPath` errors are raised when embedded documents try to
@@ -1609,7 +1609,7 @@ child elements.
 
         Band.first.touch
 
-    Update a specific time field along with the udpated_at.
+    Update a specific time field along with the updated_at.
 
         Band.first.touch(:founded)
 
@@ -1728,7 +1728,7 @@ child elements.
         # { "name" => "foo", "my_preferences" => [{ "value" => "ok" }]}
 
 * \#1806 `Model.find_or_create_by` and `Model.find_or_initialize_by` can now
-  take documents as paramters for belongs_to relations.
+  take documents as parameters for belongs_to relations.
 
         person = Person.first
         Game.find_or_create_by(person: person)
@@ -2506,7 +2506,7 @@ child elements.
 * \#2038 Allow inverse relations to be determined by foreign keys alone
   if defined on both sides, not just an inverse_of declaration.
 
-* \#2023 Allow serilialization of dynamic types that conflict with core
+* \#2023 Allow serialization of dynamic types that conflict with core
   Ruby methods to still be serialized.
 
 * \#2008 Presence validation should hit the db to check validity if the
@@ -2611,7 +2611,7 @@ child elements.
   had the foreign key link persisted.
 
 * \#1820 Destroying embedded documents in an embeds_many should also
-  removed the document from the underlying _uncoped target and reindex
+  removed the document from the underlying _unscoped target and reindex
   the relation.
 
 * \#1814 Don't cascade callbacks on after_initialize.
@@ -3059,7 +3059,7 @@ child elements.
   key without error.
 
 * \#1350, \#1351 Fixed errors in the string conversions of double quotes and
-  tilde when paramterizing keys.
+  tilde when parametrizing keys.
 
 * \#1349 Mongoid documents should not blow up when including Enumerable.
   (Jonas Nicklas)

data/README.md

@@ -8,13 +8,9 @@ Mongoid is an ODM (Object-Document Mapper) framework for MongoDB in Ruby.
 Documentation
 -------------
 
-Mongoid has [extensive user documentation](https://www.mongodb.com/docs/mongoid/current/).
-[API documentation](https://www.mongodb.com/docs/mongoid/current/api/) is also available.
-
-Mongoid is built on top of the MongoDB Ruby driver which has
-[its own user documentation](https://www.mongodb.com/docs/ruby-driver/current/).
-
-High-level Mongoid documentation including tutorials and the reference that were in the docs folder can now be found at the docs-mongoid repository, [here](https://github.com/mongodb/docs-mongoid).
+* [User documentation](https://www.mongodb.com/docs/mongoid/current/)
+* [API documentation](https://www.mongodb.com/docs/mongoid/current/api/)
+* [Ruby driver documentation](https://www.mongodb.com/docs/ruby-driver/current/) (which Mongoid is built on top of)
 
 Compatibility
 -------------
@@ -28,20 +24,18 @@ Mongoid supports and is tested against:
 Issues
 ------
 
-Please use the [MONGOID project](https://jira.mongodb.org/browse/MONGOID/)
-in MongoDB JIRA to report issues with Mongoid.
+Please use the [MONGOID project](https://jira.mongodb.org/browse/MONGOID/) in MongoDB JIRA to report issues with Mongoid.
 
 Support
 -------
 
 * [Stack Overflow](http://stackoverflow.com/questions/tagged/mongoid)
 * [MongoDB Community Forum](https://developer.mongodb.com/community/forums/tags/c/drivers-odms-connectors/7/mongoid-odm)
-* [#mongoid](http://webchat.freenode.net/?channels=mongoid) on Freenode IRC
 
 License
 -------
 
-Copyright (c) 2015-Present MongoDB Inc.
+Copyright (c) 2015-Present MongoDB Inc. \
 Copyright (c) 2009-2016 Durran Jordan
 
 Permission is hereby granted, free of charge, to any person obtaining

data/Rakefile

@@ -1,19 +1,16 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "bundler"
+require 'bundler'
 Bundler.setup
 
 ROOT = File.expand_path(File.join(File.dirname(__FILE__)))
 
 $: << File.join(ROOT, 'spec/shared/lib')
 
-require "rake"
-require "rspec/core/rake_task"
+require 'rake'
+require 'rspec/core/rake_task'
 
-if File.exist?('./spec/shared/lib/tasks/candidate.rake')
-  load 'spec/shared/lib/tasks/candidate.rake'
-end
+load 'spec/shared/lib/tasks/candidate.rake' if File.exist?('./spec/shared/lib/tasks/candidate.rake')
 
 desc 'Build the gem'
 task :build do
@@ -26,9 +23,9 @@ end
 # `rake version` is used by the deployment system so get the release version
 # of the product beng deployed. It must do nothing more than just print the
 # product version number.
-# 
+#
 # See the mongodb-labs/driver-github-tools/ruby/publish Github action.
-desc "Print the current value of Mongoid::VERSION"
+desc 'Print the current value of Mongoid::VERSION'
 task :version do
   require 'mongoid/version'
 
@@ -59,13 +56,13 @@ task :release do
   system 'gem', 'push', "mongoid-#{Mongoid::VERSION}.gem"
 end
 
-RSpec::Core::RakeTask.new("spec") do |spec|
-  spec.pattern = "spec/**/*_spec.rb"
+RSpec::Core::RakeTask.new('spec') do |spec|
+  spec.pattern = 'spec/**/*_spec.rb'
 end
 
 RSpec::Core::RakeTask.new('spec:progress') do |spec|
-  spec.rspec_opts = %w(--format progress)
-  spec.pattern = "spec/**/*_spec.rb"
+  spec.rspec_opts = %w[--format progress]
+  spec.pattern = 'spec/**/*_spec.rb'
 end
 
 RUBOCOPABLE = %w[ examples gemfiles perf lib spec mongoid.gemspec Gemfile Rakefile upload-api-docs ].freeze
@@ -140,18 +137,18 @@ namespace :generate do
 end
 
 CLASSIFIERS = [
-  [%r,^mongoid/attribute,, :attributes],
-  [%r,^mongoid/association/[or],, :associations_referenced],
-  [%r,^mongoid/association,, :associations],
-  [%r,^mongoid,, :unit],
-  [%r,^integration,, :integration],
-  [%r,^rails,, :rails],
+  [ %r{^mongoid/attribute}, :attributes ],
+  [ %r{^mongoid/association/[or]}, :associations_referenced ],
+  [ %r{^mongoid/association}, :associations ],
+  [ /^mongoid/, :unit ],
+  [ /^integration/, :integration ],
+  [ /^rails/, :rails ]
 ]
 
-RUN_PRIORITY = %i(
+RUN_PRIORITY = %i[
   unit attributes associations_referenced associations
   integration rails
-)
+]
 
 def spec_organizer
   require 'mrss/spec_organizer'
@@ -159,7 +156,7 @@ def spec_organizer
   Mrss::SpecOrganizer.new(
     root: ROOT,
     classifiers: CLASSIFIERS,
-    priority_order: RUN_PRIORITY,
+    priority_order: RUN_PRIORITY
   )
 end
 
@@ -167,31 +164,31 @@ task :ci do
   spec_organizer.run
 end
 
-task :bucket, %i(buckets) do |task, args|
+task :bucket, %i[buckets] do |_task, args|
   buckets = args[:buckets]
   buckets = if buckets.nil? || buckets.empty?
-    [nil]
-  else
-    buckets.split(':').map do |bucket|
-      if bucket.empty?
-        nil
-      else
-        bucket.to_sym
-      end
-    end
-  end
+              [ nil ]
+            else
+              buckets.split(':').map do |bucket|
+                if bucket.empty?
+                  nil
+                else
+                  bucket.to_sym
+                end
+              end
+            end
   spec_organizer.run_buckets(*buckets)
 end
 
-task :default => :spec
+task default: :spec
 
-desc "Generate all documentation"
-task :docs => 'docs:yard'
+desc 'Generate all documentation'
+task docs: 'docs:yard'
 
 namespace :docs do
-  desc "Generate yard documentation"
+  desc 'Generate yard documentation'
   task :yard do
-    require "mongoid/version"
+    require 'mongoid/version'
 
     out = File.join('yard-docs', Mongoid::VERSION)
     FileUtils.rm_rf(out)

data/lib/config/locales/en.yml

@@ -712,6 +712,17 @@ en:
             the expression %{javascript} is not allowed."
           resolution: "Please provide a standard hash to #where when the criteria
             is for an embedded association."
+        unsupported_isolation_level:
+          message: "The isolation level '%{level}' is not supported."
+          summary: >
+            You requested an isolation level of '%{level}', which is not
+            supported. Only `:rails`, `:thread` and `:fiber` isolation
+            levels are currently supported; note that the `:fiber` level is
+            only supported on Ruby versions 3.2 and higher.
+          resolution: >
+            Use `:thread` as the isolation level. If you are using Ruby 3.2
+            or higher, you may also use `:fiber`. If using Rails 7+, you
+            may also use `:rails` to inherit the isolation level from Rails.
         validations:
           message: "Validation of %{document} failed."
           summary: "The following errors were found: %{errors}"

data/lib/mongoid/association/accessors.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Association
-
     # This module contains all the behavior related to accessing associations
     # through the getters and setters, and how to delegate to builders to
     # create new ones.
@@ -45,10 +43,8 @@ module Mongoid
         key = @attributes[association.inverse_type]
         type = key ? association.resolver.model_for(key) : nil
         target = if t = association.build(self, object, type, selected_fields)
-          association.create_relation(self, t)
-        else
-          nil
-        end
+                   association.create_relation(self, t)
+                 end
 
         # Only need to do this on embedded associations. The pending callbacks
         # are only added when materializing the documents, which only happens
@@ -71,9 +67,9 @@ module Mongoid
       #
       # @param [ Symbol ] name The name of the association.
       def reset_relation_criteria(name)
-        if instance_variable_defined?("@_#{name}")
-          send(name).reset_unloaded
-        end
+        return unless instance_variable_defined?("@_#{name}")
+
+        send(name).reset_unloaded
       end
 
       # Set the supplied association to an instance variable on the class with the
@@ -119,8 +115,8 @@ module Mongoid
         if !without_autobuild? && association.embedded? && attribute_missing?(field_name)
           # We always allow accessing the parent document of an embedded one.
           try_get_parent = association.is_a?(
-                             Mongoid::Association::Embedded::EmbeddedIn
-                           ) && field_name == association.key
+            Mongoid::Association::Embedded::EmbeddedIn
+          ) && field_name == association.key
           raise Mongoid::Errors::AttributeNotLoaded.new(self.class, field_name) unless try_get_parent
         end
 
@@ -131,6 +127,10 @@ module Mongoid
             _loading do
               if object && needs_no_database_query?(object, association)
                 __build__(name, object, association)
+              # Check if data was loaded via $lookup aggregation
+              elsif !association.embedded? && attributes.key?(name.to_s)
+                # Use the pre-loaded association data from $lookup
+                __build__(name, attributes[name.to_s], association)
               else
                 selected_fields = _mongoid_filter_selected_fields(association.key)
                 __build__(name, attributes[association.key], association, selected_fields)
@@ -162,8 +162,7 @@ module Mongoid
         # and the provided list does not include the association, any of its
         # fields should be allowed.
         if __selected_fields.values.all? { |v| v == 0 } &&
-          __selected_fields.keys.none? { |k| k.split('.', 2).first == assoc_key }
-        then
+           __selected_fields.keys.none? { |k| k.split('.', 2).first == assoc_key }
           return nil
         end
 
@@ -214,16 +213,14 @@ module Mongoid
         # document that the $ is referring to should be retrieved with all
         # fields. See https://www.mongodb.com/docs/manual/reference/operator/projection/positional/
         # and https://jira.mongodb.org/browse/MONGOID-4769.
-        if filtered.keys == %w($)
-          filtered = nil
-        end
+        filtered = nil if filtered.keys == %w[$]
 
         filtered
       end
 
       def needs_no_database_query?(object, association)
         object.is_a?(Document) && !object.embedded? &&
-            object._id == attributes[association.key]
+          object[association.try(:primary_key) || :_id] == attributes[association.key]
       end
 
       # Is the current code executing without autobuild functionality?
@@ -245,10 +242,10 @@ module Mongoid
       #
       # @return [ Object ] The result of the yield.
       def without_autobuild
-        Threaded.begin_execution("without_autobuild")
+        Threaded.begin_execution('without_autobuild')
         yield
       ensure
-        Threaded.exit_execution("without_autobuild")
+        Threaded.exit_execution('without_autobuild')
       end
 
       # Parse out the attributes and the options from the args passed to a
@@ -261,7 +258,7 @@ module Mongoid
       #
       # @return [ Array<Hash> ] The attributes and options.
       def parse_args(*args)
-        [args.first || {}, args.size > 1 ? args[1] : {}]
+        [ args.first || {}, (args.size > 1) ? args[1] : {} ]
       end
 
       # Adds the existence check for associations.
@@ -304,9 +301,7 @@ module Mongoid
         association.inverse_class.tap do |klass|
           klass.re_define_method(name) do |reload = false|
             value = get_relation(name, association, nil, reload)
-            if value.nil? && association.autobuilding? && !without_autobuild?
-              value = send("build_#{name}")
-            end
+            value = send("build_#{name}") if value.nil? && association.autobuilding? && !without_autobuild?
             value
           end
         end
@@ -347,9 +342,7 @@ module Mongoid
           klass.re_define_method("#{name}=") do |object|
             without_autobuild do
               if value = get_relation(name, association, object)
-                if !value.respond_to?(:substitute)
-                  value = __build__(name, value, association) 
-                end
+                value = __build__(name, value, association) unless value.respond_to?(:substitute)
 
                 set_relation(name, value.substitute(object.substitutable))
               else

data/lib/mongoid/association/bindable.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Association
-
     # Superclass for all objects that bind associations together.
     module Bindable
       include Threaded::Lifecycle
@@ -31,10 +29,10 @@ module Mongoid
       #
       # @return [ Object ] The result of the yield.
       def binding
-        unless _binding?
-          _binding do
-            yield(self) if block_given?
-          end
+        return if _binding?
+
+        _binding do
+          yield(self) if block_given?
         end
       end
 
@@ -51,26 +49,26 @@ module Mongoid
       #
       # @raise [ Errors::InverseNotFound ] If no inverse found.
       def check_inverse!(doc)
-        unless _association.bindable?(doc)
-          raise Errors::InverseNotFound.new(
-              _base.class,
-              _association.name,
-              doc.class,
-              _association.foreign_key
-          )
-        end
+        return if _association.bindable?(doc)
+
+        raise Errors::InverseNotFound.new(
+          _base.class,
+          _association.name,
+          doc.class,
+          _association.foreign_key
+        )
       end
 
       # Remove the associated document from the inverse's association.
       #
       # @param [ Document ] doc The document to remove.
       def remove_associated(doc)
-        if inverse = _association.inverse(doc)
-          if _association.many?
-            remove_associated_many(doc, inverse)
-          elsif _association.in_to?
-            remove_associated_in_to(doc, inverse)
-          end
+        return unless inverse = _association.inverse(doc)
+
+        if _association.many?
+          remove_associated_many(doc, inverse)
+        elsif _association.in_to?
+          remove_associated_in_to(doc, inverse)
         end
       end
 
@@ -83,13 +81,13 @@ module Mongoid
       def remove_associated_many(doc, inverse)
         # We only want to remove the inverse association when the inverse
         # document is in memory.
-        if inv = doc.ivar(inverse)
-          # This first condition is needed because when assigning the
-          # embeds_many association using the same embeds_many
-          # association, we delete from the array we are about to assign.
-          if _base != inv && (associated = inv.ivar(_association.name))
-            associated.delete(doc)
-          end
+        return unless inv = doc.ivar(inverse)
+
+        # This first condition is needed because when assigning the
+        # embeds_many association using the same embeds_many
+        # association, we delete from the array we are about to assign.
+        if _base != inv && (associated = inv.ivar(_association.name))
+          associated.delete(doc)
         end
       end
 
@@ -103,9 +101,9 @@ module Mongoid
       def remove_associated_in_to(doc, inverse)
         # We only want to remove the inverse association when the inverse
         # document is in memory.
-        if associated = doc.ivar(inverse)
-          associated.send(_association.setter, nil)
-        end
+        return unless associated = doc.ivar(inverse)
+
+        associated.send(_association.setter, nil)
       end
 
       # Set the id of the related document in the foreign key field on the
@@ -119,9 +117,9 @@ module Mongoid
       # @param [ Document ] keyed The document that stores the foreign key.
       # @param [ Object ] id The id of the bound document.
       def bind_foreign_key(keyed, id)
-        unless keyed.frozen?
-          try_method(keyed, _association.foreign_key_setter, id)
-        end
+        return if keyed.frozen?
+
+        try_method(keyed, _association.foreign_key_setter, id)
       end
 
       # Set the type of the related document on the foreign type field, used
@@ -135,9 +133,9 @@ module Mongoid
       # @param [ Document ] typed The document that stores the type field.
       # @param [ String ] name The name of the model.
       def bind_polymorphic_type(typed, name)
-        if _association.type && !typed.frozen?
-          try_method(typed, _association.type_setter, name)
-        end
+        return unless _association.type && !typed.frozen?
+
+        try_method(typed, _association.type_setter, name)
       end
 
       # Set the type of the related document on the foreign type field, used
@@ -151,9 +149,9 @@ module Mongoid
       # @param [ Document ] typed The document that stores the type field.
       # @param [ String ] name The name of the model.
       def bind_polymorphic_inverse_type(typed, name)
-        if _association.inverse_type && !typed.frozen?
-          try_method(typed, _association.inverse_type_setter, name)
-        end
+        return unless _association.inverse_type && !typed.frozen?
+
+        try_method(typed, _association.inverse_type_setter, name)
       end
 
       # Bind the inverse document to the child document so that the in memory
@@ -167,9 +165,9 @@ module Mongoid
       # @param [ Document ] doc The base document.
       # @param [ Document ] inverse The inverse document.
       def bind_inverse(doc, inverse)
-        if doc.respond_to?(_association.inverse_setter) && !doc.frozen?
-          try_method(doc, _association.inverse_setter, inverse)
-        end
+        return unless doc.respond_to?(_association.inverse_setter) && !doc.frozen?
+
+        try_method(doc, _association.inverse_setter, inverse)
       end
 
       # Bind the provided document with the base from the parent association.
@@ -204,9 +202,9 @@ module Mongoid
       # @return [ true | false ] If the association changed.
       def set_base_association
         inverse_association = _association.inverse_association(_target)
-        if inverse_association != _association && !inverse_association.nil?
-          _base._association = inverse_association
-        end
+        return unless inverse_association != _association && !inverse_association.nil?
+
+        _base._association = inverse_association
       end
 
       # Bind the provided document with the base from the parent association.

data/lib/mongoid/association/builders.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Association
-
     # This module is responsible for defining the build and create methods used
     # in one to one associations.
     #
@@ -22,17 +20,20 @@ module Mongoid
 
       private
 
-      # Parse out the attributes and the options from the args passed to a
-      # build_ or create_ methods.
-      #
-      # @example Parse the args.
-      #   doc.parse_args(:name => "Joe")
+      # Parses the arguments passed to the build and create methods. The first
+      # argument is always the attributes (defaulting to {} if not given). The
+      # last argument is always the options hash, if the last argument is a
+      # hash. The first of any remaining arguments is the type.
       #
-      # @param [ Hash... ] *args The arguments.
+      # @param [ Array ] args The arguments passed to the method.
       #
-      # @return [ Array<Hash> ] The attributes and options.
-      def parse_args(*args)
-        [ args.first || {}, args.size > 1 ? args[1] : {} ]
+      # @return [ Array ] An array containing the attributes, type, and options.
+      def parse_args(args)
+        attributes = args.shift || {}
+        opts = args.last.is_a?(Hash) ? args.pop : {}
+        type = args.shift
+
+        [ attributes, type, opts ]
       end
 
       # Defines a builder method. This is defined as #build_name.
@@ -46,8 +47,9 @@ module Mongoid
       def self.define_builder!(association)
         association.inverse_class.tap do |klass|
           klass.re_define_method("build_#{association.name}") do |*args|
-            attributes, _options = parse_args(*args)
-            document = Factory.execute_build(association.relation_class, attributes, execute_callbacks: false)
+            attributes, type, _opts = parse_args(args)
+
+            document = Factory.execute_build(type || association.relation_class, attributes, execute_callbacks: false)
             _building do
               child = send("#{association.name}=", document)
               child.run_pending_callbacks
@@ -70,8 +72,9 @@ module Mongoid
       def self.define_creator!(association)
         association.inverse_class.tap do |klass|
           klass.re_define_method("create_#{association.name}") do |*args|
-            attributes, _options = parse_args(*args)
-            document = Factory.execute_build(association.relation_class, attributes, execute_callbacks: false)
+            attributes, type, _opts = parse_args(args)
+
+            document = Factory.execute_build(type || association.relation_class, attributes, execute_callbacks: false)
             doc = _assigning do
               send("#{association.name}=", document)
             end

data/lib/mongoid/association/constrainable.rb

@@ -1,13 +1,10 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Association
-
     # Used for converting foreign key values to the correct type based on the
     # types of ids that the document stores.
     module Constrainable
-
       # Convert the supplied object to the appropriate type to set as the
       # foreign key for an association.
       #
@@ -19,11 +16,12 @@ module Mongoid
       # @return [ Object ] The object cast to the correct type.
       def convert_to_foreign_key(object)
         return convert_polymorphic(object) if polymorphic?
-        field = relation_class.fields["_id"]
+
+        field = relation_class.fields['_id']
         if relation_class.using_object_ids?
           BSON::ObjectId.mongoize(object)
         elsif object.is_a?(::Array)
-          object.map!{ |obj| field.mongoize(obj) }
+          object.map! { |obj| field.mongoize(obj) }
         else
           field.mongoize(object)
         end