@AntonKhorev commented on this pull request.


> @@ -121,6 +135,30 @@ OSM = {
     return mapParams;
   },
 
+  parseGeoURI: function (geoURI) {
+    if (typeof geoURI !== "string") return;
+    let url;
+    try {
+      url = new URL(geoURI.toLowerCase());
+    } catch (e) { return; }
+    if (url.protocol !== "geo:" || !url.pathname) return;
+    const [path, ...params] = url.pathname.split(";");
+    let coords;
+    try {
+      coords = L.latLng(path.split(","));
+    } catch (e) { return; }
+    if (!coords) return;
+    const searchParams = new URLSearchParams(params.join("&"));

@Zverik I said "to move ... here", obviously from line 142.

If anyone knows about any implementation that treats `z` as case-insensitive, 
please tell. There's no formal specification for it as far as I know.

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

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

Reply via email to