@@ -31,7 +31,7 @@ module ShareLayouts::Helper end def find_page - page = Page.find_by_url(request.request_uri) rescue nil + page = Page.find_by_url(request.request_uri.split("?").first) rescue nil page.is_a?(RailsPage) ? page : RailsPage.new(:class_name => "RailsPage") end end lib/share_layouts/helper.rb @@ -113,4 +113,16 @@ TEXT assert_not_equal pages(:rails_page), find_page assert_kind_of RailsPage, find_page end + + def test_should_find_page_ignoring_any_get_params + @request.request_uri = "/app?param1=value1;moreparams=radiant" + assert_equal pages(:rails_page), find_page + assert_kind_of RailsPage, find_page + @request.request_uri = "/app/something?param1=value1;moreparams=radiant" + assert_equal pages(:rails_page), find_page + assert_kind_of RailsPage, find_page + @request.request_uri = "/some-other/url/?param1=value1;moreparams=radiant" + assert_not_equal pages(:rails_page), find_page + assert_kind_of RailsPage, find_page + end end test/unit/share_layouts_helper_test.rb 6cd1b5281f6bbfbd5d37a2bc65882839ff933c94 Chris Parrish chris.parrish-gitmail@swankinnovations.com http://github.com/chrisparrish/radiant-share-layouts-extension/commit/eefcfdc0c9541e19cae17e796074fc80707cd5c4 eefcfdc0c9541e19cae17e796074fc80707cd5c4 2008-11-15T00:54:12-08:00 2008-11-15T00:54:12-08:00 Added support for GET params in RailsPage url ec418b67575f4df5b034b63bed50164720a305cf Chris Parrish chris.parrish-gitmail@swankinnovations.com