Do not use short-circuit return

1 files changed, 3 insertions, 2 deletions

diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb
index 6c4c7c8d80..4c20974ac7 100644
--- a/actionpack/lib/action_dispatch/routing/mapper.rb
+++ b/actionpack/lib/action_dispatch/routing/mapper.rb
@@ -171,8 +171,9 @@ module ActionDispatch
           end
 
           def verify_callable_constraint(callable_constraint)
-            return if callable_constraint.respond_to?(:call) || callable_constraint.respond_to?(:matches?)
-            raise ArgumentError, "Invalid constraint: #{callable_constraint.inspect} must respond to :call or :matches?"
+            unless callable_constraint.respond_to?(:call) || callable_constraint.respond_to?(:matches?)
+              raise ArgumentError, "Invalid constraint: #{callable_constraint.inspect} must respond to :call or :matches?"
+            end
           end
 
           def normalize_conditions!