mongoid 8.0.13 → 8.1.0

data/lib/mongoid/tasks/database.rake

@@ -5,6 +5,11 @@ namespace :db do
     task :load_models do
     end
 
+    desc "Create collections for Mongoid models"
+    task :create_collections => [:environment, :load_models] do
+      ::Mongoid::Tasks::Database.create_collections
+    end
+
     desc "Create indexes specified in Mongoid models"
     task :create_indexes => [:environment, :load_models] do
       ::Mongoid::Tasks::Database.create_indexes
@@ -34,5 +39,12 @@ namespace :db do
     task :purge => :environment do
       ::Mongoid.purge!
     end
+
+    namespace :create_collections do
+      desc "Drop and create collections for Mongoid models"
+      task :force => [:environment, :load_models] do
+        ::Mongoid::Tasks::Database.create_collections(force: true)
+      end
+    end
   end
 end

data/lib/mongoid/tasks/database.rb

@@ -5,6 +5,26 @@ module Mongoid
     module Database
       extend self
 
+      # Create collections for each model given the provided globs and the class is
+      # not embedded.
+      #
+      # @param [ Array<Mongoid::Document> ] models. Array of document classes for
+      #   which collections should be created. Defaulted to all document classes
+      #   in the application.
+      # @param [ true | false ] force If true, the method will drop existing
+      #   collections before creating new ones. If false, the method will create
+      #   only new collection (that do not exist in the database).
+      def create_collections(models = ::Mongoid.models, force: false)
+        models.each do |model|
+          if !model.embedded? || model.cyclic?
+            model.create_collection(force: force)
+            logger.info("MONGOID: Created collection for #{model}:")
+          else
+            logger.info("MONGOID: collection options ignored on: #{model}, please define in the root model.")
+          end
+        end
+      end
+
       # Create indexes for each model given the provided globs and the class is
       # not embedded.
       #

data/lib/mongoid/threaded.rb

@@ -30,9 +30,6 @@ module Mongoid
     # executed on documents.
     EXECUTE_CALLBACKS = '[mongoid]:execute-callbacks'
 
-    # The key for the time left in the current in-memory regexp budget.
-    REGEXP_BUDGET_KEY = 'regexp-budget'
-
     extend self
 
     # Begin entry into a named thread local stack.

data/lib/mongoid/timestamps/created.rb

@@ -22,20 +22,13 @@ module Mongoid
       # @example Set the created at time.
       #   person.set_created_at
       def set_created_at
-        if able_to_set_created_at?
+        if !timeless? && !created_at
           time = Time.configured.now
           self.updated_at = time if is_a?(Updated) && !updated_at_changed?
           self.created_at = time
         end
         clear_timeless_option
       end
-
-      # Is the created timestamp able to be set?
-      #
-      # @return [ true, false ] If the timestamp can be set.
-      def able_to_set_created_at?
-        !frozen? && !timeless? && !created_at
-      end
     end
   end
 end

data/lib/mongoid/traversable.rb

@@ -300,18 +300,6 @@ module Mongoid
         !!(Mongoid::Document > superclass)
       end
 
-      # Returns the root class of the STI tree that the current
-      # class participates in. If the class is not an STI subclass, this
-      # returns the class itself.
-      #
-      # @return [ Mongoid::Document ] the root of the STI tree
-      def root_class
-        root = self
-        root = root.superclass while root.hereditary?
-
-        root
-      end
-
       # When inheriting, we want to copy the fields from the parent class and
       # set the on the child to start, mimicking the behavior of the old
       # class_inheritable_accessor that was deprecated in Rails edge.

