From 5d31eb2495c2c2cce3ab83867f0d2d01e5d1693c 2016-07-10 16:47:49 From: Jean-Philippe Lang Date: 2016-07-10 16:47:49 Subject: [PATCH] Don't generate pagination links with params. git-svn-id: http://svn.redmine.org/redmine/trunk@15627 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- diff --git a/lib/redmine/pagination.rb b/lib/redmine/pagination.rb index a28236a..0457a4b 100644 --- a/lib/redmine/pagination.rb +++ b/lib/redmine/pagination.rb @@ -145,7 +145,7 @@ module Redmine if block_given? yield text, parameters, options else - link_to text, params.merge(parameters), options + link_to text, {:params => request.query_parameters.merge(parameters)}, options end end end diff --git a/test/integration/issues_test.rb b/test/integration/issues_test.rb index fb80002..5d94a57 100644 --- a/test/integration/issues_test.rb +++ b/test/integration/issues_test.rb @@ -135,11 +135,11 @@ class IssuesTest < Redmine::IntegrationTest end end - def test_pagination_links_on_index_without_project_id_in_url + def test_pagination_links_should_preserve_query_parameters with_settings :per_page_options => '2' do - get '/issues', :project_id => 'ecookbook' - - assert_select 'a[href=?]', '/projects/ecookbook/issues?page=2', :text => '2' + get '/projects/ecookbook/issues?foo=bar' + + assert_select 'a[href=?]', '/projects/ecookbook/issues?foo=bar&page=2', :text => '2' end end