mongoid 3.1.7 → 4.0.0.alpha1

data/lib/mongoid/relations/embedded/batchable.rb

@@ -6,7 +6,7 @@ module Mongoid
       # Contains behaviour for executing operations in batch on embedded
       # documents.
       module Batchable
-        include Mongoid::Atomic::Positionable
+        include Positional
 
         # Insert new documents as a batch push ($pushAll). This ensures that
         # all callbacks are run at the appropriate time and only 1 request is
@@ -200,7 +200,7 @@ module Mongoid
             docs.map do |doc|
               attributes = { metadata: metadata, _parent: base }
               attributes.merge!(doc)
-              Factory.build(klass, attributes, base.send(:mass_assignment_options))
+              Factory.build(klass, attributes)
             end
           else
             docs
@@ -296,7 +296,7 @@ module Mongoid
           docs.map do |doc|
             self.path = doc.atomic_path unless path
             execute_callback :before_remove, doc
-            if !_assigning? && !metadata.versioned?
+            unless _assigning?
               doc.cascade!
               doc.run_before_callbacks(:destroy) if method == :destroy
             end
@@ -346,7 +346,6 @@ module Mongoid
             doc.run_after_callbacks(:destroy) if method == :destroy
             doc.freeze
             doc.destroyed = true
-            IdentityMap.remove(doc)
           end
         end
       end

data/lib/mongoid/relations/embedded/many.rb

@@ -71,7 +71,6 @@ module Mongoid
         #
         # @overload build(attributes = {}, options = {}, type = nil)
         #   @param [ Hash ] attributes The attributes to build the document with.
-        #   @param [ Hash ] options The scoped assignment options.
         #   @param [ Class ] type Optional class to build the document with.
         #
         # @overload build(attributes = {}, type = nil)
@@ -79,11 +78,8 @@ module Mongoid
         #   @param [ Class ] type Optional class to build the document with.
         #
         # @return [ Document ] The new document.
-        def build(attributes = {}, options = {}, type = nil)
-          if options.is_a?(Class)
-            options, type = {}, options
-          end
-          doc = Factory.build(type || metadata.klass, attributes, options)
+        def build(attributes = {}, type = nil)
+          doc = Factory.build(type || metadata.klass, attributes)
           append(doc)
           doc.apply_post_processed_defaults
           yield(doc) if block_given?
@@ -133,13 +129,9 @@ module Mongoid
           execute_callback :before_remove, document
           doc = target.delete_one(document)
           if doc && !_binding?
-            _unscoped.delete_one(doc) unless doc.paranoid?
+            _unscoped.delete_one(doc)
             if _assigning?
-              if doc.paranoid?
-                doc.destroy(suppress: true)
-              else
-                base.add_atomic_pull(doc)
-              end
+              base.add_atomic_pull(doc)
             else
               doc.delete(suppress: true)
               unbind_one(doc)
@@ -150,18 +142,6 @@ module Mongoid
           doc
         end
 
-        # For use only with Mongoid::Paranoia - will be removed in 4.0.
-        #
-        # @example Get the deleted documents from the relation.
-        #   person.paranoid_phones.deleted
-        #
-        # @return [ Criteria ] The deleted documents.
-        #
-        # @since 3.0.10
-        def deleted
-          unscoped.deleted
-        end
-
         # Delete all the documents in the association without running callbacks.
         #
         # @example Delete all documents from the relation.
@@ -228,10 +208,10 @@ module Mongoid
         # methods.
         #
         # @example Find a document by its id.
-        #   person.addresses.find(Moped::BSON::ObjectId.new)
+        #   person.addresses.find(BSON::ObjectId.new)
         #
         # @example Find documents for multiple ids.
-        #   person.addresses.find([ Moped::BSON::ObjectId.new, Moped::BSON::ObjectId.new ])
+        #   person.addresses.find([ BSON::ObjectId.new, BSON::ObjectId.new ])
         #
         # @param [ Array<Object> ] args Various arguments.
         #
