This takes #5585 a bit further and drops the separate visibility filtering in 
favour of ensuring that we always filter for that as part of the initial 
database query.

Technically this creates a slight change in behaviour in that previous nodes 
which were visible but outside the bounding box would be returned if they were 
part of an invisible way that had nodes inside the bounding box but that seems 
like it was more of a bug than a feature.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5589

-- Commit Summary --

  * Filter out invisible nodes and ways during initial selection

-- File Changes --

    M app/controllers/api/maps_controller.rb (16)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5589.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5589.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5589
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to