okfn/ckan — pull request #1258 by joetsoi (2013-11-05 11:00:07+00:00)
Title: ValueError for non-int limit and offset query params

[These lines](https://github.com/okfn/ckan/blob/e3c3e23a7e1c8e28fbd776e5d8c26a371ad7c76c/ckan/logic/action/get.py#L2069-2071) obviously cause problems for non-int query params. This hit me as a 500 in production.

Since this is at least the third time for this kind of error to pop up, I'm strongly suggesting a proper pagination implementation to be used across resources. Something along the lines of:

```python
pagination = get_pagination(data_dict, max_limit=config.get('ckan.activity_list_limit', 31))
activity_objects = model.activity.user_activity_list(user.id, **pagination.get_kwargs())
```

There are many other places in `get.py` where this is a problem.