maint(pat slides): Use URL web API instead custom url api from core.

Author: thet

src/pat/slides/slides.js

@@ -6,7 +6,6 @@
 import $ from "jquery";
 import registry from "../../core/registry";
 import utils from "../../core/utils";
-import url from "../../core/url";
 import "../../core/remove";
 
 var slides = {
@@ -24,10 +23,12 @@ var slides = {
         }
         await import("slides/src/slides"); // loads ``Presentation`` globally.
 
-        var parameters = url.parameters();
-        if (parameters.slides !== undefined) {
-            var requested_ids = slides._collapse_ids(parameters.slides);
-            if (requested_ids) slides._remove_slides($el, requested_ids);
+        const slides_filter = new URL(window.location).searchParams.get("slides");
+        if (slides_filter) {
+            const requested_ids = slides._collapse_ids(slides_filter);
+            if (requested_ids) {
+                slides._remove_slides($el, requested_ids);
+            }
         }
         $el.each(function () {
             var presentation = new window.Presentation(this),
@@ -61,17 +62,8 @@ var slides = {
         });
     },
 
-    _collapse_ids: function (params) {
-        var ids = [];
-        params.forEach(function (param) {
-            if (param)
-                ids = ids.concat(
-                    param.split(",").filter(function (id) {
-                        return !!id;
-                    })
-                );
-        });
-        return ids;
+    _collapse_ids: function (id_string) {
+        return (id_string || "").split(",").filter((it) => !!it);
     },
 
     _remove_slides: function ($shows, ids) {

src/pat/slides/slides.test.js

@@ -10,27 +10,23 @@ describe("pat-slides", function () {
 
     describe("_collapse_ids", function () {
         it("Single id", function () {
-            expect(pattern._collapse_ids(["foo"])).toEqual(["foo"]);
+            expect(pattern._collapse_ids("foo")).toEqual(["foo"]);
         });
 
         it("Comma-separated list of ids", function () {
-            expect(pattern._collapse_ids(["foo,bar"])).toEqual(["foo", "bar"]);
+            expect(pattern._collapse_ids("foo,bar")).toEqual(["foo", "bar"]);
         });
 
         it("Skip empty ids", function () {
-            expect(pattern._collapse_ids(["foo,,bar"])).toEqual(["foo", "bar"]);
+            expect(pattern._collapse_ids("foo,,bar")).toEqual(["foo", "bar"]);
         });
 
         it("Parameter without value", function () {
-            expect(pattern._collapse_ids([null])).toEqual([]);
+            expect(pattern._collapse_ids(null)).toEqual([]);
         });
 
         it("Parameter with empty value", function () {
-            expect(pattern._collapse_ids([""])).toEqual([]);
-        });
-
-        it("Multiple parameters", function () {
-            expect(pattern._collapse_ids(["foo", "bar"])).toEqual(["foo", "bar"]);
+            expect(pattern._collapse_ids("")).toEqual([]);
         });
     });