From dfae2d231e39c81827d08fd8834d736c5b3005b1 Mon Sep 17 00:00:00 2001 From: Vijay A Date: Tue, 31 Aug 2021 16:25:57 +0530 Subject: refactor: Use Ferdi.safeParseInt to ensure that parsing is done consistently in all recipes. --- recipes/ciscospark/package.json | 2 +- recipes/ciscospark/webview.js | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'recipes/ciscospark') diff --git a/recipes/ciscospark/package.json b/recipes/ciscospark/package.json index ec9f3c8..dbf031e 100644 --- a/recipes/ciscospark/package.json +++ b/recipes/ciscospark/package.json @@ -1,7 +1,7 @@ { "id": "ciscospark", "name": "Cisco Spark", - "version": "1.1.0", + "version": "1.1.1", "license": "MIT", "config": { "serviceURL": "https://web.ciscospark.com", diff --git a/recipes/ciscospark/webview.js b/recipes/ciscospark/webview.js index 4de3fb1..8de5b8d 100644 --- a/recipes/ciscospark/webview.js +++ b/recipes/ciscospark/webview.js @@ -11,10 +11,10 @@ module.exports = Ferdi => { const elements = document.querySelectorAll('.navigation-bar-list .listItemWrapper'); if (elements.length > 0 && elements[1].querySelector(UNREAD_BADGE_SELECTOR)) { - directCount = parseInt(elements[1].querySelector(UNREAD_BADGE_SELECTOR).textContent, 10); + directCount = Ferdi.safeParseInt(elements[1].querySelector(UNREAD_BADGE_SELECTOR).textContent); } - if (elements.length > 0 && elements[2].querySelector(UNREAD_BADGE_SELECTOR)) { - indirectCount = parseInt(elements[2].querySelector(UNREAD_BADGE_SELECTOR).textContent, 10); + if (elements.length > 1 && elements[2].querySelector(UNREAD_BADGE_SELECTOR)) { + indirectCount = Ferdi.safeParseInt(elements[2].querySelector(UNREAD_BADGE_SELECTOR).textContent); } Ferdi.setBadge(directCount, indirectCount); -- cgit v1.2.3-70-g09d2