[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