From ba8a0e83d2df060a6211e3a5c8bd78e07741a30d Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Fri, 18 Apr 2014 11:19:39 -0300
Subject: [PATCH 1/3] fix p2p connecting

---
 js/config.js                    | 10 +++++-----
 js/controllers/transactions.js  | 14 ++++++++------
 js/models/core/Wallet.js        | 16 ++++++++++++++++
 js/models/core/WalletFactory.js |  6 +++---
 test/test.Wallet.js             |  9 ++++++++-
 test/test.Walletfactory.js      |  6 ++++++
 6 files changed, 46 insertions(+), 15 deletions(-)

diff --git a/js/config.js b/js/config.js
index cf2793f1f..a16d9120f 100644
--- a/js/config.js
+++ b/js/config.js
@@ -3,12 +3,12 @@
 var config = {
   networkName: 'testnet',
   network: {
-    key: 'lwjd5qra8257b9',
+    // key: 'lwjd5qra8257b9',
     //   This is for running local peerJs with params: ./peerjs  -p 10009  -k 'sdfjhwefh'
-    // key: 'sdfjhwefh',
-    // host: 'localhost',
-    // port: 10009,
-    // path: '/',
+    key: 'sdfjhwefh',
+    host: 'localhost',
+    port: 10009,
+    path: '/',
     maxPeers: 3,
     debug: 3,
   },
diff --git a/js/controllers/transactions.js b/js/controllers/transactions.js
index 83b19a245..89476fce5 100644
--- a/js/controllers/transactions.js
+++ b/js/controllers/transactions.js
@@ -7,11 +7,13 @@ angular.module('copay.transactions').controller('TransactionsController',
     $scope.oneAtATime = true;
 
     var _updateTxs = function() {
-      var inT = $rootScope.wallet.getTxProposals();
-      var ts = [];
+      var w   =$rootScope.wallet;
+      var inT = w.getTxProposals();
+      var ts  = [];
+
       inT.forEach(function(i){
-        var b =i.txp.builder;
-        var tx = b.build();
+        var b   = i.txp.builder;
+        var tx  = b.build();
         var one = {
           valueOutSat: b.valueOutSat,
           feeSat: b.feeSat,
@@ -21,7 +23,8 @@ angular.module('copay.transactions').controller('TransactionsController',
         tx.outs.forEach(function(o) {
           var s = o.getScript();
           var aStr = bitcore.Address.fromScript(s, config.networkName).toString();
-          outs.push({address: aStr, value: bitcore.util.valueToBigInt(o.getValue())});
+          if (!w.addressIsOwn(aStr))
+            outs.push({address: aStr, value: bitcore.util.valueToBigInt(o.getValue())});
         });
         one.outs = outs;
         ts.push(one);
@@ -42,7 +45,6 @@ angular.module('copay.transactions').controller('TransactionsController',
     $scope.sign = function (ntxid) {
       var w = $rootScope.wallet;
       var ret = w.sign(ntxid);
-console.log('[transactions.js.28:ret:]',ret); //TODO
       $rootScope.flashMessage = {type:'success', message: 'Transactions SEND! : ' + ret};
       _updateTxs();
     };
diff --git a/js/models/core/Wallet.js b/js/models/core/Wallet.js
index a2ed268e2..c0f01854b 100644
--- a/js/models/core/Wallet.js
+++ b/js/models/core/Wallet.js
@@ -310,6 +310,22 @@ Wallet.prototype.getAddressesStr = function(onlyMain) {
   return ret;
 };
 
+
+Wallet.prototype.addressIsOwn = function(addrStr) {
+  var addrList = this.getAddressesStr();
+  var l = addrList.length;
+  var ret = false;
+
+  for(var i=0; i<l; i++) {
+    if (addrList[i] === addrStr) {
+      ret = true; 
+      break;
+    }
+  }
+  return ret;
+};
+
+
 Wallet.prototype.getTotalBalance = function(cb) {
   this.getBalance(this.getAddressesStr(), function(balance) {
     return cb(balance);
diff --git a/js/models/core/WalletFactory.js b/js/models/core/WalletFactory.js
index 1b1c209e0..a3e0a4475 100644
--- a/js/models/core/WalletFactory.js
+++ b/js/models/core/WalletFactory.js
@@ -44,12 +44,12 @@ WalletFactory.prototype._checkRead = function(walletId) {
     s.get(walletId, 'opts') &&
     s.get(walletId, 'privateKey')
   ;
-  return ret;
+  return ret?true:false;
 };
 
 WalletFactory.prototype.read = function(walletId) {
   if (! this._checkRead(walletId))
-    throw Error('Check read failed');
+    return false;
 
   var s = this.storage;
   var opts = s.get(walletId, 'opts');
@@ -175,7 +175,7 @@ WalletFactory.prototype.connectTo = function(peerId, cb) {
   self.network.start(function() {
     self.network.connectTo(peerId)
     self.network.on('walletId', function(walletId) {
-console.log('[WalletFactory.js.187]'); //TODO
+      self.log('Opening walletId:' + walletId);
       return cb(self.open(walletId));
     });
   });
diff --git a/test/test.Wallet.js b/test/test.Wallet.js
index f7eaad05a..9b3582cce 100644
--- a/test/test.Wallet.js
+++ b/test/test.Wallet.js
@@ -133,7 +133,14 @@ describe('Wallet model', function() {
     Object.keys(t.txps[0].seenBy).length.should.equal(1);
   });
 
-
+  it('#addressIsOwn', function () {
+    var w = createW2();
+    var l = w.getAddressesStr();
+    for (var i=0; i<l.length; i++)
+      w.addressIsOwn(l[i]).should.equal(true);
+    w.addressIsOwn('mmHqhvTVbxgJTnePa7cfweSRjBCy9bQQXJ').should.equal(false);
+    w.addressIsOwn('mgtUfP9sTJ6vPLoBxZLPEccGpcjNVryaCX').should.equal(false);
+  });
 
   it('#create. Signing with derivate keys', function () {
 
diff --git a/test/test.Walletfactory.js b/test/test.Walletfactory.js
index 38fe165f8..21160a0dd 100644
--- a/test/test.Walletfactory.js
+++ b/test/test.Walletfactory.js
@@ -39,6 +39,12 @@ describe('WalletFactory model', function() {
       var wf = new WalletFactory(config);
       should.exist(wf);
     });
+    it('#_checkRead should return fail', function() {
+      var wf = new WalletFactory(config);
+      wf._checkRead('dummy').should.equal(false);
+      wf.read('dummy').should.equal(false);
+    });
+ 
     it('should be able to create wallets', function() {
       var wf = new WalletFactory(config);
       var w = wf.create();

From 77a662467be17abf5839b84e917c0a0f25bf1919 Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Fri, 18 Apr 2014 11:23:07 -0300
Subject: [PATCH 2/3] remote server

---
 js/config.js | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/js/config.js b/js/config.js
index a16d9120f..c5dde3556 100644
--- a/js/config.js
+++ b/js/config.js
@@ -3,12 +3,12 @@
 var config = {
   networkName: 'testnet',
   network: {
-    // key: 'lwjd5qra8257b9',
+     key: 'lwjd5qra8257b9',
     //   This is for running local peerJs with params: ./peerjs  -p 10009  -k 'sdfjhwefh'
-    key: 'sdfjhwefh',
-    host: 'localhost',
-    port: 10009,
-    path: '/',
+    // key: 'sdfjhwefh',
+    // host: 'localhost',
+    // port: 10009,
+    // path: '/',
     maxPeers: 3,
     debug: 3,
   },

From 40ef2bc23e0ac9051555dbd3534cb52fe41beed3 Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Fri, 18 Apr 2014 11:23:45 -0300
Subject: [PATCH 3/3] .

---
 test/test.Walletfactory.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test.Walletfactory.js b/test/test.Walletfactory.js
index 21160a0dd..98157d952 100644
--- a/test/test.Walletfactory.js
+++ b/test/test.Walletfactory.js
@@ -39,7 +39,7 @@ describe('WalletFactory model', function() {
       var wf = new WalletFactory(config);
       should.exist(wf);
     });
-    it('#_checkRead should return fail', function() {
+    it('#_checkRead should return false', function() {
       var wf = new WalletFactory(config);
       wf._checkRead('dummy').should.equal(false);
       wf.read('dummy').should.equal(false);