refactor: Use Ferdi.safeParseInt to ensure that parsing is done consistently in all recipes.

1 files changed, 8 insertions, 5 deletions

diff --git a/recipes/github/webview.js b/recipes/github/webview.js
index 9f35b5e..fdc65a8 100644
--- a/recipes/github/webview.js
+++ b/recipes/github/webview.js
@@ -1,14 +1,17 @@
 module.exports = Ferdi => {
   const getMessages = function getMessages() {
     const directCountElement = document.querySelector('.filter-list .count');
-    const indirectCountElement = document.querySelector('[class*="mail-status unread"]');
-    let directCount;
-    let indirectCount;
+    let directCount = 0;
     if (directCountElement) {
-      directCount = parseInt(directCountElement.innerHTML, 10);
-    } else if (indirectCountElement) {
+      directCount = Ferdi.safeParseInt(directCountElement.innerHTML);
+    }
+
+    const indirectCountElement = document.querySelector('[class*="mail-status unread"]');
+    let indirectCount = 0;
+    if (indirectCountElement) {
       indirectCount = 1;
     }
+
     Ferdi.setBadge(directCount, indirectCount);
   };