@@ -635,7 +615,7 @@ module Mongoid
           # @since 2.1.0
           def valid_options
             [
-              :as, :cascade_callbacks, :cyclic, :order, :versioned, :store_as,
+              :as, :cascade_callbacks, :cyclic, :order, :store_as,
               :before_add, :after_add, :before_remove, :after_remove
             ]
           end

data/lib/mongoid/relations/macros.rb

@@ -51,9 +51,6 @@ module Mongoid
         # @param [ Hash ] options The relation options.
         # @param [ Proc ] block Optional block for defining extensions.
         def embedded_in(name, options = {}, &block)
-          if ancestors.include?(Mongoid::Versioning)
-            raise Errors::VersioningNotOnRoot.new(self)
-          end
           meta = characterize(name, Embedded::In, options, &block)
           self.embedded = true
           relate(name, meta)
@@ -306,6 +303,7 @@ module Mongoid
               key,
               type: type,
               identity: true,
+              overwrite: true,
               metadata: metadata,
               default: metadata.foreign_key_default
             )
@@ -351,7 +349,7 @@ module Mongoid
         # @param [ Metadata ] metadata The metadata for the relation.
         def relate(name, metadata)
           self.relations = relations.merge(name.to_s => metadata)
-          getter(name, metadata).setter(name, metadata).existence_check(name, metadata)
+          getter(name, metadata).setter(name, metadata).existence_check(name)
         end
       end
     end

data/lib/mongoid/relations/many.rb

@@ -21,11 +21,11 @@ module Mongoid
         size == 0
       end
 
-        # Creates a new document on the references many relation. This will
-        # save the document if the parent has been persisted.
-        #
-        # @example Create and save the new document.
-        #   person.posts.create(:text => "Testing")
+      # Creates a new document on the references many relation. This will
+      # save the document if the parent has been persisted.
+      #
+      # @example Create and save the new document.
+      #   person.posts.create(:text => "Testing")
       #
       # @overload create(attributes = nil, options = {}, type = nil)
       #   @param [ Hash ] attributes The attributes to create with.
@@ -39,10 +39,14 @@ module Mongoid
       # @return [ Document ] The newly created document.
       #
       # @since 2.0.0.beta.1
-      def create(attributes = nil, options = {}, type = nil, &block)
-        doc = build(attributes, options, type, &block)
-        base.persisted? ? doc.save : raise_unsaved(doc)
-        doc
+      def create(attributes = nil, type = nil, &block)
+        if attributes.is_a?(::Array)
+          attributes.map { |attrs| create(attrs, type, &block) }
+        else
+          doc = build(attributes, type, &block)
+          base.persisted? ? doc.save : raise_unsaved(doc)
+          doc
+        end
       end
 
       # Creates a new document on the references many relation. This will
@@ -54,7 +58,6 @@ module Mongoid
       #
       # @overload create!(attributes = nil, options = {}, type = nil)
       #   @param [ Hash ] attributes The attributes to create with.
-      #   @param [ Hash ] options The scoped assignment options.
       #   @param [ Class ] type The optional type of document to create.
       #
       # @overload create!(attributes = nil, type = nil)
@@ -66,10 +69,14 @@ module Mongoid
       # @return [ Document ] The newly created document.
       #
       # @since 2.0.0.beta.1
-      def create!(attributes = nil, options = {}, type = nil, &block)
-        doc = build(attributes, options, type, &block)
-        base.persisted? ? doc.save! : raise_unsaved(doc)
-        doc
+      def create!(attributes = nil, type = nil, &block)
+        if attributes.is_a?(::Array)
+          attributes.map { |attrs| create!(attrs, type, &block) }
+        else
+          doc = build(attributes, type, &block)
+          base.persisted? ? doc.save! : raise_unsaved(doc)
+          doc
+        end
       end
 
       # Find the first document given the conditions, or creates a new document
