mongoid 9.0.11 → 9.1.0

data/lib/mongoid/clients/sessions.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Clients
-
     # Encapsulates behavior for using sessions and transactions.
     module Sessions
-
       # Add class method mixin functionality.
       #
       # @todo Replace with ActiveSupport::Concern
@@ -15,10 +12,9 @@ module Mongoid
       end
 
       module ClassMethods
-
         # Actions that can be used to trigger transactional callbacks.
         # @api private
-        CALLBACK_ACTIONS = [:create, :destroy, :update]
+        CALLBACK_ACTIONS = %i[create destroy update]
 
         # Execute a block within the context of a session.
         #
@@ -40,24 +36,21 @@ module Mongoid
         #
         # @yieldparam [ Mongo::Session ] The session being used for the block.
         def with_session(options = {})
-          if Threaded.get_session(client: persistence_context.client)
-            raise Mongoid::Errors::InvalidSessionNesting.new
-          end
+          raise Mongoid::Errors::InvalidSessionNesting.new if Threaded.get_session(client: persistence_context.client)
+
           session = persistence_context.client.start_session(options)
           Threaded.set_session(session, client: persistence_context.client)
           yield(session)
-        rescue Mongo::Error::InvalidSession => ex
-          if Mongo::Error::SessionsNotSupported === ex
-            raise Mongoid::Errors::SessionsNotSupported.new
-          else
-            raise ex
-          end
-        rescue Mongo::Error::OperationFailure => ex
-          if (ex.code == 40415 && ex.server_message =~ /startTransaction/) ||
-             (ex.code == 20 && ex.server_message =~ /Transaction/)
+        rescue Mongo::Error::InvalidSession => e
+          raise Mongoid::Errors::SessionsNotSupported.new if e.is_a?(Mongo::Error::SessionsNotSupported)
+
+          raise e
+        rescue Mongo::Error::OperationFailure => e
+          if (e.code == 40_415 && e.server_message =~ /startTransaction/) ||
+             (e.code == 20 && e.server_message =~ /Transaction/)
             raise Mongoid::Errors::TransactionsNotSupported.new
           else
-            raise ex
+            raise e
           end
         rescue *transactions_not_supported_exceptions
           raise Mongoid::Errors::TransactionsNotSupported
@@ -87,26 +80,22 @@ module Mongoid
         #   by MongoDB deployment or MongoDB driver.
         #
         # @yield Provided block will be executed inside a transaction.
-        def transaction(options = {}, session_options: {})
+        def transaction(options = {}, session_options: {}, &block)
           with_session(session_options) do |session|
-            begin
-              session.with_transaction(options) do
-                yield
-              end.tap { run_commit_callbacks(session) }
-            rescue *transactions_not_supported_exceptions
-              raise Mongoid::Errors::TransactionsNotSupported
-            rescue Mongoid::Errors::Rollback
-              run_abort_callbacks(session)
-            rescue Mongoid::Errors::InvalidSessionNesting
-              # Session should be ended here.
-              raise Mongoid::Errors::InvalidTransactionNesting.new
-            rescue Mongo::Error::InvalidSession, Mongo::Error::InvalidTransactionOperation => e
-              run_abort_callbacks(session)
-              raise Mongoid::Errors::TransactionError.new(e)
-            rescue StandardError => e
-              run_abort_callbacks(session)
-              raise e
-            end
+            session.with_transaction(options, &block).tap { run_commit_callbacks(session) }
+          rescue *transactions_not_supported_exceptions
+            raise Mongoid::Errors::TransactionsNotSupported
+          rescue Mongoid::Errors::Rollback
+            run_abort_callbacks(session)
+          rescue Mongoid::Errors::InvalidSessionNesting
+            # Session should be ended here.
+            raise Mongoid::Errors::InvalidTransactionNesting.new
+          rescue Mongo::Error::InvalidSession, Mongo::Error::InvalidTransactionOperation => e
+            run_abort_callbacks(session)
+            raise Mongoid::Errors::TransactionError.new(e)
+          rescue StandardError => e
+            run_abort_callbacks(session)
+            raise e
           end
         end
 
@@ -123,7 +112,7 @@ module Mongoid
 
         # Shortcut for +after_commit :hook, on: [ :create, :update ]+
         def after_save_commit(*args, &block)
