##// END OF EJS Templates
Don't generate pagination links with params....
Jean-Philippe Lang -
r15245:5d31eb2495c2
parent child
Show More
@@ -145,7 +145,7 module Redmine
145 if block_given?
145 if block_given?
146 yield text, parameters, options
146 yield text, parameters, options
147 else
147 else
148 link_to text, params.merge(parameters), options
148 link_to text, {:params => request.query_parameters.merge(parameters)}, options
149 end
149 end
150 end
150 end
151 end
151 end
@@ -135,11 +135,11 class IssuesTest < Redmine::IntegrationTest
135 end
135 end
136 end
136 end
137
137
138 def test_pagination_links_on_index_without_project_id_in_url
138 def test_pagination_links_should_preserve_query_parameters
139 with_settings :per_page_options => '2' do
139 with_settings :per_page_options => '2' do
140 get '/issues', :project_id => 'ecookbook'
140 get '/projects/ecookbook/issues?foo=bar'
141
141
142 assert_select 'a[href=?]', '/projects/ecookbook/issues?page=2', :text => '2'
142 assert_select 'a[href=?]', '/projects/ecookbook/issues?foo=bar&page=2', :text => '2'
143 end
143 end
144 end
144 end
145
145
General Comments 0
You need to be logged in to leave comments. Login now