mongoid 3.1.7 → 4.0.0.alpha1

data/lib/mongoid/relations/referenced/in.rb

@@ -139,24 +139,8 @@ module Mongoid
             type.where(metadata.primary_key => object)
           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 target docs.
-          #
-          # @return [ Criteria ] The criteria to eager load the relation.
-          #
-          # @since 2.2.0
-          def eager_load(metadata, ids)
-            raise Errors::EagerLoad.new(metadata.name) if metadata.polymorphic?
-            klass, _ = metadata.klass, metadata.foreign_key
-            klass.any_in("_id" => ids.uniq).each do |doc|
-              IdentityMap.set(doc)
-            end
+          def eager_load_klass
+            Relations::Eager::BelongsTo
           end
 
           # Returns true if the relation is an embedded one. In this case

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

@@ -78,12 +78,8 @@ module Mongoid
         # @return [ Document ] The new document.
         #
         # @since 2.0.0.beta.1
-        def build(attributes = {}, options = {}, type = nil)
-          if options.is_a? Class
-            options, type = {}, options
-          end
-
-          doc = Factory.build(type || klass, attributes, options)
+        def build(attributes = {}, type = nil)
+          doc = Factory.build(type || klass, attributes)
           append(doc)
           doc.apply_post_processed_defaults
           yield(doc) if block_given?
@@ -186,15 +182,15 @@ module Mongoid
         # conditions.
         #
         # @example Find by an id.
-        #   person.posts.find(Moped::BSON::ObjectId.new)
+        #   person.posts.find(BSON::ObjectId.new)
         #
         # @example Find by multiple ids.
-        #   person.posts.find([ Moped::BSON::ObjectId.new, Moped::BSON::ObjectId.new ])
+        #   person.posts.find([ BSON::ObjectId.new, BSON::ObjectId.new ])
         #
         # @note This will keep matching documents in memory for iteration
         #   later.
         #
-        # @param [ Moped::BSON::ObjectId, Array<Moped::BSON::ObjectId> ] arg The ids.
+        # @param [ BSON::ObjectId, Array<BSON::ObjectId> ] arg The ids.
         #
         # @return [ Document, Criteria ] The matching document(s).
         #
@@ -324,6 +320,8 @@ module Mongoid
         #
         # @since 2.0.0.rc.1
         def append(document)
+          document.with(@persistence_options) if @persistence_options
+
           execute_callback :before_add, document
           target.push(document)
           characterize_one(document)
@@ -571,19 +569,8 @@ module Mongoid
             )
           end
 
-          # Eager load the relation based on the criteria.
-          #
-          # @example Eager load the 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(metadata, ids)
-            eager_load_ids(metadata, ids) { |doc, key| IdentityMap.set_many(doc, key) }
+          def eager_load_klass
+            Relations::Eager::HasMany
           end
 
           # Returns true if the relation is an embedded one. In this case

data/lib/mongoid/relations/referenced/many_to_many.rb

@@ -29,7 +29,7 @@ module Mongoid
           return concat(docs) if docs.size > 1
           if doc = docs.first
             append(doc)
-            base.add_to_set(foreign_key, doc.send(metadata.primary_key))
+            base.add_to_set(foreign_key => doc.send(metadata.primary_key))
             if child_persistable?(doc)
               doc.save
             end
@@ -65,7 +65,7 @@ module Mongoid
             end
           end
           if persistable? || _creating?
-            base.push_all(foreign_key, ids.keys)
+            base.push(foreign_key => ids.keys)
           end
           persist_delayed(docs, inserts)
           self
@@ -77,25 +77,19 @@ module Mongoid
         # @example Build a new document on the relation.
         #   person.posts.build(:title => "A new post")
         #
-        # @overload build(attributes = {}, options = {}, type = nil)
+        # @overload build(attributes = {}, type = nil)
         #   @param [ Hash ] attributes The attributes of the new document.
-        #   @param [ Hash ] options The scoped assignment options.
         #   @param [ Class ] type The optional subclass to build.
         #
         # @overload build(attributes = {}, type = nil)
         #   @param [ Hash ] attributes The attributes of the new document.
-        #   @param [ Hash ] options The scoped assignment options.
         #   @param [ Class ] type The optional subclass to build.
         #
         # @return [ Document ] The new document.
         #
         # @since 2.0.0.beta.1
-        def build(attributes = {}, options = {}, type = nil)
-          if options.is_a? Class
-            options, type = {}, options
-          end
-
-          doc = Factory.build(type || klass, attributes, options)
+        def build(attributes = {}, type = nil)
+          doc = Factory.build(type || klass, attributes)
           base.send(foreign_key).push(doc.id)
           append(doc)
           doc.apply_post_processed_defaults
@@ -120,7 +114,7 @@ module Mongoid
         def delete(document)
           doc = super
           if doc && persistable?