@@ -78,9 +85,8 @@ module Mongoid
       # @example Find or create.
       #   person.posts.find_or_create_by(:title => "Testing")
       #
-      # @overload find_or_create_by(attributes = nil, options = {}, type = nil)
+      # @overload find_or_create_by(attributes = nil, type = nil)
       #   @param [ Hash ] attributes The attributes to search or create with.
-      #   @param [ Hash ] options The scoped assignment options.
       #   @param [ Class ] type The optional type of document to create.
       #
       # @overload find_or_create_by(attributes = nil, type = nil)
@@ -88,8 +94,8 @@ module Mongoid
       #   @param [ Class ] type The optional type of document to create.
       #
       # @return [ Document ] An existing document or newly created one.
-      def find_or_create_by(attrs = {}, options = {}, type = nil, &block)
-        find_or(:create, attrs, options, type, &block)
+      def find_or_create_by(attrs = {}, type = nil, &block)
+        find_or(:create, attrs, type, &block)
       end
 
       # Find the first +Document+ given the conditions, or instantiates a new document
@@ -98,9 +104,8 @@ module Mongoid
       # @example Find or initialize.
       #   person.posts.find_or_initialize_by(:title => "Test")
       #
-      # @overload find_or_initialize_by(attributes = {}, options = {}, type = nil)
+      # @overload find_or_initialize_by(attributes = {}, type = nil)
       #   @param [ Hash ] attributes The attributes to search or initialize with.
-      #   @param [ Hash ] options The scoped assignment options.
       #   @param [ Class ] type The optional subclass to build.
       #
       # @overload find_or_initialize_by(attributes = {}, type = nil)
@@ -108,8 +113,8 @@ module Mongoid
       #   @param [ Class ] type The optional subclass to build.
       #
       # @return [ Document ] An existing document or newly instantiated one.
-      def find_or_initialize_by(attrs = {}, options = {}, type = nil, &block)
-        find_or(:build, attrs, options, type, &block)
+      def find_or_initialize_by(attrs = {}, type = nil, &block)
+        find_or(:build, attrs, type, &block)
       end
 
       # This proxy can never be nil.
@@ -191,10 +196,9 @@ module Mongoid
       # @example Find or create|initialize.
       #   person.addresses.find_or(:create, :street => "Bond")
       #
-      # @overload find_or(method, attributes = {}, options = {}, type = nil)
+      # @overload find_or(method, attributes = {}, type = nil)
       #   @param [ Symbol ] method The method name, create or new.
       #   @param [ Hash ] attributes The attributes to search or build with.
-      #   @param [ Hash ] options The scoped assignment options.
       #   @param [ Class ] type The optional subclass to build.
       #
       # @overload find_or(attributes = {}, type = nil)
@@ -203,14 +207,9 @@ module Mongoid
       #   @param [ Class ] type The optional subclass to build.
       #
       # @return [ Document ] A matching document or a new/created one.
-      def find_or(method, attrs = {}, options = {}, type = nil, &block)
-        if options.is_a? Class
-          options, type = {}, options
-        end
-
+      def find_or(method, attrs = {}, type = nil, &block)
         attrs["_type"] = type.to_s if type
-
-        where(attrs).first || send(method, attrs, options, type, &block)
+        where(attrs).first || send(method, attrs, type, &block)
       end
     end
   end

data/lib/mongoid/relations/metadata.rb

@@ -229,20 +229,6 @@ module Mongoid
         !!dependent
       end
 
-      # Get the criteria needed to eager load this relation.
-      #
-      # @example Get the eager loading criteria.
-      #   metadata.eager_load(criteria)
-      #
-      # @param [ Array<Object> ] ids The ids of the returned parents.
-      #
-      # @return [ Criteria ] The eager loading criteria.
-      #
-      # @since 2.2.0
-      def eager_load(ids)
-        relation.eager_load(self, ids)
-      end
-
       # Will determine if the relation is an embedded one or not. Currently
       # only checks against embeds one and many.
       #