-          set_options_for_callbacks!(args, on: [ :create, :update ])
+          set_options_for_callbacks!(args, on: %i[create update])
           set_callback(:commit, :after, *args, &block)
         end
 
@@ -216,14 +205,14 @@ module Mongoid
           options = args.extract_options!.merge(enforced_options)
           args << options
 
-          if options[:on]
-            fire_on = Array(options[:on])
-            assert_valid_transaction_action(fire_on)
-            options[:if] = [
-              -> { transaction_include_any_action?(fire_on) },
-              *options[:if]
-            ]
-          end
+          return unless options[:on]
+
+          fire_on = Array(options[:on])
+          assert_valid_transaction_action(fire_on)
+          options[:if] = [
+            -> { transaction_include_any_action?(fire_on) },
+            *options[:if]
+          ]
         end
 
         # Asserts that the given actions are valid for after_commit
@@ -232,9 +221,10 @@ module Mongoid
         # @param [ Array<Symbol> ] actions Actions to be checked.
         # @raise [ ArgumentError ] If any of the actions is not valid.
         def assert_valid_transaction_action(actions)
-          if (actions - CALLBACK_ACTIONS).any?
-            raise ArgumentError, ":on conditions for after_commit and after_rollback callbacks have to be one of #{CALLBACK_ACTIONS}"
-          end
+          return unless (actions - CALLBACK_ACTIONS).any?
+
+          raise ArgumentError,
+                ":on conditions for after_commit and after_rollback callbacks have to be one of #{CALLBACK_ACTIONS}"
         end
 
         def transaction_include_any_action?(actions)
@@ -274,17 +264,17 @@ module Mongoid
         # a model within a transaction, where the model is not itself
         # controlled by that transaction. this is potentially a bug, so
         # let's tell them about it.
-        if session.nil?
-          # This is hacky; we're hijacking Mongoid::Errors::MongoidError in
-          # order to get the spiffy error message translation. If we later
-          # decide to raise an error instead of just writing a message, we can
-          # subclass MongoidError and raise that exception here.
-          message = Errors::MongoidError.new.compose_message(
-            'client_session_mismatch',
-            model: self.class.name
-          )
-          logger.info(message)
-        end
+        return unless session.nil?
+
+        # This is hacky; we're hijacking Mongoid::Errors::MongoidError in
+        # order to get the spiffy error message translation. If we later
+        # decide to raise an error instead of just writing a message, we can
+        # subclass MongoidError and raise that exception here.
+        message = Errors::MongoidError.new.compose_message(
+          'client_session_mismatch',
+          model: self.class.name
+        )
+        logger.info(message)
       end
     end
   end

data/lib/mongoid/clients/storage_options.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Clients
-
     # Mixin module included into Mongoid::Document which adds
     # the ability to set the collection in which to store the
     # document by default.
@@ -46,7 +44,6 @@ module Mongoid
       end
 
       module ClassMethods
-
         # Give this model specific custom default storage options.
         #
         # @example Store this model by default in "artists"
@@ -82,7 +79,7 @@ module Mongoid
         # @return [ Class ] The model class.
         def store_in(options)
           Validators::Storage.validate(self, options)
-          self.storage_options = self.storage_options.merge(options)
+          self.storage_options = storage_options.merge(options)
         end
 
         # Reset the store_in options

data/lib/mongoid/clients/validators/storage.rb

@@ -1,16 +1,14 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Clients
     module Validators
-
       # Validates the options passed to :store_in.
       module Storage
         extend self
 
         # The valid options for storage.
-        VALID_OPTIONS = [ :collection, :collection_options, :database, :client ].freeze
+        VALID_OPTIONS = %i[collection collection_options database client].freeze
 
         # Validate the options provided to :store_in.
         #
@@ -37,6 +35,7 @@ module Mongoid
         # @return [ true | false ] If all keys are valid.
         def valid_keys?(options)
           return false unless options.is_a?(::Hash)
+
           options.keys.all? do |key|
             VALID_OPTIONS.include?(key)
           end

data/lib/mongoid/clients/validators.rb

@@ -1,4 +1,3 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "mongoid/clients/validators/storage"
+require 'mongoid/clients/validators/storage'

data/lib/mongoid/clients.rb

