return early from add_request_method

1 files changed, 11 insertions, 11 deletions

diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb
index cc81b23947..5182d9e8e1 100644
--- a/actionpack/lib/action_dispatch/routing/mapper.rb
+++ b/actionpack/lib/action_dispatch/routing/mapper.rb
@@ -209,18 +209,18 @@ module ActionDispatch
           end
 
           def add_request_method(via, conditions)
-            unless via == [:all]
-              if via.empty?
-                msg = "You should not use the `match` method in your router without specifying an HTTP method.\n" \
-                      "If you want to expose your action to both GET and POST, add `via: [:get, :post]` option.\n" \
-                      "If you want to expose your action to GET, use `get` in the router:\n" \
-                      "  Instead of: match \"controller#action\"\n" \
-                      "  Do: get \"controller#action\""
-                raise ArgumentError, msg
-              end
-
-              conditions[:request_method] = via.map { |m| m.to_s.dasherize.upcase }
+            return if via == [:all]
+
+            if via.empty?
+              msg = "You should not use the `match` method in your router without specifying an HTTP method.\n" \
+                    "If you want to expose your action to both GET and POST, add `via: [:get, :post]` option.\n" \
+                    "If you want to expose your action to GET, use `get` in the router:\n" \
+                    "  Instead of: match \"controller#action\"\n" \
+                    "  Do: get \"controller#action\""
+              raise ArgumentError, msg
             end
+
+            conditions[:request_method] = via.map { |m| m.to_s.dasherize.upcase }
           end
 
           def app(blocks)