data/lib/mongoid/utils.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Mongoid
+
+  # @api private
+  module Utils
+
+    # This function should be used if you need to measure time.
+    # @example Calculate elapsed time.
+    #   starting = Utils.monotonic_time
+    #   # do something time consuming
+    #   ending = Utils.monotonic_time
+    #   puts "It took #{(ending - starting).to_i} seconds"
+    #
+    # @see https://blog.dnsimple.com/2018/03/elapsed-time-with-ruby-the-right-way/
+    #
+    # @return [Float] seconds according to monotonic clock
+    module_function def monotonic_time
+      Process.clock_gettime(Process::CLOCK_MONOTONIC)
+    end
+  end
+end

data/lib/mongoid/validatable/associated.rb

@@ -15,113 +15,32 @@ module Mongoid
     #
     #     validates_associated :name, :addresses
     #   end
-    class AssociatedValidator < ActiveModel::Validator
-      # Required by `validates_with` so that the validator
-      # gets added to the correct attributes.
-      def attributes
-        options[:attributes]
-      end
+    class AssociatedValidator < ActiveModel::EachValidator
 
-      # Checks that the named associations of the given record
-      # (`attributes`) are valid. This does NOT load the associations
-      # from the database, and will only validate records that are dirty
-      # or unpersisted.
+      # Validates that the associations provided are either all nil or all
+      # valid. If neither is true then the appropriate errors will be added to
+      # the parent document.
       #
-      # If anything is not valid, appropriate errors will be added to
-      # the `document` parameter.
-      #
-      # @param [ Mongoid::Document ] document the document with the
-      #   associations to validate.
-      def validate(document)
-        options[:attributes].each do |attr_name|
-          validate_association(document, attr_name)
-        end
-      end
-
-      private
-
-      # Validates that the given association provided is either nil,
-      # persisted and unchanged, or invalid. Otherwise, the appropriate errors
-      # will be added to the parent document.
+      # @example Validate the association.
+      #   validator.validate_each(document, :name, name)
       #
       # @param [ Document ] document The document to validate.
       # @param [ Symbol ] attribute The association to validate.
-      def validate_association(document, attribute)
-        # grab the proxy from the instance variable directly; we don't want
-        # any loading logic to run; we just want to see if it's already
-        # been loaded.
-        proxy = document.ivar(attribute)
-        return unless proxy
-
-        # if the variable exists, now we see if it is a proxy, or an actual
-        # document. It might be a literal document instead of a proxy if this
-        # document was created with a Document instance as a provided attribute,
-        # e.g. "Post.new(message: Message.new)".
-        target = proxy.respond_to?(:_target) ? proxy._target : proxy
-
-        # Now, fetch the list of documents from the target. Target may be a
-        # single value, or a list of values, and in the case of HasMany,
-        # might be a rather complex collection. We need to do this without
-        # triggering a load, so it's a bit of a delicate dance.
-        list = get_target_documents(target)
-
-        valid = document.validating do
-          # Now, treating the target as an array, look at each element
-          # and see if it is valid, but only if it has already been
-          # persisted, or changed, and hasn't been flagged for destroy.
-          #
-          # use map.all? instead of just all?, because all? will do short-circuit
-          # evaluation and terminate on the first failed validation.
-          list.map do |value|
-            if value && !value.flagged_for_destroy? && (!value.persisted? || value.changed?)
-              value.validated? ? true : value.valid?
-            else
+      # @param [ Object ] value The value of the association.
+      def validate_each(document, attribute, value)
+        begin
+          document.begin_validate
+          valid = Array.wrap(value).collect do |doc|
+            if doc.nil? || doc.flagged_for_destroy?
               true
+            else
+              doc.validated? ? true : doc.valid?
             end
           end.all?
+        ensure
+          document.exit_validate
         end