@@ -398,8 +384,7 @@ module Mongoid
   order:        #{order.inspect}
   polymorphic:  #{polymorphic?}
   relation:     #{relation}
-  setter:       #{setter}
-  versioned:    #{versioned?}>
+  setter:       #{setter}>
 }
       end
 
@@ -460,21 +445,6 @@ module Mongoid
         !!inverse_class_name
       end
 
-      # Is the inverse field bindable? Ie, do we have more than one definition
-      # on the parent class with the same polymorphic name (as).
-      #
-      # @example Is the inverse of bindable?
-      #   metadata.inverse_of_bindable?
-      #
-      # @return [ true, false ] If the relation needs the inverse of field set.
-      #
-      # @since 3.0.6
-      def inverse_field_bindable?
-        @inverse_field_bindable ||= (inverse_klass.relations.values.count do |meta|
-          meta.as == as
-        end > 1)
-      end
-
       # Used for relational many to many only. This determines the name of the
       # foreign key field on the inverse side of the relation, since in this
       # case there are keys on both sides.
@@ -579,31 +549,6 @@ module Mongoid
         @inverse_type_setter ||= inverse_type.__setter__
       end
 
-      # Returns the name of the field in which to store the name of the inverse
-      # field for the polymorphic relation.
-      #
-      # @example Get the name of the field.
-      #   metadata.inverse_of_field
-      #
-      # @return [ String ] The name of the field for storing the name of the
-      #   inverse field.
-      #
-      # @since 2.4.5
-      def inverse_of_field
-        @inverse_of_field ||= determine_inverse_for(:field)
-      end
-
-      # Gets the setter for the field that stores the name of the inverse field
-      # on a polymorphic relation.
-      #
-      # @example Get the inverse type setter.
-      #   metadata.inverse_of_field_setter
-      #
-      # @return [ String ] The name of the setter.
-      def inverse_of_field_setter
-        @inverse_of_field_setter ||= inverse_of_field.__setter__
-      end
-
       # This returns the key that is to be used to grab the attributes for the
       # relation or the foreign key or id that a referenced relation will use
       # to query for the object.
@@ -823,18 +768,6 @@ module Mongoid
         end
       end
 
-      # Is this relation using Mongoid's internal versioning system?
-      #
-      # @example Is this relation versioned?
-      #   metadata.versioned?
-      #
-      # @return [ true, false ] If the relation uses Mongoid versioning.
-      #
-      # @since 2.1.0
-      def versioned?
-        !!self[:versioned]
-      end
-
       # Returns the metadata itself. Here for compatibility with Rails
       # association metadata.
       #
@@ -1033,15 +966,13 @@ module Mongoid
       # @since 2.0.0.rc.1
       def determine_foreign_key
         return self[:foreign_key].to_s if self[:foreign_key]
-        suffix = relation.foreign_key_suffix
+        return nil if relation.embedded?
+
         if relation.stores_foreign_key?
           relation.foreign_key(name)
         else
-          if polymorphic?
-            "#{self[:as]}#{suffix}"
-          else
-            inverse_of ? "#{inverse_of}#{suffix}" : inverse_class_name.foreign_key
-          end
+          suffix = relation.foreign_key_suffix
+          "#{inverse}#{suffix}"
         end
       end
 
@@ -1112,7 +1043,7 @@ module Mongoid
       # @since 3.0.0
       def inverse_relation_candidates
         relations_metadata.select do |meta|
-          next if meta.versioned? || meta.name == name
+          next if meta.name == name
           meta.class_name == inverse_class_name
         end
       end

data/lib/mongoid/relations/nested_builder.rb

@@ -63,11 +63,11 @@ module Mongoid
       # @param [ Class ] klass The class we're trying to convert for.
       # @param [ String ] id The id, usually coming from the form.
       #
