Merge pull request #8 from trosck/fix-set-all-method

Fix a bug in the setAll method for undefined properties.
trosck · Jan 12, 2023 · 32afbc5 · 32afbc5
2 parents 70b8461 + e9487fe
commit 32afbc5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -51,11 +51,6 @@ urlParams.setAll({
     - [toString](#tostring)
   - [License](#license)
 
-<!-- 
-## Features
-  - **vue-mixin.js** for Vue that reactively updates
-  values in url -->
-
 ## Browser support
 ![Chrome](https://raw.githubusercontent.com/alrra/browser-logos/main/src/chrome/chrome_48x48.png) | ![Firefox](https://raw.githubusercontent.com/alrra/browser-logos/main/src/firefox/firefox_48x48.png) | ![Safari](https://raw.githubusercontent.com/alrra/browser-logos/main/src/safari/safari_48x48.png) | ![Opera](https://raw.githubusercontent.com/alrra/browser-logos/main/src/opera/opera_48x48.png) | ![Edge](https://raw.githubusercontent.com/alrra/browser-logos/main/src/edge/edge_48x48.png) |
 --- | --- | --- | --- | --- |

diff --git a/index.ts b/index.ts
@@ -144,7 +144,9 @@ export class URLParams implements IURLParams {
   ) {
     for (const key in properties) {
       if (properties.hasOwnProperty(key)) {
-        this._url.searchParams.set(key, properties[key].toString())
+        this._url.searchParams.set(
+          key, properties[key]?.toString?.() ?? ""
+        )
       }
     }
     changeUrl(this._url, saveState)