This project is archived and is in readonly mode.
routing not recognizing requirements for :format
-
CancelProfileIsBroken
Here's a test that fails on edge.
-
CancelProfileIsBroken
- Tag changed from routing error, 2.3-rc1, routes, routing to routing error, 2.3-rc1, bugmash, routes, routing
-
Dan Croak
I couldn't get Mike's patch to apply on 2-3-stable so I've attached a patch with his test (slightly formatted to fit in my text editor).
I couldn't figure out where to look to make the test pass, but found some contenders:
Someone else needs to take it across the goaline. Definitely looks like a bug.
-
Kieran P
+1 for Dan Croaks patch for 2-3.
Mikes patch does not apply to the current version of master. Uploading new one that does.
-
Jeremy Kemper
- Assigned user set to josh
- State changed from new to open
- Milestone changed from 2.x to 2.3.4
-
RomKo
+1 for test patch, verified applies to master.
I dug around a bit, it seems that the :format requirement drops in RouteBuilder#assign_route_options since there's an optional segment for :format called OptionalFormatSegment.
-
Derander
+1 verified, applies to master.
I read the test, looks like it is testing the right thing. I'll look into a patch.
-
josh
- State changed from open to incomplete
-
josh
- State changed from incomplete to stale