-
-        document.errors.add(attribute, :invalid) unless valid
-      end
-
-      private
-
-      # Examine the given target object and return an array of
-      # documents (possibly empty) that the target represents.
-      #
-      # @param [ Array | Mongoid::Document | Mongoid::Association::Proxy | HasMany::Enumerable ] target
-      #   the target object to examine.
-      #
-      # @return [ Array<Mongoid::Document> ] the list of documents
-      def get_target_documents(target)
-        if target.respond_to?(:_loaded?)
-          get_target_documents_for_has_many(target)
-        else
-          get_target_documents_for_other(target)
-        end
-      end
-
-      # Returns the list of all currently in-memory values held by
-      # the target. The target will not be loaded.
-      #
-      # @param [ HasMany::Enumerable ] target the target that will
-      #   be examined for in-memory documents.
-      #
-      # @return [ Array<Mongoid::Document> ] the in-memory documents
-      #   held by the target.
-      def get_target_documents_for_has_many(target)
-        [ *target._loaded.values, *target._added.values ]
-      end
-
-      # Returns the target as an array. If the target represents a single
-      # value, it is wrapped in an array.
-      #
-      # @param [ Array | Mongoid::Document | Mongoid::Association::Proxy ] target
-      #   the target to return.
-      #
-      # @return [ Array<Mongoid::Document> ] the target, as an array.
-      def get_target_documents_for_other(target)
-        Array.wrap(target)
+        document.errors.add(attribute, :invalid, **options) unless valid
       end
     end
   end

data/lib/mongoid/validatable/macros.rb

@@ -18,7 +18,7 @@ module Mongoid
       #     validates_associated :name, :addresses
       #   end
       #
-      # @param [ Array ] args The arguments to pass to the validator.
+      # @param [ Object... ] *args The arguments to pass to the validator.
       def validates_associated(*args)
         validates_with(AssociatedValidator, _merge_attributes(args))
       end
@@ -35,7 +35,7 @@ module Mongoid
       #     validates_uniqueness_of :title
       #   end
       #
-      # @param [ Array ] args The arguments to pass to the validator.
+      # @param [ Object... ] *args The arguments to pass to the validator.
       def validates_uniqueness_of(*args)
         validates_with(UniquenessValidator, _merge_attributes(args))
       end
@@ -50,7 +50,7 @@ module Mongoid
       #     validates_format_of :title, with: /\A[a-z0-9 \-_]*\z/i
       #   end
       #
-      # @param [ Array ] args The names of the fields to validate.
+      # @param [ Object... ] *args The names of the field(s) to validate.
       def validates_format_of(*args)
         validates_with(FormatValidator, _merge_attributes(args))
       end
@@ -65,7 +65,7 @@ module Mongoid
       #     validates_length_of :title, minimum: 100
       #   end
       #
-      # @param [ Array ] args The names of the fields to validate.
+      # @param [ Object... ] *args The names of the field(s) to validate.
       def validates_length_of(*args)
         validates_with(LengthValidator, _merge_attributes(args))
       end
@@ -80,25 +80,10 @@ module Mongoid
       #     validates_presence_of :title
       #   end
       #
-      # @param [ Array ] args The names of the fields to validate.
+      # @param [ Object... ] *args The names of the field(s) to validate.
       def validates_presence_of(*args)
         validates_with(PresenceValidator, _merge_attributes(args))
       end
-
-      # Validates whether or not a field contains a numeric value.
-      #
-      # @example
-      #   class Person
-      #     include Mongoid::Document
-      #     field :cost
-      #
-      #     validates_numericality_of :cost
-      #   end
-      #
-      # @param [ Object... ] *args The names of the field(s) to validate.
-      def validates_numericality_of(*args)
-        validates_with(NumericalityValidator, _merge_attributes(args))
-      end
     end
   end
 end

data/lib/mongoid/validatable.rb

@@ -5,7 +5,6 @@ require "mongoid/validatable/localizable"
 require "mongoid/validatable/associated"
 require "mongoid/validatable/format"
 require "mongoid/validatable/length"
-require "mongoid/validatable/numericality"
 require "mongoid/validatable/queryable"
 require "mongoid/validatable/presence"
 require "mongoid/validatable/uniqueness"
