rm senderId from setCopayers

js/models/TxProposal.js

@@ -404,7 +404,7 @@ TxProposal.prototype.getSent = function() {
   return this.sentTs;
 }
 
-TxProposal.prototype.setCopayers = function(senderId, keyMap) {
+TxProposal.prototype.setCopayers = function(pubkeyToCopayerMap) {
   var newCopayer = {},
     oldCopayers = {},
     newSignedBy = {},
@@ -431,7 +431,7 @@ TxProposal.prototype.setCopayers = function(senderId, keyMap) {
 
   var iSig = this.getSignersPubKeys();
   for (var i in iSig) {
-    var copayerId = keyMap[iSig[i]];
+    var copayerId = pubkeyToCopayerMap[iSig[i]];
 
     if (!copayerId)
       throw new Error('Found unknown signature')
@@ -448,7 +448,8 @@ TxProposal.prototype.setCopayers = function(senderId, keyMap) {
     throw new Error('New TX must have only 1 new signature');
 
   // Handler creator / createdTs.
-  // from senderId, and must be signed by senderId
+  // from senderId, and must be signed by senderId * DISABLED*
+  //
   if (isNew) {
     this.creator = Object.keys(newCopayer)[0];
     this.seenBy[this.creator] = this.createdTs = Date.now();

js/models/Wallet.js

@@ -520,7 +520,7 @@ Wallet.prototype._onTxProposal = function(senderId, data) {
     }
 
     var keyMap = self._getPubkeyToCopayerMap(incomingTx);
-    incomingTx.setCopayers(senderId, keyMap);
+    incomingTx.setCopayers(keyMap);
 
     self.txProposals.add(incomingTx);
     self.emitAndKeepAlive('txProposalEvent', {

test/TxProposal.js

@@ -480,7 +480,7 @@ describe('TxProposal', function() {
         };
         delete txp['creator'];
         (function() {
-          txp.setCopayers('juan', {
+          txp.setCopayers({
             pk1: 'pepe'
           })
         }).should.throw('no creator');
@@ -495,7 +495,7 @@ describe('TxProposal', function() {
           ['pkX']
         ];
         (function() {
-          txp.setCopayers('juan', {
+          txp.setCopayers({
             pk1: 'pepe'
           })
         }).should.throw('creator');
@@ -512,7 +512,7 @@ describe('TxProposal', function() {
           ['pk0', 'pkX']
         ];
         (function() {
-          txp.setCopayers('juan', {
+          txp.setCopayers({
             pk1: 'pepe'
           })
         }).should.throw('unknown sig');
@@ -529,7 +529,7 @@ describe('TxProposal', function() {
           'creator': Date.now()
         };
         (function() {
-          txp.setCopayers('juan', {
+          txp.setCopayers({
             pk0: 'creator',
             pk1: 'pepe',
             pk2: 'john'
@@ -546,7 +546,7 @@ describe('TxProposal', function() {
         txp.signedBy = {
           'creator': Date.now()
         };
-        txp.setCopayers('pepe', {
+        txp.setCopayers({
           pk0: 'creator',
           pk1: 'pepe',
           pk2: 'john'
@@ -562,7 +562,7 @@ describe('TxProposal', function() {
         txp.signedBy = {};
         delete txp['creator'];
         delete txp['creatorTs'];
-        txp.setCopayers('creator', {
+        txp.setCopayers({
           pk0: 'creator',
           pk1: 'pepe',
           pk2: 'john'
@@ -581,7 +581,7 @@ describe('TxProposal', function() {
         sinon.stub(txp,'getSignersPubKeys').returns(['pk0', 'pk1']);
         (function() {
           txp.setCopayers(
-            'creator', {
+            {
               pk0: 'creator',
               pk1: 'pepe',
               pk2: 'john'
@@ -600,7 +600,7 @@ describe('TxProposal', function() {
         txp.signedBy = {
           'creator': 1
         };
-        txp.setCopayers('pepe', {
+        txp.setCopayers({
           pk0: 'creator',
           pk1: 'pepe',
           pk2: 'john'