Change some collection types in the wallet object

Breeze/src/Breeze.Wallet/Wallet.cs

@@ -121,7 +121,7 @@ namespace Breeze.Wallet
         /// The accounts used in the wallet.
         /// </summary>
         [JsonProperty(PropertyName = "accounts")]
-        public IEnumerable<HdAccount> Accounts { get; set; }
+        public ICollection<HdAccount> Accounts { get; set; }
 
         /// <summary>
         /// Gets the first account that contains no transaction.
@@ -227,13 +227,13 @@ namespace Breeze.Wallet
         /// The list of external addresses, typically used for receiving money.
         /// </summary>
         [JsonProperty(PropertyName = "externalAddresses")]
-        public IEnumerable<HdAddress> ExternalAddresses { get; set; }
+        public ICollection<HdAddress> ExternalAddresses { get; set; }
 
         /// <summary>
         /// The list of internal addresses, typically used to receive change.
         /// </summary>
         [JsonProperty(PropertyName = "internalAddresses")]
-        public IEnumerable<HdAddress> InternalAddresses { get; set; }
+        public ICollection<HdAddress> InternalAddresses { get; set; }
 
         /// <summary>
         /// Gets the type of coin this account is for.
@@ -374,7 +374,7 @@ namespace Breeze.Wallet
         /// A list of transactions involving this address.
         /// </summary>
         [JsonProperty(PropertyName = "transactions")]
-        public IEnumerable<TransactionData> Transactions { get; set; }
+        public ICollection<TransactionData> Transactions { get; set; }
     }
 
     /// <summary>

Breeze/src/Breeze.Wallet/WalletManager.cs

@@ -289,14 +289,14 @@ namespace Breeze.Wallet
                 BitcoinPubKeyAddress address = this.GenerateAddress(account.ExtendedPubKey, i, isChange, network);
 
                 // add address details
-                addresses = addresses.Concat(new[] {new HdAddress
+                addresses.Add(new HdAddress
                 {
                     Index = i,
                     HdPath = CreateBip44Path(account.GetCoinType(), account.Index, i, isChange),
                     ScriptPubKey = address.ScriptPubKey,
                     Address = address.ToString(),
                     Transactions = new List<TransactionData>()
-                }});
+                });
 
                 addressesCreated.Add(address.ToString());
             }
@@ -497,7 +497,7 @@ namespace Breeze.Wallet
         {
             // selects all the transactions we already have in the wallet
             var txs = this.Wallets.SelectMany(w => w.GetAllTransactionsByCoinType(coinType));
-            
+
             // add this transaction if it is not in the list
             if (txs.All(t => t.Id != transactionHash || t.Index != index))
             {
@@ -514,17 +514,14 @@ namespace Breeze.Wallet
                             var address = receivingAddress ?? changeAddress;
                             if (address != null)
                             {
-                                address.Transactions = address.Transactions.Concat(new[]
+                                address.Transactions.Add(new TransactionData
                                 {
-                                    new TransactionData
-                                    {
-                                        Amount = amount,
-                                        BlockHeight = blockHeight,
-                                        Confirmed = blockHeight.HasValue,
-                                        Id = transactionHash,
-                                        CreationTime = DateTimeOffset.FromUnixTimeMilliseconds(blockTime ?? time),
-                                        Index = index
-                                    }
+                                    Amount = amount,
+                                    BlockHeight = blockHeight,
+                                    Confirmed = blockHeight.HasValue,
+                                    Id = transactionHash,
+                                    CreationTime = DateTimeOffset.FromUnixTimeMilliseconds(blockTime ?? time),
+                                    Index = index
                                 });
 
                                 // notify a transaction has been found