rewrite of controller_extensions method

lib/merit/controller_extensions.rb

@@ -46,20 +46,24 @@ def had_errors?
       target_object.respond_to?(:errors) && target_object.errors.try(:present?)
     end
 
-    def warn_no_object_found
-        str = '[merit] No object found, you might need a ' \
-          "'@#{controller_name.singularize}' variable in " \
-          "'#{controller_path}_controller' if no reputation is applied. " \
-          'If you are using `model_name` option in the rule this is ok.'
-        Rails.logger.warn str
-    end
-
     def target_object
       variable_name = :"@#{controller_name.singularize}"
       if instance_variable_defined?(variable_name)
-        target_obj = instance_variable_get(variable_name)
+        if target_obj = instance_variable_get(variable_name)
+          target_obj
+        else
+          warn_no_object_found
+        end
       end
-      target_obj.nil? ? warn_no_object_found : target_obj
+    end
+
+    def warn_no_object_found
+      str = '[merit] No object found, you might need a ' \
+        "'@#{controller_name.singularize}' variable in " \
+        "'#{controller_path}_controller' if no reputation is applied. " \
+        'If you are using `model_name` option in the rule this is ok.'
+      Rails.logger.warn str
+      nil
     end
 
     def target_id