From 091dcf2500e356aba494f4805212773f7a40d9ae Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 4 May 2018 15:28:47 -0300
Subject: [PATCH 1/5] Fix wallets order

---
 src/pages/add/create-wallet/create-wallet.ts       |  1 -
 src/pages/add/import-wallet/import-wallet.ts       |  1 -
 src/pages/add/join-wallet/join-wallet.ts           |  1 -
 src/pages/onboarding/onboarding.html               |  4 ++--
 src/pages/onboarding/tour/tour.ts                  |  1 -
 src/pages/settings/bitcoin-cash/bitcoin-cash.ts    |  1 -
 src/providers/persistence/persistence.ts           | 14 ++++++++++----
 src/providers/persistence/storage/file-storage.ts  |  5 +----
 src/providers/persistence/storage/local-storage.ts |  8 ++------
 src/providers/profile/profile.ts                   | 14 +++++---------
 10 files changed, 20 insertions(+), 30 deletions(-)

diff --git a/src/pages/add/create-wallet/create-wallet.ts b/src/pages/add/create-wallet/create-wallet.ts
index 5549b092c6b..b810c0c0306 100644
--- a/src/pages/add/create-wallet/create-wallet.ts
+++ b/src/pages/add/create-wallet/create-wallet.ts
@@ -201,7 +201,6 @@ export class CreateWalletPage implements OnInit {
       this.events.publish('status:updated');
       this.walletProvider.updateRemotePreferences(wallet);
       this.pushNotificationsProvider.updateSubscription(wallet);
-      this.profileProvider.setWalletOrder(wallet.credentials.walletId, null, wallet.coin);
 
       if (this.createForm.value.selectedSeed == 'set') {
         this.profileProvider.setBackupFlag(wallet.credentials.walletId);
diff --git a/src/pages/add/import-wallet/import-wallet.ts b/src/pages/add/import-wallet/import-wallet.ts
index 0eebb744163..1bc292bf4e5 100644
--- a/src/pages/add/import-wallet/import-wallet.ts
+++ b/src/pages/add/import-wallet/import-wallet.ts
@@ -208,7 +208,6 @@ export class ImportWalletPage {
     this.walletProvider.updateRemotePreferences(wallet).then(() => {
       this.profileProvider.setBackupFlag(wallet.credentials.walletId);
       this.events.publish('status:updated');
-      this.profileProvider.setWalletOrder(wallet.credentials.walletId, null, wallet.coin);
       this.pushNotificationsProvider.updateSubscription(wallet);
       if (this.fromOnboarding) {
         this.profileProvider.setOnboardingCompleted().then(() => {
diff --git a/src/pages/add/join-wallet/join-wallet.ts b/src/pages/add/join-wallet/join-wallet.ts
index 874a6721d90..1c32fe40f8f 100644
--- a/src/pages/add/join-wallet/join-wallet.ts
+++ b/src/pages/add/join-wallet/join-wallet.ts
@@ -153,7 +153,6 @@ export class JoinWalletPage {
       this.onGoingProcessProvider.clear();
       this.events.publish('status:updated');
       this.walletProvider.updateRemotePreferences(wallet);
-      this.profileProvider.setWalletOrder(wallet.credentials.walletId, null, wallet.coin);
       this.pushNotificationsProvider.updateSubscription(wallet);
       if (!wallet.isComplete()) {
         this.navCtrl.popToRoot();
diff --git a/src/pages/onboarding/onboarding.html b/src/pages/onboarding/onboarding.html
index 8a9052ebd5b..f57b8e66556 100644
--- a/src/pages/onboarding/onboarding.html
+++ b/src/pages/onboarding/onboarding.html
@@ -1,7 +1,7 @@
 <ion-content no-bounce>
   <div class="logo-tagline">
     <img src='assets/img/app/logo-negative.svg' />
-    <p [ngClass]="{'primary-color': !isCopay, 'comment-color': isCopay}" translate>Take control of your money, get started with bitcoin.</p>
+    <p padding [ngClass]="{'primary-color': !isCopay, 'comment-color': isCopay}" translate>Take control of your money, get started with bitcoin.</p>
   </div>
   <div class="bottom-absolute">
     <button ion-button class="button-standard e2e-get-started" (click)="getStarted()">
@@ -11,4 +11,4 @@
       {{'Restore from backup' | translate}}
     </button>
   </div>
-</ion-content>
\ No newline at end of file
+</ion-content>
diff --git a/src/pages/onboarding/tour/tour.ts b/src/pages/onboarding/tour/tour.ts
index e6ea2c4ebd7..0a526ac68ca 100644
--- a/src/pages/onboarding/tour/tour.ts
+++ b/src/pages/onboarding/tour/tour.ts
@@ -72,7 +72,6 @@ export class TourPage {
     this.profileProvider.createDefaultWallet().then((wallet) => {
       this.onGoingProcessProvider.clear();
       this.persistenceProvider.setOnboardingCompleted();
-      this.profileProvider.setWalletOrder(wallet.credentials.walletId, null, wallet.coin);
       this.navCtrl.push(CollectEmailPage, { walletId: wallet.id });
     })
   }
diff --git a/src/pages/settings/bitcoin-cash/bitcoin-cash.ts b/src/pages/settings/bitcoin-cash/bitcoin-cash.ts
index 75dcae637dd..de2dab43a4c 100644
--- a/src/pages/settings/bitcoin-cash/bitcoin-cash.ts
+++ b/src/pages/settings/bitcoin-cash/bitcoin-cash.ts
@@ -177,7 +177,6 @@ export class BitcoinCashPage {
 
         this.walletProvider.updateRemotePreferences(newWallet);
         this.pushNotificationsProvider.updateSubscription(newWallet);
-        this.profileProvider.setWalletOrder(newWallet.credentials.walletId, null, newWallet.coin);
 
         addCopayers(newWallet, isNew, (err) => {
           this.onGoingProcessProvider.clear();
diff --git a/src/providers/persistence/persistence.ts b/src/providers/persistence/persistence.ts
index 9630920572e..a83bd06e982 100644
--- a/src/providers/persistence/persistence.ts
+++ b/src/providers/persistence/persistence.ts
@@ -37,6 +37,7 @@ const Keys = {
   REMOTE_PREF_STORED: 'remotePrefStored',
   TX_CONFIRM_NOTIF: txid => 'txConfirmNotif-' + txid,
   TX_HISTORY: walletId => 'txsHistory-' + walletId,
+  ORDER_WALLET: walletId => 'order-' + walletId
 };
 
 interface Storage {
@@ -346,7 +347,8 @@ export class PersistenceProvider {
   removeAllWalletData(walletId: string): Promise<void> {
     return this.clearLastAddress(walletId)
       .then(() => this.removeTxHistory(walletId))
-      .then(() => this.clearBackupFlag(walletId));
+      .then(() => this.clearBackupFlag(walletId))
+      .then(() => this.removeWalletOrder(walletId));
   };
 
   setAmazonGiftCards(network: string, gcs: any): Promise<void> {
@@ -503,12 +505,16 @@ export class PersistenceProvider {
     return this.storage.remove('shapeShift-' + network);
   };
 
-  setWalletOrder(walletId: string, index: number): Promise<void> {
-    return this.storage.set('order-' + walletId, index);
+  setWalletOrder(walletId: string, order): Promise<void> {
+    return this.storage.set(Keys.ORDER_WALLET(walletId), order);
   };
 
   getWalletOrder(walletId: string): Promise<void> {
-    return this.storage.get('order-' + walletId);
+    return this.storage.get(Keys.ORDER_WALLET(walletId));
+  };
+
+  removeWalletOrder(walletId: string): Promise<void> {
+    return this.storage.remove(Keys.ORDER_WALLET(walletId));
   };
 
   setLockStatus(isLocked: string): Promise<void> {
diff --git a/src/providers/persistence/storage/file-storage.ts b/src/providers/persistence/storage/file-storage.ts
index 3466d817ac2..d5d1f53f2f9 100644
--- a/src/providers/persistence/storage/file-storage.ts
+++ b/src/providers/persistence/storage/file-storage.ts
@@ -126,10 +126,7 @@ export class FileStorage implements IStorage {
               };
 
               if (_.isObject(v)) v = JSON.stringify(v);
-
-              if (v && !_.isString(v)) {
-                v = v.toString();
-              }
+              if (!_.isString(v)) v = v.toString();
               fileWriter.write(v);
             });
           });
diff --git a/src/providers/persistence/storage/local-storage.ts b/src/providers/persistence/storage/local-storage.ts
index c404e5aa260..016e05ae7dc 100644
--- a/src/providers/persistence/storage/local-storage.ts
+++ b/src/providers/persistence/storage/local-storage.ts
@@ -79,12 +79,8 @@ export class LocalStorage implements IStorage {
 
   set(k: string, v: any): Promise<void> {
     return new Promise<void>(resolve => {
-      if (_.isObject(v)) {
-        v = JSON.stringify(v);
-      }
-      if (v && !_.isString(v)) {
-        v = v.toString();
-      }
+      if (_.isObject(v)) v = JSON.stringify(v);
+      if (!_.isString(v)) v = v.toString();
 
       if (this.platformProvider.isNW) {
         let obj = {};
diff --git a/src/providers/profile/profile.ts b/src/providers/profile/profile.ts
index 8f71fa709b6..cd1eb47bd99 100644
--- a/src/providers/profile/profile.ts
+++ b/src/providers/profile/profile.ts
@@ -58,20 +58,16 @@ export class ProfileProvider {
     // });
   }
 
-  public setWalletOrder(walletId: string, index?: number, coin?: string): void {
-    if (index === null) {
-      let wallets = this.getWallets({ coin });
-      let maxIndex = _.maxBy(wallets, 'order') ? _.maxBy(wallets, 'order').order : null;
-      index = maxIndex != null ? maxIndex + 1 : 0;
-    }
-    this.persistenceProvider.setWalletOrder(walletId, index);
-    this.logger.debug('Wallet new order stored ' + this.wallet[walletId].name + ': ' + index);
+  public setWalletOrder(walletId: string, index: number): void {
+    this.persistenceProvider.setWalletOrder(walletId, index).then( () => {
+      this.logger.debug('Wallet new order stored ' + this.wallet[walletId].name + ': ' + index);
+    });
     this.wallet[walletId].order = index;
   }
 
   private getWalletOrder(wallet): Promise<any> {
     return new Promise((resolve, reject) => {
-      this.persistenceProvider.getWalletOrder(wallet.credentials.walletId).then((order) => {
+      this.persistenceProvider.getWalletOrder(wallet.credentials.walletId).then((order:any) => {
         return resolve(order);
       });
     })

From 502725a9fafd975c130980e1521e9927b31bc3bd Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 4 May 2018 15:43:10 -0300
Subject: [PATCH 2/5] Fix home

---
 src/pages/home/home.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/pages/home/home.ts b/src/pages/home/home.ts
index 7ce52b867a3..419cbbe2e35 100644
--- a/src/pages/home/home.ts
+++ b/src/pages/home/home.ts
@@ -547,7 +547,7 @@ export class HomePage {
     this.walletsBtc.splice(indexes.from, 1);
     this.walletsBtc.splice(indexes.to, 0, element);
     _.each(this.walletsBtc, (wallet: any, index: number) => {
-      this.profileProvider.setWalletOrder(wallet.id, index, 'btc');
+      this.profileProvider.setWalletOrder(wallet.id, index);
     });
   }
 
@@ -556,7 +556,7 @@ export class HomePage {
     this.walletsBch.splice(indexes.from, 1);
     this.walletsBch.splice(indexes.to, 0, element);
     _.each(this.walletsBch, (wallet: any, index: number) => {
-      this.profileProvider.setWalletOrder(wallet.id, index, 'bch');
+      this.profileProvider.setWalletOrder(wallet.id, index);
     });
   }
 

From 6fd556363780e0212abbbb5110da0c69ebe25cde Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 4 May 2018 15:52:50 -0300
Subject: [PATCH 3/5] revert view

---
 src/pages/onboarding/onboarding.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pages/onboarding/onboarding.html b/src/pages/onboarding/onboarding.html
index f57b8e66556..bb0122f81b6 100644
--- a/src/pages/onboarding/onboarding.html
+++ b/src/pages/onboarding/onboarding.html
@@ -1,7 +1,7 @@
 <ion-content no-bounce>
   <div class="logo-tagline">
     <img src='assets/img/app/logo-negative.svg' />
-    <p padding [ngClass]="{'primary-color': !isCopay, 'comment-color': isCopay}" translate>Take control of your money, get started with bitcoin.</p>
+    <p [ngClass]="{'primary-color': !isCopay, 'comment-color': isCopay}" translate>Take control of your money, get started with bitcoin.</p>
   </div>
   <div class="bottom-absolute">
     <button ion-button class="button-standard e2e-get-started" (click)="getStarted()">

From 01dc15a2e43fa6872f2ae507d51bbed1297b6f53 Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 4 May 2018 17:13:04 -0300
Subject: [PATCH 4/5] Testing persistence provider

---
 src/providers/persistence/persistence.spec.ts | 33 +++++++++++++++++++
 src/providers/persistence/persistence.ts      |  2 +-
 2 files changed, 34 insertions(+), 1 deletion(-)

diff --git a/src/providers/persistence/persistence.spec.ts b/src/providers/persistence/persistence.spec.ts
index 0f81b4753b1..6f5707a470e 100644
--- a/src/providers/persistence/persistence.spec.ts
+++ b/src/providers/persistence/persistence.spec.ts
@@ -145,5 +145,38 @@ describe('Persistence Provider', () => {
         })
         .then(done);
     });
+
+    it('should set and get wallet order', done => {
+      let walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
+      let order:number = 1;
+      persistenceProvider
+        .setWalletOrder(walletId, order)
+        .catch(err => expect(err).toBeNull)
+        .then(() => {
+          return persistenceProvider.getWalletOrder(walletId);
+        })
+        .then(order => {
+          expect(typeof order).toEqual('number');
+          expect(order).toEqual(1);
+        })
+        .then(done);
+    });
+
+    it('should remove wallet order', done => {
+      let walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
+      let order:number = 1;
+      persistenceProvider
+        .setWalletOrder(walletId, order)
+        .catch(err => expect(err).toBeNull)
+        .then(() => {
+          persistenceProvider.removeWalletOrder(walletId).then(() => {
+            return persistenceProvider.getWalletOrder(walletId);
+          });
+        })
+        .then(order => {
+          expect(typeof order).toEqual('undefined');
+        })
+        .then(done);
+    });
   });
 });
diff --git a/src/providers/persistence/persistence.ts b/src/providers/persistence/persistence.ts
index a83bd06e982..f75978ac4e3 100644
--- a/src/providers/persistence/persistence.ts
+++ b/src/providers/persistence/persistence.ts
@@ -505,7 +505,7 @@ export class PersistenceProvider {
     return this.storage.remove('shapeShift-' + network);
   };
 
-  setWalletOrder(walletId: string, order): Promise<void> {
+  setWalletOrder(walletId: string, order: number): Promise<void> {
     return this.storage.set(Keys.ORDER_WALLET(walletId), order);
   };
 

From 11a14b40f4326d184302b3dcffe277a8a16d7b5d Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 4 May 2018 18:49:47 -0300
Subject: [PATCH 5/5] Profile tests

---
 src/providers/persistence/persistence.spec.ts |  9 +++---
 src/providers/profile/profile.spec.ts         | 32 +++++++++++++++++--
 src/providers/profile/profile.ts              |  8 ++---
 3 files changed, 37 insertions(+), 12 deletions(-)

diff --git a/src/providers/persistence/persistence.spec.ts b/src/providers/persistence/persistence.spec.ts
index 6f5707a470e..3044131f9af 100644
--- a/src/providers/persistence/persistence.spec.ts
+++ b/src/providers/persistence/persistence.spec.ts
@@ -147,8 +147,8 @@ describe('Persistence Provider', () => {
     });
 
     it('should set and get wallet order', done => {
-      let walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
-      let order:number = 1;
+      const walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
+      const order:number = 1;
       persistenceProvider
         .setWalletOrder(walletId, order)
         .catch(err => expect(err).toBeNull)
@@ -157,14 +157,13 @@ describe('Persistence Provider', () => {
         })
         .then(order => {
           expect(typeof order).toEqual('number');
-          expect(order).toEqual(1);
         })
         .then(done);
     });
 
     it('should remove wallet order', done => {
-      let walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
-      let order:number = 1;
+      const walletId:string = '647b39d8-a88c-42d5-8728-0ba898dcdd90';
+      const order:number = 1;
       persistenceProvider
         .setWalletOrder(walletId, order)
         .catch(err => expect(err).toBeNull)
diff --git a/src/providers/profile/profile.spec.ts b/src/providers/profile/profile.spec.ts
index 3bc106574ed..e3044c438cd 100644
--- a/src/providers/profile/profile.spec.ts
+++ b/src/providers/profile/profile.spec.ts
@@ -48,7 +48,8 @@ describe('Profile Provider', () => {
       },
       isComplete: () => {
         return true;
-      }
+      },
+      order: ''
     },
     api2: {
       id: 'zxccv25b-d6ab-4b11-8b76-88570d822222',
@@ -65,7 +66,8 @@ describe('Profile Provider', () => {
       },
       isComplete: () => {
         return true;
-      }
+      }, 
+      order: 2
     },
     api3: {
       id: 'qwert25b-d6ab-4b11-8b76-88570d833333',
@@ -82,7 +84,8 @@ describe('Profile Provider', () => {
       },
       isComplete: () => {
         return true;
-      }
+      },
+      order: 3
     }
   };
 
@@ -98,6 +101,12 @@ describe('Profile Provider', () => {
     getBalanceCache(walletId: any) {
       return Promise.resolve('0.00 BTC');
     }
+    setWalletOrder(walletId: string, order: number) {
+      return Promise.resolve();
+    }
+    getWalletOrder(walletId: string) {
+      return Promise.resolve(1);
+    }
   }
 
   beforeEach(() => {
@@ -166,4 +175,21 @@ describe('Profile Provider', () => {
       expect(wallets).toEqual([]);
     });
   });
+
+  describe('wallet order', () => {
+    it('should get null order', () => {
+      const walletId:string = 'eabee25b-d6ab-4b11-8b76-88570d826914';
+      profileProvider.getWalletOrder(walletId).then((order) => {
+        expect(order).toBe('');
+      });
+    });
+    it('should set the order', () => { 
+      const walletId:string = 'eabee25b-d6ab-4b11-8b76-88570d826914';
+      const order:number = 2;
+      profileProvider.setWalletOrder(walletId, order);
+      expect(profileProvider.wallet.api1.order).toBeDefined();
+      profileProvider.wallet.api1.order = order;
+      expect(profileProvider.wallet.api1.order).toBe(2);
+    });
+  });
 });
diff --git a/src/providers/profile/profile.ts b/src/providers/profile/profile.ts
index cd1eb47bd99..744c5af2734 100644
--- a/src/providers/profile/profile.ts
+++ b/src/providers/profile/profile.ts
@@ -62,15 +62,15 @@ export class ProfileProvider {
     this.persistenceProvider.setWalletOrder(walletId, index).then( () => {
       this.logger.debug('Wallet new order stored ' + this.wallet[walletId].name + ': ' + index);
     });
-    this.wallet[walletId].order = index;
+    if (this.wallet[walletId] && this.wallet[walletId].order) this.wallet[walletId].order = index;
   }
 
-  private getWalletOrder(wallet): Promise<any> {
-    return new Promise((resolve, reject) => {
+  public getWalletOrder(wallet): Promise<any> {
+    return new Promise(resolve => {
       this.persistenceProvider.getWalletOrder(wallet.credentials.walletId).then((order:any) => {
         return resolve(order);
       });
-    })
+    });
   }
 
   public setBackupFlag(walletId: string): void {