Trying to restore an account but not finishing up the process yields redis:nil error upon refresh

Closes https://code.vereign.com/code/dashboard/issues/303

javascript/src/iframe/viamapi-iframe.js

@@ -1742,7 +1742,9 @@ const connection = Penpal.connectToParent({
                 delete executeResult.data["Uuid"];
                 delete executeResult.data["Session"];
                 getProfileData(loginIdentity).then(executeResult1 => {
-                  result(executeResult);
+                  setIdentityInLocalStorage(loginIdentity).then(() => {
+                    result(executeResult);
+                  });
                 });
               } else {
                 result(executeResult);
@@ -1906,12 +1908,7 @@ const connection = Penpal.connectToParent({
         viamApi.setIdentity(restoreAccessIdentity.authentication.publicKey);
 
         executeRestfulFunction("public", viamApi, viamApi.identityRestoreAccess, identificator).then(executeResult => {
-          if (executeResult.code === "200") {
-            setIdentityInLocalStorage(restoreAccessIdentity);
             result(executeResult);
-          } else {
-            result(executeResult);
-          }
         });
       });
     },

[Viktor Popov]

Are you sure this should be executeResult and not executeResult1? Also consider proper naming, may be something like 'setIdentityResult' or similar.

Other than that I'm ok with the change.

[Alexey Lunin]

Yep. I checked, it should be exactly executeResult. result renamed to setIdentityResult

[Viktor Popov]

What I meant is just to rename 'executeResult1' to something more meaningful, because numbers in variable names is a very bad practice, but I see that this is not written by you.