style(Style/OptionalBooleanParameter): manual

Author: antonsavitskiy

.rubocop.yml

@@ -238,6 +238,12 @@ Style/NumericLiterals:
 Style/NumericPredicate:
   Enabled: false
 
+Style/OptionalBooleanParameter:
+  AllowedMethods:
+    - respond_to?
+    - respond_to_missing?
+    - set_parent=
+
 Style/RaiseArgs:
   EnforcedStyle: compact
 

.rubocop_todo.yml

@@ -826,25 +826,6 @@ Style/OpenStructUse:
     - 'spec/rails/controller_extension/controller_runtime_spec.rb'
     - 'spec/support/rails_mock.rb'
 
-# Offense count: 17
-# Configuration parameters: AllowedMethods.
-# AllowedMethods: respond_to_missing?
-Style/OptionalBooleanParameter:
-  Exclude:
-    - 'lib/mongoid/association/embedded/cyclic.rb'
-    - 'lib/mongoid/association/many.rb'
-    - 'lib/mongoid/association/one.rb'
-    - 'lib/mongoid/association/referenced/has_many/enumerable.rb'
-    - 'lib/mongoid/atomic.rb'
-    - 'lib/mongoid/attributes/dynamic.rb'
-    - 'lib/mongoid/criteria.rb'
-    - 'lib/mongoid/criteria/queryable/extensions/string.rb'
-    - 'lib/mongoid/criteria/queryable/extensions/symbol.rb'
-    - 'lib/mongoid/criteria/queryable/key.rb'
-    - 'lib/mongoid/criteria/queryable/options.rb'
-    - 'spec/support/models/address.rb'
-    - 'spec/support/models/name.rb'
-
 # Offense count: 242
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.

lib/mongoid/association/embedded/cyclic.rb

@@ -65,7 +65,7 @@ def recursively_embeds_many(options = {})
           # or its children.
           def recursively_embeds_one(options = {})
             embeds_one(
-              cyclic_child_name(false),
+              cyclic_child_name(many: false),
               options.merge(class_name: name, cyclic: true)
             )
             embedded_in cyclic_parent_name, class_name: name, cyclic: true
@@ -92,7 +92,7 @@ def cyclic_parent_name
           #
           # @return [ String ] "child_" plus the class name underscored in
           #   singular or plural form.
-          def cyclic_child_name(many = true)
+          def cyclic_child_name(many: true)
             :"child_#{name.demodulize.underscore.send(many ? :pluralize : :singularize)}"
           end
         end

lib/mongoid/atomic.rb

@@ -116,7 +116,7 @@ def atomic_array_add_to_sets
     #   person.atomic_updates(children)
     #
     # @return [ Hash ] The updates and their modifiers.
-    def atomic_updates(_use_indexes = false)
+    def atomic_updates(_use_indexes: false)
       process_flagged_destroys
       mods = Modifiers.new
       generate_atomic_updates(mods, self)

lib/mongoid/criteria/queryable/expandable.rb

@@ -37,7 +37,7 @@ module Expandable
         #
         # @return [ Array<String, Object> ] The expanded field and value.
         def expand_one_condition(field, value)
-          kv = field.__expr_part__(value.__expand_complex__, negating?)
+          kv = field.__expr_part__(value.__expand_complex__, negating: negating?)
           [kv.keys.first.to_s, kv.values.first]
         end
 

lib/mongoid/criteria/queryable/extensions/string.rb

@@ -63,8 +63,8 @@ def __sort_option__
           # @param [ true | false ] negating If the selection should be negated.
           #
           # @return [ Hash ] The selection.
-          def __expr_part__(value, negating = false)
-            ::String.__expr_part__(self, value, negating)
+          def __expr_part__(value, negating: false)
+            ::String.__expr_part__(self, value, negating: negating)
           end
 
           module ClassMethods
@@ -79,7 +79,7 @@ module ClassMethods
             # @param [ true | false ] negating If the selection should be negated.
             #
             # @return [ Hash ] The selection.
-            def __expr_part__(key, value, negating = false)
+            def __expr_part__(key, value, negating: false)
               if negating
                 { key => { "$#{value.regexp? ? 'not' : 'ne'}" => value } }
               else

lib/mongoid/criteria/queryable/extensions/symbol.rb

@@ -17,8 +17,8 @@ module Symbol
           # @param [ true | false ] negating If the selection should be negated.
           #
           # @return [ Hash ] The selection.
-          def __expr_part__(value, negating = false)
-            ::String.__expr_part__(self, value, negating)
+          def __expr_part__(value, negating: false)
+            ::String.__expr_part__(self, value, negating: negating)
           end
 
           module ClassMethods

lib/mongoid/criteria/queryable/key.rb

@@ -131,8 +131,8 @@ def initialize(name, strategy, operator, expanded = nil, &block)
         # @param [ true | false ] negating If the selection should be negated.
         #
         # @return [ Hash ] The raw MongoDB selector.
-        def __expr_part__(object, negating = false)
-          { name.to_s => transform_value(object, negating) }
+        def __expr_part__(object, negating: false)
+          { name.to_s => transform_value(object, negating: negating) }
         end
 
         # Gets the raw selector condition that would be passed to Mongo.
@@ -144,7 +144,7 @@ def __expr_part__(object, negating = false)
         # @param [ true | false ] negating If the selection should be negated.
         #
         # @return [ Hash ] The raw MongoDB selector.
-        def transform_value(value, negating = false)
+        def transform_value(value, negating: false)
           expr = block ? block[value] : value
           expr = { expanded => expr } if expanded
           expr = { operator => expr }

lib/mongoid/criteria/queryable/mergeable.rb

@@ -261,7 +261,7 @@ def _mongoid_expand_keys(expr)
 
           result = BSON::Document.new
           expr.each do |field, value|
-            field.__expr_part__(value.__expand_complex__, negating?).each do |k, v|
+            field.__expr_part__(value.__expand_complex__, negating: negating?).each do |k, v|
               if (existing = result[k])
                 if existing.is_a?(Hash)
                   # Existing value is an operator.

lib/mongoid/criteria/queryable/optional.rb

@@ -119,7 +119,7 @@ def only(*args)
             options.store(
               :fields,
               args.inject(options[:fields] || {}) { |sub, field| sub.tap { sub[field] = 1 } },
-              false
+              localize: false
             )
           end
         end
@@ -241,7 +241,7 @@ def without(*args)
             options.store(
               :fields,
               args.inject(options[:fields] || {}) { |sub, field| sub.tap { sub[field] = 0 } },
-              false
+              localize: false
             )
           end
         end