Merge pull request #19 from gabrielbazan7/fix/tapOnNot

remove token from localstorage and general refactor

public/views/preferencesGlobal.html

@@ -48,7 +48,7 @@
       <div translate>Use Unconfirmed Funds</div>
     </li>
   </ul>
-  <div ng-show="index.usePushNotifications">
+  <div ng-show="prefGlobal.usePushNotifications">
     <h4></h4>
     <ul class="no-bullet m0">
       <li>

src/js/controllers/preferencesGlobal.js

@@ -1,7 +1,7 @@
 'use strict';
 
 angular.module('copayApp.controllers').controller('preferencesGlobalController',
-  function($scope, $rootScope, $log, configService, uxLanguage, pushNotificationsService, profileService, feeService) {
+  function($scope, $rootScope, $log, configService, uxLanguage, isCordova, isMobile, pushNotificationsService, profileService, feeService) {
 
     this.init = function() {
       var config = configService.getSync();
@@ -13,6 +13,7 @@ angular.module('copayApp.controllers').controller('preferencesGlobalController',
       };
       this.feeOpts = feeService.feeOpts;
       this.currentFeeLevel = feeService.getCurrentFeeLevel();
+      this.usePushNotifications = isCordova && !isMobile.Windows();
       $scope.spendUnconfirmed = config.wallet.spendUnconfirmed;
       $scope.glideraEnabled = config.glidera.enabled;
       $scope.glideraTestnet = config.glidera.testnet;

src/js/services/profileService.js

@@ -150,17 +150,7 @@ angular.module('copayApp.services')
 
     root.pushNotificationsInit = function() {
       var defaults = configService.getDefaults();
-      var push = PushNotification.init(defaults.pushNotifications.config);
+      var push = pushNotificationsService.init(root.walletClients);
-
-      push.on('registration', function(data) {
-        storageService.getDeviceToken(function(err, token) {
-          if (token) return;
-          $log.debug('Starting push notification registration');
-          storageService.setDeviceToken(data.registrationId, function() {
-            pushNotificationsService.enableNotifications(root.walletClients);
-          });
-        });
-      });
 
       push.on('notification', function(data) {
         if (!data.additionalData.foreground) {
@@ -172,7 +162,7 @@ angular.module('copayApp.services')
             var walletToFind = data.additionalData.walletId;
 
             var walletFound = lodash.find(wallets, function(w) {
-              return (lodash.isEqual(walletToFind, sjcl.hash.sha256.hash(w.id)));
+              return (lodash.isEqual(walletToFind, sjcl.codec.hex.fromBits(sjcl.hash.sha256.hash(w.id))));
             });
 
             if (!walletFound) return $log.debug('Wallet not found');

src/js/services/pushNotificationsService.js

@@ -4,14 +4,27 @@ angular.module('copayApp.services')
     var root = {};
     var usePushNotifications = isCordova && !isMobile.Windows();
 
+    root.init = function(walletsClients) {
+      var defaults = configService.getDefaults();
+      var push = PushNotification.init(defaults.pushNotifications.config);
+
+      push.on('registration', function(data) {
+        if (root.token) return;
+        $log.debug('Starting push notification registration');
+        root.token = data.registrationId;
+        root.enableNotifications(walletsClients);
+      });
+
+      return push;
+    }
+
     root.enableNotifications = function(walletsClients) {
       if (!usePushNotifications) return;
 
       var config = configService.getSync();
       if (!config.pushNotifications.enabled) return;
 
-      storageService.getDeviceToken(function(err, token) {
+      if (!root.token) {
-        if (err || !token) {
         $log.warn('No token available for this device. Cannot set push notifications');
         return;
       }
@@ -19,13 +32,12 @@ angular.module('copayApp.services')
       lodash.forEach(walletsClients, function(walletClient) {
         var opts = {};
         opts.type = isMobile.iOS() ? "ios" : isMobile.Android() ? "android" : null;
-          opts.token = token;
+        opts.token = root.token;
         root.subscribe(opts, walletClient, function(err, response) {
           if (err) $log.warn('Subscription error: ' + err.message + ': ' + JSON.stringify(opts));
           else $log.debug('Subscribed to push notifications service: ' + JSON.stringify(response));
         });
       });
-      });
     }
 
     root.disableNotifications = function(walletsClients) {

src/js/services/storageService.js

@@ -231,14 +231,6 @@ angular.module('copayApp.services')
       storage.get('addressbook-' + network, cb);
     };
 
-    root.setDeviceToken = function(token, cb) {
-      storage.set('pushToken', token, cb);
-    }
-
-    root.getDeviceToken = function(cb) {
-      storage.get('pushToken', cb);
-    }
-
     root.removeAddressbook = function(network, cb) {
       storage.remove('addressbook-' + network, cb);
     };