diff --git a/ckan/lib/helpers.py b/ckan/lib/helpers.py index ac41b481761..7e2bd65f233 100644 --- a/ckan/lib/helpers.py +++ b/ckan/lib/helpers.py @@ -1037,7 +1037,7 @@ def pager_url(page, partial=None, **kwargs): if routes_dict.get('id'): kwargs['id'] = routes_dict['id'] kwargs['page'] = page - return url(**kwargs) + return url_for(**kwargs) class Page(paginate.Page): diff --git a/ckan/tests/legacy/functional/test_pagination.py b/ckan/tests/legacy/functional/test_pagination.py index 59726aed2d7..6958ad56251 100644 --- a/ckan/tests/legacy/functional/test_pagination.py +++ b/ckan/tests/legacy/functional/test_pagination.py @@ -100,12 +100,12 @@ def teardown_class(self): def test_group_index(self): res = self.app.get(url_for(controller='group', action='index')) - assert 'href="/group?sort=&q=&page=2"' in res, res + assert 'href="/group?q=&sort=&page=2"' in res, res grp_numbers = scrape_search_results(res, 'group') assert_equal(['00', '01', '02', '03', '04', '05', '06', '07', '08', '09', '10', '11', '12', '13', '14', '15', '16', '17', '18', '19', '20'], grp_numbers) res = self.app.get(url_for(controller='group', action='index', page=2)) - assert 'href="/group?sort=&q=&page=1"' in res + assert 'href="/group?q=&sort=&page=1"' in res grp_numbers = scrape_search_results(res, 'group') assert_equal(['21'], grp_numbers)