Skip to content

Commit

Permalink
[#3022] Use dataset type when generating URLs on the search page
Browse files Browse the repository at this point in the history
This allows the Pager to generate URLs that point to the relevant
dataset type page, and not /dataset.
  • Loading branch information
amercader committed Dec 19, 2012
1 parent 3577abf commit 0d95e4a
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions ckan/controllers/package.py
Expand Up @@ -50,8 +50,11 @@ def url_with_params(url, params):
return url + u'?' + urlencode(params)


def search_url(params):
url = h.url_for(controller='package', action='search')
def search_url(params, package_type=None):
if not package_type or package_type == 'dataset':
url = h.url_for(controller='package', action='search')
else:
url = h.url_for('{0}_search'.format(package_type))
return url_with_params(url, params)


Expand Down Expand Up @@ -175,7 +178,7 @@ def _sort_by(fields):
if fields:
sort_string = ', '.join('%s %s' % f for f in fields)
params.append(('sort', sort_string))
return search_url(params)
return search_url(params, package_type)

c.sort_by = _sort_by
if sort_by is None:
Expand All @@ -188,7 +191,7 @@ def _sort_by(fields):
def pager_url(q=None, page=None):
params = list(params_nopage)
params.append(('page', page))
return search_url(params)
return search_url(params, package_type)

c.search_url_params = urlencode(_encode_params(params_nopage))

Expand Down

0 comments on commit 0d95e4a

Please sign in to comment.