Merge pull request #383 from JDonadio/fix/edit-note

Fix edit memo

package.json

@@ -24,7 +24,7 @@
     "angular-mocks": "1.4.10",
     "bezier-easing": "^2.0.3",
     "bhttp": "^1.2.1",
-    "bitcore-wallet-client": "4.2.1",
+    "bitcore-wallet-client": "4.3.1",
     "bower": "^1.7.9",
     "chai": "^3.5.0",
     "cordova": "5.4.1",

src/js/controllers/modals/txDetails.js

@@ -27,16 +27,26 @@ angular.module('copayApp.controllers').controller('txDetailsController', functio
   };
 
   function updateMemo() {
-    wallet.getTxNote({
-      txid: $scope.btx.txid
-    }, function(err, note) {
-      if (err || !note) {
-        $log.debug(gettextCatalog.getString('Could not fetch transaction note'));
+    walletService.getTxNote(wallet, $scope.btx.txid, function(err, note) {
+      if (err) {
+        $log.warn('Could not fetch transaction note ' + err);
         return;
       }
-      $scope.note = note;
-      $timeout(function() {
-        $scope.$apply();
+
+      if (!note) return;
+
+      $scope.btx.note = note;
+
+      walletService.getTx(wallet, $scope.btx.txid, function(err, tx) {
+        if (err) {
+          $log.error(err);
+          return;
+        }
+
+        tx.note = note;
+        $timeout(function() {
+          $scope.$apply();
+        });
       });
     });
   };
@@ -91,19 +101,13 @@ angular.module('copayApp.controllers').controller('txDetailsController', functio
         body: text
       };
 
-      wallet.editTxNote(args, function(err) {
+      walletService.editTxNote(wallet, args, function(err, res) {
         if (err) {
-          $log.debug('Could not save tx comment');
+          $log.debug('Could not save tx comment ' + err);
           return;
         }
         // This is only to refresh the current screen data
-        $scope.btx.note = null;
-        if (args.body) {
-          $scope.btx.note = {};
-          $scope.btx.note.body = text;
-          $scope.btx.note.editedByName = wallet.credentials.copayerName;
-          $scope.btx.note.editedOn = Math.floor(Date.now() / 1000);
-        }
+        updateMemo();
         $scope.btx.searcheableString = null;
         $timeout(function() {
           $scope.$apply();

src/js/services/walletService.js

@@ -512,6 +512,12 @@ angular.module('copayApp.services').factory('walletService', function($log, $tim
     });
   };
 
+  root.editTxNote = function(wallet, args, cb) {
+    wallet.editTxNote(args, function(err, res) {
+      return cb(err, res);
+    });
+  };
+
   root.getTxp = function(wallet, txpid, cb) {
     wallet.getTx(txpid, function(err, txp) {
       if (err) return cb(err);

www/views/walletDetails.html

@@ -130,7 +130,7 @@
       </div>
 
       <div class="card list" ng-show="txHistory[0]">
-        <div class="item" ng-repeat="btx in txHistory track by btx.txid" ng-click="openTxModal(btx)">
+        <div class="item" ng-repeat="btx in txHistory track by $index" ng-click="openTxModal(btx)">
           <span class="item-note text-right">
             <span class="size-16"  ng-class="{'text-bold': btx.recent}">
               <span ng-if="btx.action == 'received'">+</span>