[openstreetmap/openstreetmap-website] Clean up query related code (PR #6102)

[Marwin Hochfelsner]

Building on the changes from #6076 that didn't involve ripping out leaflet for consistency and instead prepare the bounds explicitly.
And another few things that caught my eye there.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Reuse error printing
  * Fix bounding box coordinates in OSM.location test
  * Remove unused distance function and its tests
  * Drop unnecessary custom wrap function
  * Restructure featureGeometry for clarity
  * Clean up overpass merge function
  * Remove unused runQuery parameters
  * Inline latlng variable
  * Simplify overpass query building
  * Use combined latlng parameter
  * Ensure Overpass bounds compatibility

-- File Changes --

    M app/assets/javascripts/index/query.js (97)
    M app/assets/javascripts/leaflet.map.js (4)
    M app/assets/javascripts/osm.js.erb (15)
    M test/javascripts/osm_test.js (12)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/6102 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250615/3313ef75/attachment.htm>