@@ -38,14 +37,6 @@ module Mongoid
       Threaded.exit_validate(self)
     end
 
-    # Perform a validation within the associated block.
-    def validating
-      begin_validate
-      yield
-    ensure
-      exit_validate
-    end
-
     # Given the provided options, are we performing validations?
     #
     # @example Are we performing validations?
@@ -53,6 +44,8 @@ module Mongoid
     #
     # @param [ Hash ] options The options to check.
     #
+    # @option options [ true | false ] :validate Whether or not to validate.
+    #
     # @return [ true | false ] If we are validating.
     def performing_validations?(options = {})
       options[:validate].nil? ? true : options[:validate]
@@ -138,7 +131,8 @@ module Mongoid
       # @example Validate with a specific validator.
       #   validates_with MyValidator, on: :create
       #
-      # @param [ Class<Array> | Hash ] args The validator classes and options.
+      # @param [ ActiveModel::Validator..., Hash ] *args The validator classes
+      #   and options hash.
       #
       # @note See ActiveModel::Validations::With for full options. This is
       #   overridden to add autosave functionality when presence validation is

data/lib/mongoid/version.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
 module Mongoid
-  # The current version of Mongoid
-  #
-  # Note that this file is automatically updated via `rake candidate:create`.
-  # Manual changes to this file will be overwritten by that rake task.
-  VERSION = '8.0.13'
+  VERSION = "8.1.0"
 end

data/lib/mongoid/warnings.rb

@@ -23,7 +23,22 @@ module Mongoid
 
     warning :geo_haystack_deprecated, 'The geoHaystack type is deprecated.'
     warning :as_json_compact_deprecated, '#as_json :compact option is deprecated. Please call #compact on the returned Hash object instead.'
-    warning :symbol_type_deprecated, 'The BSON Symbol type is deprecated by MongoDB. Please use String or StringifiedSymbol field types instead of the Symbol field type'
-    warning :reparenting_via_nested_attributes, 'Reparenting documents via nested attributes is insecure and is deprecated. Set Mongoid.allow_reparenting_via_nested_attributes to false and update your code to avoid reparenting documents via nested attributes.'
+    warning :symbol_type_deprecated, 'The BSON Symbol type is deprecated by MongoDB. Please use String or StringifiedSymbol field types instead of the Symbol field type.'
+    warning :legacy_readonly, 'The readonly! method will only mark the document readonly when the legacy_readonly feature flag is switched off.'
+    warning :use_activesupport_time_zone_deprecated, 'Config option :use_activesupport_time_zone is deprecated and should be removed from your config. It will always be true beginning in Mongoid 9.0.'
+    warning :broken_aggregables_deprecated, 'Config option :broken_aggregables is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :broken_alias_handling_deprecated, 'Config option :broken_alias_handling is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :broken_and_deprecated, 'Config option :broken_and is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :broken_scoping_deprecated, 'Config option :broken_scoping is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :broken_updates_deprecated, 'Config option :broken_updates is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :compare_time_by_ms_deprecated, 'Config option :compare_time_by_ms is deprecated. It will always be true beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :legacy_attributes_deprecated, 'Config option :legacy_attributes is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :legacy_pluck_distinct_deprecated, 'Config option :legacy_pluck_distinct is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :legacy_triple_equals_deprecated, 'Config option :legacy_triple_equals is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :object_id_as_json_oid_deprecated, 'Config option :object_id_as_json_oid is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :overwrite_chained_operators_deprecated, 'Config option :overwrite_chained_operators is deprecated. It will always be false beginning in Mongoid 9.0. Please use load_defaults for Mongoid 8.0 or later, then remove it from your config.'
+    warning :mutable_ids, 'In Mongoid 9.0 the _id field will be immutable. In earlier versions of 8.x, mutating the _id field was supported inconsistently. Prepare your code for 9.0 by setting Mongoid::Config.immutable_ids to true.'
+    warning :mongoid_query_cache, 'In Mongoid 9.0, Mongoid::QueryCache will be removed. Please replace it with Mongo::QueryCache.'
+    warning :mongoid_query_cache_clear, 'In Mongoid 9.0, Mongoid::QueryCache#clear_cache should be replaced it with Mongo::QueryCache#clear.'
   end
 end

