Better issue2801 Conditionals

- currency-access.ts should also catch nulls, while 0's are OK.
- currency-manager.ts is much more concise with a boolean nullish.
- As-is currency.service.js...
- ... which also had two lint warnings in it that got buffed out.

src/backend/currency/currency-access.ts

@@ -52,7 +52,7 @@ class CurrencyAccess {
         let issue2801 = false;
         const cache = db.getData("/");
         Object.keys(cache).forEach((currencyId) => {
-            if (cache[currencyId].offline === "") {
+            if (cache[currencyId].offline === null || cache[currencyId].offline === "") {
                 issue2801 = true;
                 cache[currencyId].offline = undefined;
             }

src/backend/currency/currency-manager.ts

@@ -263,7 +263,7 @@ class CurrencyManager {
         for (const currency of currencies) {
             let basePayout = currency.payout;
             if (!connectionManager.streamerIsOnline()) {
-                if (currency.offline == null || currency.offline === 0) {
+                if (!currency.offline) {
                     continue;
                 }
 

src/gui/app/services/currency.service.js

@@ -107,12 +107,12 @@
             ipcRenderer.send("refreshCurrencyCommands", {"action": "delete", "currency": currency});
         };
 
-        backendCommunicator.on("import-currency", currency => {
+        backendCommunicator.on("import-currency", (currency) => {
             if (currency == null || currency.id == null) {
                 return;
             }
 
-            if (currency.offline === "") {
+            if (!currency.offline) {
                 currency.offline = undefined;
             }
 
@@ -123,7 +123,7 @@
             }
         });
 
-        backendCommunicator.on("remove-currency", currency => {
+        backendCommunicator.on("remove-currency", (currency) => {
             service.deleteCurrency(currency);
         });