fix loop over nullish; quiet getChain error noise for old wallets

Author: kajoseph

packages/bitcore-wallet-service/src/lib/chain/index.ts

@@ -98,6 +98,9 @@ class ChainProxy {
    */
   getChain(coin: string): string {
     try {
+      if (coin === undefined) { // This happens frequently for very old btc wallets/addresses
+        return Defaults.CHAIN;
+      }
       // TODO add a warning that we are not including chain
       let normalizedChain = coin.toLowerCase();
       if (

packages/bitcore-wallet-service/src/lib/server.ts

@@ -3897,7 +3897,7 @@ export class WalletService implements IWalletService {
       tx.actions = proposal.actions.map(action => {
         return _.pick(action, ['createdOn', 'type', 'copayerId', 'copayerName', 'comment']);
       });
-      for (const output of tx.outputs) {
+      for (const output of tx.outputs || []) {
         const query = {
           toAddress: output.address,
           amount: output.amount