[openstreetmap/openstreetmap-website] Replace qs with URLSearchParams (PR #5592)

Holger Jeromin notifications at github.com
Sun Feb 2 12:29:52 UTC 2025


@HolgerJeromin commented on this pull request.



> @@ -45,34 +44,34 @@
  */
 window.updateLinks = function (loc, zoom, layers, object) {
   $(".geolink").each(function (index, link) {
-    var href = link.href.split(/[?#]/)[0],
-        args = Qs.parse(link.search.substring(1)),
-        editlink = $(link).hasClass("editlink");
+    let href = link.href.split(/[?#]/)[0];

Not sure if this is a good and correct change:
```suggestion
    let href = link.protocol + '//' + link.hostname + link.pathname;
```

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

Message ID: <openstreetmap/openstreetmap-website/pull/5592/review/2588542026 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250202/a5317a69/attachment.htm>


More information about the rails-dev mailing list