Add idIsDefined to SessionIdEditor

5k-mirrors · Sep 11, 2019 · e84368a · e84368a
1 parent 4ee7d33
commit e84368a
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/app/renderer/components/Screens/Account/LoggedIn/SessionIdEditor/SessionIdEditor.js b/app/renderer/components/Screens/Account/LoggedIn/SessionIdEditor/SessionIdEditor.js
@@ -11,18 +11,22 @@ import ButtonWithSuccessIcon from "../../../../UI/ButtonWithSuccessIcon/ButtonWi
 
 const sessionIdEditor = () => {
   const [poeSessionId, setPoeSessionId] = useState(
-    globalStore.get(storeKeys.POE_SESSION_ID, null)
+    globalStore.get(storeKeys.POE_SESSION_ID)
   );
   const [
     successIconIsVisible,
     displaySuccessIcon,
     hideSuccessIconAfterMsElapsed,
   ] = customHooks.useDisplay();
 
+  function idIsDefined() {
+    return poeSessionId !== "" && typeof poeSessionId !== "undefined";
+  }
+
   function onSave() {
     globalStore.set(
       storeKeys.POE_SESSION_ID,
-      poeSessionId === "" ? null : poeSessionId
+      idIsDefined() ? poeSessionId : null
     );
 
     displaySuccessIcon();