Improved data sanitation for the Pinned items

FMaz008 · Oct 5, 2024 · 83c80c1 · 83c80c1
1 parent 2e634e0
commit 83c80c1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/scripts/Grid.js b/scripts/Grid.js
@@ -210,6 +210,10 @@ async function createGridInterface() {
 }
 
 async function addPinnedTile(asin, queue, title, thumbnail, is_parent_asin, enrollment_guid) {
+	if (!asin) {
+		return false;
+	}
+
 	//Check if the pin already exist:
 	if (document.getElementById("vh-pin-" + asin) != undefined) return false;
 

diff --git a/scripts/PinnedListMgr.js b/scripts/PinnedListMgr.js
@@ -85,6 +85,10 @@ class PinnedListMgr {
 	}
 
 	async addItem(asin, queue, title, thumbnail, isParentAsin, enrollmentGUID, save = true, broadcast = true) {
+		if (!asin || !queue || !title || !thumbnail || !isParentAsin || !enrollmentGUID) {
+			throw new Error("Invalid data");
+		}
+
 		if (save) await this.loadFromLocalStorage(); //Load the list in case it was altered in a different tab
 
 		this.mapPin.set(asin, {
@@ -145,6 +149,10 @@ class PinnedListMgr {
 		}
 	}
 
+	async wipe() {
+		let storableVal = JSON.stringify([]);
+		await browser.storage.local.set({ pinnedItems: storableVal });
+	}
 	/**
 	 * Send new items on the server to be added or removed from the changed list.
 	 */