diff --git a/ckan/controllers/package.py b/ckan/controllers/package.py index 834b3ba38b0..872c6dd75db 100644 --- a/ckan/controllers/package.py +++ b/ckan/controllers/package.py @@ -264,8 +264,14 @@ def pager_url(q=None, page=None): c.page = h.Page(collection=[]) c.search_facets_limits = {} for facet in c.search_facets.keys(): - limit = int(request.params.get('_%s_limit' % facet, - g.facets_default_number)) + try: + limit = int(request.params.get('_%s_limit' % facet, + g.facets_default_number)) + except ValueError: + abort(400, _('Parameter "{parameter_name}" is not ' + 'an integer').format( + parameter_name='_%s_limit' % facet + )) c.search_facets_limits[facet] = limit maintain.deprecate_context_item(