@@ -1,14 +1,12 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "mongoid/clients/factory"
-require "mongoid/clients/validators"
-require "mongoid/clients/storage_options"
-require "mongoid/clients/options"
-require "mongoid/clients/sessions"
+require 'mongoid/clients/factory'
+require 'mongoid/clients/validators'
+require 'mongoid/clients/storage_options'
+require 'mongoid/clients/options'
+require 'mongoid/clients/sessions'
 
 module Mongoid
-
   # Mixin module included into Mongoid::Document which adds
   # database client connection functionality. Also contains
   # singleton class methods related to managing database clients.
@@ -19,7 +17,6 @@ module Mongoid
     include Sessions
 
     class << self
-
       # Clear all clients from the current thread.
       #
       # @example Clear all clients.
@@ -75,8 +72,10 @@ module Mongoid
       def with_name(name)
         name_as_symbol = name.to_sym
         return clients[name_as_symbol] if clients[name_as_symbol]
+
         CREATE_LOCK.synchronize do
-          if (key_vault_client = Mongoid.clients.dig(name_as_symbol, :options, :auto_encryption_options, :key_vault_client))
+          if (key_vault_client = Mongoid.clients.dig(name_as_symbol, :options, :auto_encryption_options,
+                                                     :key_vault_client))
             clients[key_vault_client.to_sym] ||= Clients::Factory.create(key_vault_client)
           end
           clients[name_as_symbol] ||= Clients::Factory.create(name)
@@ -103,8 +102,6 @@ module Mongoid
         @clients ||= {}
       end
 
-      private
-
       CREATE_LOCK = Mutex.new
     end
   end

data/lib/mongoid/collection_configurable.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
-
   # Encapsulates behavior around defining collections.
   module CollectionConfigurable
     extend ActiveSupport::Concern
@@ -26,22 +24,20 @@ module Mongoid
           # This is most probably an anonymous class, we ignore them.
           return
         end
-        if collection_name.match(/^system\./)
+        if /^system\./.match?(collection_name)
           # We do not do anything with system collections.
           return
         end
-        if force
-          collection.drop
-        end
+
+        collection.drop if force
         if coll_options = collection.database.list_collections(filter: { name: collection_name.to_s }).first
-          if force
-            raise Errors::DropCollectionFailure.new(collection_name)
-          else
-            logger.debug(
-              "MONGOID: Collection '#{collection_name}' already exists " +
-              "in database '#{database_name}' with options '#{coll_options}'."
-            )
-          end
+          raise Errors::DropCollectionFailure.new(collection_name) if force
+
+          logger.debug(
+            "MONGOID: Collection '#{collection_name}' already exists " +
+            "in database '#{database_name}' with options '#{coll_options}'."
+          )
+
         else
           begin
             collection.database[collection_name, storage_options.fetch(:collection_options, {})].create

data/lib/mongoid/composable.rb

@@ -1,30 +1,28 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "mongoid/changeable"
-require "mongoid/collection_configurable"
-require "mongoid/encryptable"
-require "mongoid/findable"
+require 'mongoid/changeable'
+require 'mongoid/collection_configurable'
+require 'mongoid/encryptable'
+require 'mongoid/findable'
 require 'mongoid/identifiable'
-require "mongoid/indexable"
-require "mongoid/inspectable"
-require "mongoid/interceptable"
-require "mongoid/matcher"
-require "mongoid/matchable"
-require "mongoid/persistable"
-require "mongoid/reloadable"
+require 'mongoid/indexable'
+require 'mongoid/inspectable'
+require 'mongoid/interceptable'
+require 'mongoid/matcher'
+require 'mongoid/matchable'
+require 'mongoid/persistable'
+require 'mongoid/reloadable'
 require 'mongoid/search_indexable'
-require "mongoid/selectable"
-require "mongoid/scopable"
-require "mongoid/serializable"
-require "mongoid/shardable"
-require "mongoid/stateful"
-require "mongoid/cacheable"
-require "mongoid/traversable"
-require "mongoid/validatable"
+require 'mongoid/selectable'
+require 'mongoid/scopable'
+require 'mongoid/serializable'
+require 'mongoid/shardable'
+require 'mongoid/stateful'
+require 'mongoid/cacheable'
+require 'mongoid/traversable'
+require 'mongoid/validatable'
 
 module Mongoid
