Merge pull request #18392 from brainopia/fix_route_requirements

Correct route requirements by overriding defaultls

2 files changed, 9 insertions, 1 deletions

diff --git a/actionpack/lib/action_dispatch/journey/route.rb b/actionpack/lib/action_dispatch/journey/route.rb
index 4d5c18984a..4698ff8cc7 100644
--- a/actionpack/lib/action_dispatch/journey/route.rb
+++ b/actionpack/lib/action_dispatch/journey/route.rb
@@ -36,7 +36,7 @@ module ActionDispatch
 
       def requirements # :nodoc:
         # needed for rails `rake routes`
-        path.requirements.merge(@defaults).delete_if { |_,v|
+        @defaults.merge(path.requirements).delete_if { |_,v|
           /.+?/ == v
         }
       end
diff --git a/actionpack/test/journey/route_test.rb b/actionpack/test/journey/route_test.rb
index 21d867aca0..9616f036b3 100644
--- a/actionpack/test/journey/route_test.rb
+++ b/actionpack/test/journey/route_test.rb
@@ -25,6 +25,14 @@ module ActionDispatch
         end
       end
 
+      def test_path_requirements_override_defaults
+        strexp    = Router::Strexp.build(':name', { name: /love/ }, ['/'])
+        path      = Path::Pattern.new strexp
+        defaults  = { name: 'tender' }
+        route     = Route.new('name', nil, path, nil, defaults)
+        assert_equal /love/, route.requirements[:name]
+      end
+
       def test_ip_address
         path  = Path::Pattern.from_string '/messages/:id(.:format)'
         route = Route.new("name", nil, path, {:ip => '192.168.1.1'},