-      # @return [ Moped::BSON::ObjectId, String, Object ] The converted id.
+      # @return [ BSON::ObjectId, String, Object ] The converted id.
       #
       # @since 2.0.0.rc.6
       def convert_id(klass, id)
-        klass.using_object_ids? ? Moped::BSON::ObjectId.mongoize(id) : id
+        klass.using_object_ids? ? BSON::ObjectId.mongoize(id) : id
       end
     end
   end

data/lib/mongoid/relations/options.rb

@@ -42,6 +42,7 @@ module Mongoid
             )
           end
         end
+        true
       end
     end
   end

data/lib/mongoid/relations/polymorphic.rb

@@ -29,7 +29,6 @@ module Mongoid
             self.polymorphic = true
             if metadata.relation.stores_foreign_key?
               field(metadata.inverse_type, type: String)
-              field(metadata.inverse_of_field, type: Symbol)
             end
           end
           self

data/lib/mongoid/relations/proxy.rb

@@ -12,13 +12,13 @@ module Mongoid
       # We undefine most methods to get them sent through to the target.
       instance_methods.each do |method|
         undef_method(method) unless
-          method =~ /(^__|^send|^object_id|^respond_to|^tap|extend_proxy)/
+          method =~ /(^__|^send|^object_id|^respond_to|^tap|extend_proxy|extend_proxies)/
       end
 
       include Threaded::Lifecycle
       include Marshalable
 
-      attr_accessor :base, :loaded, :metadata, :target
+      attr_accessor :base, :metadata, :target
 
       # Backwards compatibility with Mongoid beta releases.
       delegate :foreign_key, :inverse_foreign_key, to: :metadata
@@ -39,7 +39,12 @@ module Mongoid
       def init(base, target, metadata)
         @base, @target, @metadata = base, target, metadata
         yield(self) if block_given?
-        extend_proxy(metadata.extension) if metadata.extension?
+        extend_proxies(metadata.extension) if metadata.extension?
+      end
+
+      # Allow extension to be an array and extend each module
+      def extend_proxies(*extension)
+        extension.flatten.each {|ext| extend_proxy(ext) }
       end
 
       # Get the class from the metadata, or return nil if no metadata present.
@@ -103,7 +108,7 @@ module Mongoid
       #
       # @since 3.0.0
       def with(options)
-        Threaded.set_persistence_options(klass, options)
+        @persistence_options = options
         self
       end
 
@@ -119,6 +124,7 @@ module Mongoid
       # @since 2.0.0
       def collection
         root = base._root
+        root.with(@persistence_options)
         root.collection unless root.embedded?
       end
 
@@ -230,34 +236,6 @@ module Mongoid
         def apply_ordering(criteria, metadata)
           metadata.order ? criteria.order_by(metadata.order) : criteria
         end
-
-        # Get the criteria that is used to eager load a relation of this
-        # type.
-        #
-        # @example Get the eager load criteria.
-        #   Proxy.eager_load(metadata, criteria)
-        #
-        # @param [ Metadata ] metadata The relation metadata.
-        # @param [ Array<Object> ] ids The ids of the base docs.
-        #
-        # @return [ Criteria ] The criteria to eager load the relation.
-        #
-        # @since 2.2.0
-        def eager_load_ids(metadata, ids)
-          klass, foreign_key = metadata.klass, metadata.foreign_key
-          eager_loaded = klass.any_in(foreign_key => ids).entries
-          ids.each do |id|
-            sel = { foreign_key => id }
-            sel.merge!(metadata.type_relation) if klass.hereditary?
-            IdentityMap.clear_many(klass, sel)
-          end
-          eager_loaded.each do |doc|
-            base_id = doc.__send__(foreign_key)
-            sel = { foreign_key => base_id }
-            sel.merge!(metadata.type_relation) if klass.hereditary?
-            yield(doc,  sel)
-          end
-        end
       end
     end
   end