This project is archived and is in readonly mode.
Fix a syntax error in current_page?() that was prevent matches against URL's with multiple query parameters.
Reported by chris finne | February 4th, 2009 @ 06:11 AM | in 2.x
You couldn't check the current page with URL's that had multiple query parameters
/shop/checkout?order=desc&page=1
because the syntax error would make it look like
/shop/checkout?order=desc&page=1
Comments and changes to this ticket
-
Andrew White February 4th, 2009 @ 09:47 AM
FYI, I've already reported it but there hasn't been much interested since it's been broken for a while.
-
Repository February 5th, 2009 @ 07:38 PM
- State changed from “new” to “committed”
(from [b80fa817d422e6c7ab743f4aa5cb4315b48ec7d5]) Fix a syntax error in current_page?() that was prevent matches against URL's with multiple query parameters [#1868 state:committed]
Signed-off-by: David Heinemeier Hansson david@loudthinking.com http://github.com/rails/rails/co...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
chris finne
Attachments
Tags
Referenced by
-
1868
Fix a syntax error in current_page?() that was prevent matches against URL's with multiple query parameters.
(from [b80fa817d422e6c7ab743f4aa5cb4315b48ec7d5]) Fix a s...