show error message after click

public/views/modals/destination-address.html

@@ -58,13 +58,13 @@
       <div ng-if="!gettingAddress">
         <ul class="no-bullet">
           <li class="line-b" ng-repeat="w in wallets">
-            <a ng-click="w.needsBackup || selectWallet(w.id, w.name)" class="db oh">
+            <a ng-click="selectWallet(w.id, w.name)" class="db oh">
               <div class="avatar-wallet"
                 ng-style="{'background-color':w.color}">
                 <i class="icon-wallet size-21"></i>
               </div>
               <div class="ellipsis name-wallet text-bold">{{w.name || w.id}}
-                <span class="has-error right text-light size-12" ng-show="w.needsBackup">
+                <span class="has-error right text-light size-12" ng-show="needsBackup[w.id]">
                   <i class="icon-close-circle size-14"></i>
                   <span class="vm" translate>Needs backup</span>
                 </span>

src/js/controllers/index.js

@@ -128,7 +128,6 @@ angular.module('copayApp.controllers').controller('indexController', function($r
       self.isPrivKeyEncrypted = fc.isPrivKeyEncrypted();
       self.externalSource = fc.getPrivKeyExternalSourceName();
       self.account = fc.credentials.account;
-      self.needsBackup = profileService.isBackupNeeded(self.walletId);
 
       if (self.externalSource == 'trezor')
         self.account++;
@@ -153,16 +152,19 @@ angular.module('copayApp.controllers').controller('indexController', function($r
         }
       }
 
-      self.openWallet(function() {
+      profileService.isBackupNeeded(self.walletId, function(needsBackup) {
-        if (!self.isComplete) {
+        self.needsBackup = needsBackup;
-          $log.debug('Wallet not complete after update... redirecting');
+        self.openWallet(function() {
-          go.path('copayers');
+          if (!self.isComplete) {
-        } else {
+            $log.debug('Wallet not complete after update... redirecting');
-          if ($state.is('copayers')) {
+            go.path('copayers');
-            $log.debug('Wallet Complete after update... redirect to home');
+          } else {
-            go.walletHome();
+            if ($state.is('copayers')) {
+              $log.debug('Wallet Complete after update... redirect to home');
+              go.walletHome();
+            }
           }
-        }
+        });
       });
     });
   };
@@ -1489,7 +1491,7 @@ angular.module('copayApp.controllers').controller('indexController', function($r
     });
   });
 
-  //untilItChange FALSE
+  //untilItChange FALSE 
   lodash.each(['NewTxProposal', 'TxProposalFinallyRejected', 'TxProposalRemoved', 'NewOutgoingTxByThirdParty',
     'Local/GlideraTx'
   ], function(eventName) {

src/js/controllers/walletHome.js

@@ -246,21 +246,28 @@ angular.module('copayApp.controllers').controller('walletHomeController', functi
       };
 
       $scope.selectWallet = function(walletId, walletName) {
-        $scope.gettingAddress = true;
+        profileService.isBackupNeeded(walletId, function(needsBackup) {
-        $scope.selectedWalletName = walletName;
+          $scope.needsBackup = {};
-        $timeout(function() {
+          $scope.needsBackup[walletId] = needsBackup;
-          $scope.$apply();
+          if (needsBackup) return;
-        });
-        addressService.getAddress(walletId, false, function(err, addr) {
-          $scope.gettingAddress = false;
 
-          if (err) {
+          $scope.gettingAddress = true;
-            self.error = err;
+          $scope.selectedWalletName = walletName;
-            $modalInstance.dismiss('cancel');
+          $timeout(function() {
-            return;
+            $scope.$apply();
-          }
+          });
 
-          $modalInstance.close(addr);
+          addressService.getAddress(walletId, false, function(err, addr) {
+            $scope.gettingAddress = false;
+
+            if (err) {
+              self.error = err;
+              $modalInstance.dismiss('cancel');
+              return;
+            }
+
+            $modalInstance.close(addr);
+          });
         });
       };
     };

src/js/services/profileService.js

@@ -668,12 +668,16 @@ angular.module('copayApp.services')
       });
     };
 
-    root.isBackupNeeded = function(walletId) {
+    root.isBackupNeeded = function(walletId, cb) {
       var c = root.getClient(walletId);
-      if (c.isPrivKeyExternal()) return false;
+      if (c.isPrivKeyExternal()) return cb(false);
-      if (!c.credentials.mnemonic) return false;
+      if (!c.credentials.mnemonic) return cb(false);
-      if (c.credentials.network == 'testnet') return false;
+      if (c.credentials.network == 'testnet') return cb(false);
-      return true;
+
+      storageService.getBackupFlag(walletId, function(err, val) {
+        if (err || val) return cb(false);
+        return cb(true);
+      });
     };
 
     root.getWallets = function(network) {
@@ -690,8 +694,7 @@ angular.module('copayApp.services')
           id: c.walletId,
           network: c.network,
           color: config.colorFor[c.walletId] || '#4A90E2',
-          copayerId: c.copayerId,
+          copayerId: c.copayerId
-          needsBackup: root.isBackupNeeded(c.walletId)
         };
       });
       if (network) {