data/lib/mongoid.rb

@@ -12,8 +12,10 @@ require "active_support/inflector"
 require "active_support/time_with_zone"
 require "active_model"
 
+require 'concurrent-ruby'
+
 require "mongo"
-require 'mongo/active_support'
+require "mongo/active_support"
 
 require "mongoid/version"
 require "mongoid/deprecable"
@@ -25,6 +27,7 @@ require "mongoid/document"
 require "mongoid/tasks/database"
 require "mongoid/query_cache"
 require "mongoid/warnings"
+require "mongoid/utils"
 
 # If we are using Rails then we will include the Mongoid railtie. This has all
 # the nifty initializers that Mongoid needs.
@@ -58,9 +61,19 @@ module Mongoid
   #     }
   #   end
   #
+  # @example Using a block without an argument. Use `config` inside
+  #   the block to perform variable assignment.
+  #
+  #   Mongoid.configure do
+  #     connect_to("mongoid_test")
+  #
+  #     config.preload_models = true
+  #
   # @return [ Config ] The configuration object.
-  def configure
-    block_given? ? yield(Config) : Config
+  def configure(&block)
+    return Config unless block_given?
+
+    block.arity == 0 ? Config.instance_exec(&block) : yield(Config)
   end
 
   # Convenience method for getting the default client.

data/spec/integration/app_spec.rb

@@ -16,10 +16,6 @@ describe 'Mongoid application tests' do
       skip 'Set APP_TESTS=1 in environment to run application tests'
     end
 
-    if SpecConfig.instance.rails_version < '7.1'
-      skip 'App tests require Rails > 7.0 (see https://stackoverflow.com/questions/79360526)'
-    end
-
     require 'fileutils'
     require 'mrss/child_process_helper'
     require 'open-uri'
@@ -91,12 +87,6 @@ describe 'Mongoid application tests' do
   end
 
   context 'new application - rails' do
-    before(:all) do
-      if SpecConfig.instance.rails_version < '7.1'
-        skip '`rails new` with rails < 7.1 fails because modern concurrent-ruby removed logger dependency'
-      end
-    end
-
     it 'creates' do
       install_rails
 
@@ -280,14 +270,6 @@ describe 'Mongoid application tests' do
       line =~ /mongoid/
     end
     gemfile_lines << "gem 'mongoid', path: '#{File.expand_path(BASE)}'\n"
-
-    # json 3.0 removed the quirks_mode and max_nesting keywords that
-    # ActiveSupport::JSON (every 7.x) still passes to JSON.generate and
-    # JSON.parse, so a freshly resolved bundle picks a 3.x json and the app
-    # fails to encode any response with ArgumentError: unknown keyword:
-    # quirks_mode. Cap json at 2.x, which still accepts those keywords.
-    gemfile_lines << "gem 'json', '< 3'\n"
-
     if rails_version
       gemfile_lines.delete_if do |line|
         line =~ /rails/
@@ -344,7 +326,7 @@ describe 'Mongoid application tests' do
   end
 
   def wait_for_port(port, timeout, process)
-    deadline = Time.now + timeout
+    deadline = Mongoid::Utils.monotonic_time + timeout
     loop do
       begin
         Socket.tcp('localhost', port, nil, nil, connect_timeout: 0.5) do |socket|
@@ -354,7 +336,7 @@ describe 'Mongoid application tests' do
         unless process.alive?
           raise "Process #{process} died while waiting for port #{port}"
         end
-        if Time.now > deadline
+        if Mongoid::Utils.monotonic_time > deadline
           raise
         end
       end

