Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

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.

#1868

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

Reported by chris finne · February 4th, 2009 @ 06:11 AM

State: committed
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. Andrew White
    Andrew White

    FYI, I've already reported it but there hasn't been much interested since it's been broken for a while.

    http://rails.lighthouseapp.com/p...

    February 4th, 2009 @ 09:47 AM

  2. Repository
    Repository
    • 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 Fix a syntax error in current_page?() that was prevent matches against URL's ... state:committed]

    Signed-off-by: David Heinemeier Hansson david@loudthinking.com http://github.com/rails/rails/co...

    February 5th, 2009 @ 07:38 PM