rubocop-rails 2.33.1 → 2.33.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af326236e2f2697cca258f339f0f72d693b4b1f915acd4ba5c383c5d93c2b02c
-  data.tar.gz: 9f125d09868d129709ed2bc8bbeeda73fd25c296f033e378558224ea7592f82b
+  metadata.gz: 1dd62ec2504a075e89e8933ab9a8ecc63061d04c75f604d661d21a87b264cdee
+  data.tar.gz: a47a5b0e0d1e2f94b4f5a2976875ce86eb19c2c9fed53a1767ce5c59666d819e
 SHA512:
-  metadata.gz: 9f510dd2a447f4762d4b2364bb76e25dac66b1f166a4eae93fc0602ba379154b2d686cb04f1569d97220857a65c6cd6ff6b55cc97f3eb6d0c207b562d4d08e71
-  data.tar.gz: 21334214037ac98c9256a0b198f5d556548e6a60900d02da1553ce46127f21760d7183e8e52ba2bb3b151755848f045c927f2157a0fb7d900e93fcbb82c88937
+  metadata.gz: 5c48f2805329e13493881423a0de8549a7e3d3cfacc99be40b5a9dbd36b4a6c3d376845c5b0bcf3b492023835e8b3f5062d291c8ebe87165d82f96a77dbbc102
+  data.tar.gz: b30b183e23a29af49d9a6709d27318d79d4ff24246ba1b21ac8ae14260acbb64fd46ec54661f3d18867ffb331f71d794a33e5b760545b0743645ce902cac1daf

data/lib/rubocop/cop/rails/find_by_or_assignment_memoization.rb

@@ -13,18 +13,32 @@ module RuboCop
       #   or the code may have a different purpose than memoization.
       #
       # @example
-      #   # bad
+      #   # bad - exclusively doing memoization
       #   def current_user
       #     @current_user ||= User.find_by(id: session[:user_id])
       #   end
       #
       #   # good
       #   def current_user
-      #     if instance_variable_defined?(:@current_user)
+      #     return @current_user if defined?(@current_user)
+      #
+      #     @current_user = User.find_by(id: session[:user_id])
+      #   end
+      #
+      #   # bad - method contains other code
+      #   def current_user
+      #     @current_user ||= User.find_by(id: session[:user_id])
+      #     @current_user.do_something
+      #   end
+      #
+      #   # good
+      #   def current_user
+      #     if defined?(@current_user)
       #       @current_user
       #     else
       #       @current_user = User.find_by(id: session[:user_id])
       #     end
+      #     @current_user.do_something
       #   end
       class FindByOrAssignmentMemoization < Base
         extend AutoCorrector
@@ -40,6 +54,22 @@ module RuboCop
           )
         PATTERN
 
+        # When a method body contains only memoization, the correction can be more succinct.
+        def on_def(node)
+          find_by_or_assignment_memoization(node.body) do |varible_name, find_by|
+            add_offense(node.body) do |corrector|
+              corrector.replace(
+                node.body,
+                <<~RUBY.rstrip
+                  return #{varible_name} if defined?(#{varible_name})
+                  
+                  #{varible_name} = #{find_by.source}
+                RUBY
+              )
+            end
+          end
+        end
+
         def on_send(node)
           assignment_node = node.parent
           find_by_or_assignment_memoization(assignment_node) do |varible_name, find_by|
@@ -49,7 +79,7 @@ module RuboCop
               corrector.replace(
                 assignment_node,
                 <<~RUBY.rstrip
-                  if instance_variable_defined?(:#{varible_name})
+                  if defined?(#{varible_name})
                     #{varible_name}
                   else
                     #{varible_name} = #{find_by.source}

data/lib/rubocop/cop/rails/read_write_attribute.rb

@@ -66,7 +66,7 @@ module RuboCop
           return false unless enclosing_method
 
           shadowing_method_name = first_arg.value.to_s
-          shadowing_method_name << '=' if node.method?(:write_attribute)
+          shadowing_method_name += '=' if node.method?(:write_attribute)
           enclosing_method.method?(shadowing_method_name)
         end
 

data/lib/rubocop/rails/version.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Rails
     # This module holds the RuboCop Rails version information.
     module Version
-      STRING = '2.33.1'
+      STRING = '2.33.3'
 
       def self.document_version
         STRING.match('\d+\.\d+').to_s

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-rails
 version: !ruby/object:Gem::Version
-  version: 2.33.1
+  version: 2.33.3
 platform: ruby
 authors:
 - Bozhidar Batsov