data/spec/integration/associations/embeds_one_spec.rb

@@ -3,7 +3,8 @@
 require 'spec_helper'
 
 describe 'embeds_one associations' do
-  context 'when re-associating the same object' do
+
+  context 're-associating the same object' do
     context 'with dependent: destroy' do
       let(:canvas) do
         Canvas.create!(palette: Palette.new)
@@ -15,7 +16,7 @@ describe 'embeds_one associations' do
         canvas.palette = canvas.palette
         canvas.save!
         canvas.reload
-        expect(canvas.palette).to eq palette
+        canvas.palette.should == palette
       end
     end
   end
@@ -29,33 +30,12 @@ describe 'embeds_one associations' do
     end
 
     it 'loads the association correctly' do
-      expect { klass }.not_to raise_error
-      expect { klass.new.address }.not_to raise_error
+      expect { klass }.to_not raise_error
+      expect { klass.new.address }.to_not raise_error
       instance = klass.new
       address = Address.new
       instance.address = address
       expect(instance.address).to eq address
     end
   end
-
-  context 'when parent is persisted' do
-    let!(:person) do
-      Person.create!
-    end
-
-    context 'when assigning the new child' do
-      context 'when assigning an attribute to the child' do
-        before do
-          # person.reload
-          person.name = Name.new
-          person.name.first_name = 'Dmitry'
-          person.save!
-        end
-
-        it 'persists the child' do
-          expect(person.reload.name.first_name).to eq 'Dmitry'
-        end
-      end
-    end
-  end
 end

data/spec/integration/associations/has_and_belongs_to_many_spec.rb

@@ -2,28 +2,6 @@
 
 require 'spec_helper'
 
-module HabtmSpec
-  class Page
-    include Mongoid::Document
-    embeds_many :blocks, class_name: 'HabtmSpec::Block'
-  end
-
-  class Block
-    include Mongoid::Document
-    embedded_in :page, class_name: 'HabtmSpec::Page'
-  end
-
-  class ImageBlock < Block
-    has_and_belongs_to_many :attachments, inverse_of: nil, class_name: 'HabtmSpec::Attachment'
-    accepts_nested_attributes_for :attachments
-  end
-
-  class Attachment
-    include Mongoid::Document
-    field :file, type: String
-  end
-end
-
 describe 'has_and_belongs_to_many associations' do
 
   context 'when an anonymous class defines a has_and_belongs_to_many association' do
@@ -40,37 +18,4 @@ describe 'has_and_belongs_to_many associations' do
       expect(klass.new.movies.build).to be_a Movie
     end
   end
-
-  context 'when an embedded has habtm relation' do
-    let(:attachment) { HabtmSpec::Attachment.create!(file: 'foo.jpg') }
-
-    let(:page) { HabtmSpec::Page.create! }
-
-    let(:image_block) do
-      image_block = page.blocks.build({
-        _type: 'HabtmSpec::ImageBlock',
-        attachment_ids: [ attachment.id.to_s ],
-        attachments_attributes: { '1234' => { file: 'bar.jpg', id: attachment.id.to_s } }
-      })
-    end
-
-    # The nested attributes are processed before the attachment_ids
-    # assignment is applied, so at that point the id is not yet in the
-    # association and resolving it requires a collection-wide lookup.
-    context 'when allow_reparenting_via_nested_attributes is false' do
-      config_override :allow_reparenting_via_nested_attributes, false
-
-      it 'raises a document not found error' do
-        expect { image_block.save! }.to raise_error(Mongoid::Errors::DocumentNotFound)
-      end
-    end
-
-    context 'when allow_reparenting_via_nested_attributes is true' do
-      config_override :allow_reparenting_via_nested_attributes, true
-
-      it 'does not raise on save' do
-        expect { image_block.save! }.not_to raise_error
-      end
-    end
-  end
 end