-            base.pull(foreign_key, doc.send(metadata.primary_key))
+            base.pull(foreign_key => doc.send(metadata.primary_key))
             target._unloaded = criteria
             unsynced(base, foreign_key)
           end
@@ -140,13 +134,10 @@ module Mongoid
             execute_callback :before_remove, doc
           end
           unless metadata.forced_nil_inverse?
-            criteria.pull(inverse_foreign_key, base.id)
+            criteria.pull(inverse_foreign_key => base.id)
           end
           if persistable?
-            base.set(
-              foreign_key,
-              base.send(foreign_key).clear
-            )
+            base.set(foreign_key => base.send(foreign_key).clear)
           end
           after_remove_error = nil
           many_to_many = target.clear do |doc|
@@ -320,22 +311,8 @@ module Mongoid
             )
           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 documents to load.
-          #
-          # @return [ Criteria ] The criteria to eager load the relation.
-          #
-          # @since 2.2.0
-          def eager_load(metadata, ids)
-            metadata.klass.any_in(_id: ids).each do |doc|
-              IdentityMap.set(doc)
-            end
+          def eager_load_klass
+            Relations::Eager::HasAndBelongsToMany
           end
 
           # Returns true if the relation is an embedded one. In this case

data/lib/mongoid/relations/referenced/one.rb

@@ -122,33 +122,14 @@ module Mongoid
           # @since 2.1.0
           def criteria(metadata, object, type = nil)
             crit = metadata.klass.where(metadata.foreign_key => object)
-
             if metadata.polymorphic?
               crit = crit.where(metadata.type => type.name)
             end
-
-            inverse_metadata = metadata.inverse_metadata(metadata.klass)
-            if inverse_metadata.inverse_of_field
-              crit = crit.any_in(inverse_metadata.inverse_of_field => [metadata.name, nil])
-            end
-
             crit
           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(metadata, ids)
-            eager_load_ids(metadata, ids) { |doc, key| IdentityMap.set_one(doc, key) }
+          def eager_load_klass
+            Relations::Eager::HasOne
           end
 
           # Returns true if the relation is an embedded one. In this case

data/lib/mongoid/relations/synchronization.rb

@@ -61,7 +61,7 @@ module Mongoid
       def remove_inverse_keys(meta)
         foreign_keys = send(meta.foreign_key)
         unless foreign_keys.nil? || foreign_keys.empty?
-          meta.criteria(foreign_keys, self.class).pull(meta.inverse_foreign_key, id)
+          meta.criteria(foreign_keys, self.class).pull(meta.inverse_foreign_key => id)
         end
       end
 
@@ -92,10 +92,10 @@ module Mongoid
           end
 
           unless adds.empty?
-            meta.criteria(adds, self.class).without_options.add_to_set(meta.inverse_foreign_key, id)
+            meta.criteria(adds, self.class).without_options.add_to_set(meta.inverse_foreign_key => id)
           end
           unless subs.empty?
-            meta.criteria(subs, self.class).without_options.pull(meta.inverse_foreign_key, id)
+            meta.criteria(subs, self.class).without_options.pull(meta.inverse_foreign_key => id)
           end
         end
       end

data/lib/mongoid/relations/touchable.rb

@@ -4,6 +4,39 @@ module Mongoid
     module Touchable
       extend ActiveSupport::Concern
 
+      # Touch the document, in effect updating its updated_at timestamp and
+      # optionally the provided field to the current time. If any belongs_to
+      # relations exist with a touch option, they will be updated as well.
+      #
+      # @example Update the updated_at timestamp.
+      #   document.touch
+      #
+      # @example Update the updated_at and provided timestamps.
+      #   document.touch(:audited)
+      #
+      # @note This will not autobuild relations if those options are set.
+      #
+      # @param [ Symbol ] field The name of an additional field to update.
+      #
+      # @return [ true/false ] false if record is new_record otherwise true.
+      #
+      # @since 3.0.0
+      def touch(field = nil)
+        return false if _root.new_record?
+        current = Time.now
+        field = database_field_name(field)
+        write_attribute(:updated_at, current) if respond_to?("updated_at=")
+        write_attribute(field, current) if field
+
+        touches = touch_atomic_updates(field)
+        unless touches.empty?
+          selector = atomic_selector
+          _root.collection.find(selector).update(positionally(selector, touches))
+        end
+        run_callbacks(:touch)
+        true
+      end
+
       module ClassMethods
 
         # Add the metadata to the touchable relations if the touch option was

data/lib/mongoid/{reloading.rb → reloadable.rb}

@@ -1,7 +1,10 @@
 # encoding: utf-8
 module Mongoid
+
   # This module handles reloading behaviour of documents.
-  module Reloading
+  #
+  # @since 4.0.0
+  module Reloadable
 
     # Reloads the +Document+ attributes from the database. If the document has
     # not been saved then an error will get raised if the configuration option
@@ -25,7 +28,6 @@ module Mongoid
       changed_attributes.clear
       apply_defaults
       reload_relations
