From 79fd185230b5b874bf6990e5fdc699c74df7c43b Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Thu, 1 May 2014 10:07:30 -0300
Subject: [PATCH] better opts for get address

---
 js/models/core/PublicKeyRing.js | 22 +++++++++++++---------
 js/models/core/Wallet.js        | 10 +++++-----
 2 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/js/models/core/PublicKeyRing.js b/js/models/core/PublicKeyRing.js
index 6c19f3797..d9073c792 100644
--- a/js/models/core/PublicKeyRing.js
+++ b/js/models/core/PublicKeyRing.js
@@ -218,15 +218,17 @@ PublicKeyRing.prototype.generateAddress = function(isChange) {
   return ret;
 };
 
-PublicKeyRing.prototype.getAddresses = function(excludeChange) {
-  return this.getAddressesInfo(excludeChange).map(function(info) {
+PublicKeyRing.prototype.getAddresses = function(opts) {
+  return this.getAddressesInfo(opts).map(function(info) {
     return info.address;
   });
 };
 
-PublicKeyRing.prototype.getAddressesInfo = function(excludeChange) {
+PublicKeyRing.prototype.getAddressesInfo = function(opts) {
+  opts = opts || {};
+
   var ret = [];
-  if (!excludeChange) {
+  if (!opts.excludeChange) {
     for (var i=0; i<this.changeAddressIndex; i++) {
       ret.unshift({
         address: this.getAddress(i,true),
@@ -235,11 +237,13 @@ PublicKeyRing.prototype.getAddressesInfo = function(excludeChange) {
     }
   }
 
-  for (var i=0; i<this.addressIndex; i++) {
-    ret.unshift({
-      address: this.getAddress(i,false),
-      isChange: false
-    });
+  if (!opts.excludeMain) {
+    for (var i=0; i<this.addressIndex; i++) {
+      ret.unshift({
+        address: this.getAddress(i,false),
+        isChange: false
+      });
+    }
   }
 
   return ret;
diff --git a/js/models/core/Wallet.js b/js/models/core/Wallet.js
index 01238a763..8307baaa5 100644
--- a/js/models/core/Wallet.js
+++ b/js/models/core/Wallet.js
@@ -407,12 +407,12 @@ Wallet.prototype.addSeenToTxProposals = function() {
 };
 
 // TODO: remove this method and use getAddressesInfo everywhere
-Wallet.prototype.getAddresses = function(excludeChange) {
-  return this.publicKeyRing.getAddresses(excludeChange);
+Wallet.prototype.getAddresses = function(opts) {
+  return this.publicKeyRing.getAddresses(opts);
 };
 
-Wallet.prototype.getAddressesStr = function(excludeChange) {
-  return this.getAddresses(excludeChange).map(function(a) {
+Wallet.prototype.getAddressesStr = function(opts) {
+  return this.getAddresses(opts).map(function(a) {
     return a.toString();
   });
 };
@@ -422,7 +422,7 @@ Wallet.prototype.getAddressesInfo = function(excludeChange) {
 };
 
 Wallet.prototype.addressIsOwn = function(addrStr, excludeChange) {
-  var addrList = this.getAddressesStr(excludeChange);
+  var addrList = this.getAddressesStr({excludeMain:true});
   var l = addrList.length;
   var ret = false;