[openstreetmap-website] Creating a user configurable limit for results of a changeset query (#535)
Tom Hughes
notifications at github.com
Thu Dec 5 13:51:12 UTC 2013
> @@ -202,6 +202,12 @@ def query
> bbox = BoundingBox.from_bbox_params(params)
> end
>
> + if params[:limit] and params[:limit].to_i > 0 and params[:limit].to_i < 10000
> + limit = params[:limit].to_i
> + else
> + limit = 100
Obviously I didn't notice it in the notes stuff, but I don't think it's ideal to just ignore an out of range value, no.
---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/535/files#r8128889
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20131205/a8b221c6/attachment.html>
More information about the rails-dev
mailing list