-      IdentityMap.set(self)
       run_callbacks(:find) unless _find_callbacks.empty?
       run_callbacks(:initialize) unless _initialize_callbacks.empty?
       self
@@ -55,7 +57,7 @@ module Mongoid
     #
     # @since 2.3.2
     def reload_root_document
-      {}.merge(with(consistency: :strong).collection.find(_id: id).one || {})
+      {}.merge(with(read: :primary).collection.find(_id: id).one || {})
     end
 
     # Reload the embedded document.
@@ -68,7 +70,7 @@ module Mongoid
     # @since 2.3.2
     def reload_embedded_document
       extract_embedded_attributes({}.merge(
-        _root.with(consistency: :strong).collection.find(_id: _root.id).one
+        _root.with(read: :primary).collection.find(_id: _root.id).one
       ))
     end
 

data/lib/mongoid/{scoping.rb → scopable.rb}

@@ -3,7 +3,9 @@ module Mongoid
 
   # This module contains behaviour for all Mongoid scoping - named scopes,
   # default scopes, and criteria accessors via scoped and unscoped.
-  module Scoping
+  #
+  # @since 4.0.0
+  module Scopable
     extend ActiveSupport::Concern
 
     included do
@@ -12,6 +14,27 @@ module Mongoid
       self._declared_scopes = {}
     end
 
+    private
+
+    # Apply the default scoping to the attributes of the document, as long as
+    # they are not complex queries.
+    #
+    # @api private
+    #
+    # @example Apply the default scoping.
+    #   document.apply_default_scoping
+    #
+    # @return [ true, false ] If default scoping was applied.
+    #
+    # @since 4.0.0
+    def apply_default_scoping
+      if default_scoping
+        default_scoping.call.selector.each do |field, value|
+          attributes[field] = value unless value.respond_to?(:each_pair)
+        end
+      end
+    end
+
     module ClassMethods
 
       # Returns a hash of all the scopes defined for this class, including
@@ -298,11 +321,11 @@ module Mongoid
       #
       # @since 3.0.0
       def define_scope_method(name)
-        (class << self; self; end).class_eval <<-SCOPE, __FILE__, __LINE__ + 1
+        singleton_class.class_eval <<-SCOPE, __FILE__, __LINE__ + 1
           def #{name}(*args)
             scoping = _declared_scopes[:#{name}]
             scope, extension = scoping[:scope][*args], scoping[:extension]
-            criteria = with_default_scope.merge(scope || all)
+            criteria = with_default_scope.merge(scope || queryable)
             criteria.extend(extension)
             criteria
           end

data/lib/mongoid/selectable.rb

@@ -0,0 +1,59 @@
+# encoding: utf-8
+module Mongoid
+
+  # Provides behaviour for generating the selector for a specific document.
+  #
+  # @since 4.0.0
+  module Selectable
+    extend ActiveSupport::Concern
+
+    # Get the atomic selector for the document. This is a hash in the simplest
+    # case { "_id" => id }, but can become more complex for embedded documents
+    # and documents that use a shard key.
+    #
+    # @example Get the document's atomic selector.
+    #   document.atomic_selector
+    #
+    # @return [ Hash ] The document's selector.
+    #
+    # @since 1.0.0
+    def atomic_selector
+      @atomic_selector ||=
+        (embedded? ? embedded_atomic_selector : root_atomic_selector)
+    end
+
+    private
+
+    # Get the atomic selector for an embedded document.
+    #
+    # @api private
+    #
+    # @example Get the embedded atomic selector.
+    #   document.embedded_atomic_selector
+    #
+    # @return [ Hash ] The embedded document selector.
+    #
+    # @since 4.0.0
+    def embedded_atomic_selector
+      if persisted? && _id_changed?
+        _parent.atomic_selector
+      else
+        _parent.atomic_selector.merge("#{atomic_path}._id" => _id)
+      end
+    end
+
+    # Get the atomic selector for a root document.
+    #
+    # @api private
+    #
+    # @example Get the root atomic selector.
+    #   document.root_atomic_selector
+    #
+    # @return [ Hash ] The root document selector.
+    #
+    # @since 4.0.0
+    def root_atomic_selector
+      { "_id" => _id }.merge!(shard_key_selector)
+    end
+  end
+end

data/lib/mongoid/{serialization.rb → serializable.rb}

@@ -3,9 +3,18 @@ module Mongoid
 
   # This module provides the extra behaviour for including relations in JSON
   # and XML serialization.
-  module Serialization
+  #
+  # @since 4.0.0
+  module Serializable
     extend ActiveSupport::Concern
 
+    # We need to redefine where the JSON configuration is getting defined,
+    # similar to +ActiveRecord+.
+    included do
+      undef_method :include_root_in_json
+      delegate :include_root_in_json, to: ::Mongoid
+    end
+
     # Gets the document as a serializable hash, used by ActiveModel's JSON
     # serializer.
     #