-
   # This module provides inclusions of all behavior in a Mongoid document.
   module Composable
     extend ActiveSupport::Concern
@@ -105,21 +103,19 @@ module Mongoid
     # separately.
     #
     # @return [ Array<Symbol> ] A list of reserved method names.
-    RESERVED_METHOD_NAMES = [ :fields,
-                              :aliased_fields,
-                              :localized_fields,
-                              :index_specifications,
-                              :shard_key_fields,
-                              :nested_attributes,
-                              :readonly_attributes,
-                              :storage_options,
-                              :cascades,
-                              :cyclic,
-                              :cache_timestamp_format
-                            ]
+    RESERVED_METHOD_NAMES = %i[fields
+                               aliased_fields
+                               localized_fields
+                               index_specifications
+                               shard_key_fields
+                               nested_attributes
+                               readonly_attributes
+                               storage_options
+                               cascades
+                               cyclic
+                               cache_timestamp_format]
 
     class << self
-
       # Get a list of methods that would be a bad idea to define as field names
       # or override when including Mongoid::Document.
       #

data/lib/mongoid/config/defaults.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Config
-
     # Encapsulates logic for loading defaults.
     module Defaults
-
       # Load the defaults for the feature flags in the given Mongoid version.
       # Note that this method will load the *new* functionality introduced in
       # the given Mongoid version.
@@ -33,7 +30,13 @@ module Mongoid
           load_defaults '9.0'
 
         when '9.0'
-          # All flag defaults currently reflect 9.0 behavior.
+          self.allow_reparenting_via_nested_attributes = true
+          self.autosave_saves_unchanged_documents = true
+
+          load_defaults '9.1'
+
+        when '9.1'
+          # All flag defaults currently reflect 9.1 behavior.
 
         else
           raise ArgumentError, "Unknown version: #{version}"

data/lib/mongoid/config/encryption.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 require 'mongoid/extensions/boolean'
 require 'mongoid/stringified_symbol'
@@ -24,6 +23,7 @@ module Mongoid
         visited = Set.new
         models.each_with_object({}) do |model, map|
           next if visited.include?(model)
+
           visited << model
           next if model.embedded?
           next unless model.encrypted?
@@ -182,8 +182,6 @@ module Mongoid
           DETERMINISTIC_ALGORITHM
         when false
           RANDOM_ALGORITHM
-        else
-          nil
         end
       end
 

data/lib/mongoid/config/environment.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Config
-
     # Encapsulates logic for getting environment information.
     module Environment
       extend self
@@ -25,13 +23,10 @@ module Mongoid
       # @return [ String ] The name of the current environment.
       # @api public
       def env_name
-        if defined?(::Rails)
-          return ::Rails.env
-        end
-        if defined?(::Sinatra)
-          return ::Sinatra::Base.environment.to_s
-        end
-        ENV["RACK_ENV"] || ENV["MONGOID_ENV"] or raise Errors::NoEnvironment
+        return ::Rails.env if defined?(::Rails)
+        return ::Sinatra::Base.environment.to_s if defined?(::Sinatra)
+
+        ENV['RACK_ENV'] || ENV['MONGOID_ENV'] or raise Errors::NoEnvironment
       end
 
       # Load the yaml from the provided path and return the settings for the
@@ -51,9 +46,7 @@ module Mongoid
         env = environment ? environment.to_s : env_name
 
         contents = File.read(path)
-        if contents.empty?
-          raise Mongoid::Errors::EmptyConfigFile.new(path)
-        end
+        raise Mongoid::Errors::EmptyConfigFile.new(path) if contents.empty?
 
         # These are the classes that can be used in a Mongoid
         # configuration file in addition to standard YAML types.
@@ -62,15 +55,13 @@ module Mongoid
           Symbol,
           # BSON::Binary occur as keyId values for FLE (more precisely,
           # the keyIds are UUIDs).
-          BSON::Binary,
+          BSON::Binary
         ]
 
         result = ERB.new(contents).result
         data = YAML.safe_load(result, permitted_classes: permitted_classes, aliases: true)
 
-        unless data.is_a?(Hash)
-          raise Mongoid::Errors::InvalidConfigFile.new(path)
-        end
+        raise Mongoid::Errors::InvalidConfigFile.new(path) unless data.is_a?(Hash)
 
         data[env]
       end

