unspent coins info handling improvements

cw_bitcoin/lib/bitcoin_address_record.dart

@@ -6,7 +6,7 @@ abstract class BaseBitcoinAddressRecord {
   BaseBitcoinAddressRecord(
     this.address, {
     required this.index,
-    this.isHidden = false,
+    required this.isHidden,
     int txCount = 0,
     int balance = 0,
     String name = '',
@@ -56,7 +56,7 @@ class BitcoinAddressRecord extends BaseBitcoinAddressRecord {
   BitcoinAddressRecord(
     super.address, {
     required super.index,
-    super.isHidden = false,
+    required super.isHidden,
     super.txCount = 0,
     super.balance = 0,
     super.name = '',

cw_bitcoin/lib/electrum_wallet.dart

@@ -924,13 +924,13 @@ abstract class ElectrumWalletBase
       );
     } else {
       // Here, lastOutput already is change, return the amount left without the fee to the user's address.
-      updatedOutputs[updatedOutputs.length - 1] = BitcoinOutput(
+      updatedOutputs.last = BitcoinOutput(
         address: lastOutput.address,
         value: BigInt.from(amountLeftForChange),
         isSilentPayment: lastOutput.isSilentPayment,
         isChange: true,
       );
-      outputs[outputs.length - 1] = BitcoinOutput(
+      outputs.last = BitcoinOutput(
         address: lastOutput.address,
         value: BigInt.from(amountLeftForChange),
         isSilentPayment: lastOutput.isSilentPayment,
@@ -1377,20 +1377,16 @@ abstract class ElectrumWalletBase
       updatedUnspentCoins.addAll(await fetchUnspent(address));
     }));
 
-    unspentCoins = updatedUnspentCoins;
+    unspentCoins = await updateCoinsWithInfoFromBox(updatedUnspentCoins);
-
-    if (unspentCoinsInfo.length != updatedUnspentCoins.length) {
-      unspentCoins.forEach((coin) => addCoinInfo(coin));
-      return;
-    }
-
-    await updateCoins(unspentCoins);
     await _refreshUnspentCoinsInfo();
   }
 
-  Future<void> updateCoins(List<BitcoinUnspent> newUnspentCoins) async {
+  Future<List<BitcoinUnspent>> updateCoinsWithInfoFromBox(List<BitcoinUnspent> newUnspentCoins) async {
+    // this function updates and returns unspent coins list (freshly fetched from the server)
+    // with info from the box, if the box doesn't have info for some of the unspents, it adds them to the box
+
     if (newUnspentCoins.isEmpty) {
-      return;
+      return newUnspentCoins;
     }
 
     newUnspentCoins.forEach((coin) {
@@ -1413,12 +1409,14 @@ abstract class ElectrumWalletBase
         addCoinInfo(coin);
       }
     });
+
+    return newUnspentCoins;
   }
 
   @action
   Future<void> updateUnspentsForAddress(BitcoinAddressRecord address) async {
     final newUnspentCoins = await fetchUnspent(address);
-    await updateCoins(newUnspentCoins);
+    unspentCoins = await updateCoinsWithInfoFromBox(newUnspentCoins);
   }
 
   @action

cw_bitcoin/lib/litecoin_wallet.dart

@@ -837,11 +837,11 @@ abstract class LitecoinWalletBase extends ElectrumWallet with Store {
       mwebUnspentCoins.add(unspent);
     });
 
-    // copy coin control attributes to mwebCoins:
+    // copy coin control attributes to coinsInfo:
-    await updateCoins(mwebUnspentCoins);
+    mwebUnspentCoins = await updateCoinsWithInfoFromBox(mwebUnspentCoins);
     // get regular ltc unspents (this resets unspentCoins):
     await super.updateAllUnspents();
-    // add the mwebCoins:
+    // add back the mwebCoins:
     unspentCoins.addAll(mwebUnspentCoins);
   }
 

cw_bitcoin/lib/litecoin_wallet_addresses.dart

@@ -106,6 +106,7 @@ abstract class LitecoinWalletAddressesBase extends ElectrumWalletAddresses with
                 index: e.key,
                 type: SegwitAddresType.mweb,
                 network: network,
+                isHidden: false,
               ))
           .toList();
       addMwebAddresses(addressRecords);
@@ -195,6 +196,7 @@ abstract class LitecoinWalletAddressesBase extends ElectrumWalletAddresses with
         index: 0,
         type: SegwitAddresType.mweb,
         network: network,
+        isHidden: true,
       );
     }