data/lib/mongoid/config/introspection.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Config
-
     # This module provides a way to inspect not only the defined configuration
     # settings and their defaults (which are available via
     # `Mongoid::Config.settings`), but also the documentation about them. It
@@ -69,7 +67,7 @@ module Mongoid
         #
         # @return [ String ] the reformatted comment
         def indented_comment(indent: 2, indent_first_line: false)
-          comment.gsub(/^/, " " * indent).tap do |result|
+          comment.gsub(/^/, ' ' * indent).tap do |result|
             result.strip! unless indent_first_line
           end
         end
@@ -79,17 +77,17 @@ module Mongoid
         #
         # @return [ true | false ] whether the option is deprecated or not.
         def deprecated?
-          comment.include?("(Deprecated)")
+          comment.include?('(Deprecated)')
         end
 
         # Compare self with the given option.
         #
         # @return [ true | false ] If name, default, and comment are all the
         #   same, return true. Otherwise, false.
-        def ==(option)
-          name == option.name &&
-            default == option.default &&
-            comment == option.comment
+        def ==(other)
+          name == other.name &&
+            default == other.default &&
+            comment == other.comment
         end
 
         private
@@ -101,7 +99,7 @@ module Mongoid
         #
         # @return [ String ] the unindented text.
         def unindent(text)
-          text.strip.gsub(/^\s+/, "")
+          text.strip.gsub(/^\s+/, '')
         end
       end
 
@@ -117,7 +115,7 @@ module Mongoid
       #   1: the (potentially multiline) comment
       #   2: the option's name
       #   3: the option's default value
-      OPTION_PATTERN = %r{
+      OPTION_PATTERN = /
         (
           ((?:^\s*\#.*\n)+)  # match one or more lines of comments
           ^\s+option\s+      # followed immediately by a line declaring an option
@@ -125,11 +123,12 @@ module Mongoid
           default:\s+(.*?)   # match the default value for the option
           (?:,.*?)?          # skip any other configuration
         \n)                  # end with a newline
-      }x
+      /x
 
       # The full path to the source file of the Mongoid::Config module.
       CONFIG_RB_PATH = File.absolute_path(File.join(
-        File.dirname(__FILE__), "../config.rb"))
+                                            File.dirname(__FILE__), '../config.rb'
+                                          ))
 
       # Extracts the available configuration options from the Mongoid::Config
       # source file, and returns them as an array of hashes.
@@ -142,11 +141,10 @@ module Mongoid
       def options(include_deprecated: false)
         src = File.read(CONFIG_RB_PATH)
         src.scan(OPTION_PATTERN)
-          .map { |opt| Option.from_captures(opt) }
-          .reject { |opt| !include_deprecated && opt.deprecated? }
-          .sort_by { |opt| opt.name }
+           .map { |opt| Option.from_captures(opt) }
+           .reject { |opt| !include_deprecated && opt.deprecated? }
+           .sort_by { |opt| opt.name }
       end
     end
-
   end
 end

data/lib/mongoid/config/options.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
 module Mongoid
   module Config
-
     # Encapsulates logic for setting options.
     module Options
-
       # Get the defaults or initialize a new empty hash.
       #
       # @example Get the defaults.
@@ -40,8 +37,17 @@ module Mongoid
           end
 
           define_method("#{name}=") do |value|
+            old_value = settings[name]
             settings[name] = value
-            options[:on_change]&.call(value)
+
+            begin
+              options[:on_change]&.call(value)
+            rescue StandardError
+              # If the on_change callback raises an error, we need to roll
+              # the change back.
+              settings[name] = old_value
+              raise
+            end
           end
 
           define_method("#{name}?") do
@@ -81,13 +87,13 @@ module Mongoid
       #
       # @return [ Integer ] The log level.
       def log_level
-        if level = settings[:log_level]
-          unless level.is_a?(Integer)
-            # JRuby String#constantize does not work here.
-            level = Logger.const_get(level.upcase.to_s)
-          end
-          level
+        return unless level = settings[:log_level]
+
+        unless level.is_a?(Integer)
+          # JRuby String#constantize does not work here.
+          level = Logger.const_get(level.upcase.to_s)
         end
+        level
       end
     end
   end