Fix label show in history for >1 & <100 confirmations
MODIFY
WalletWasabi.Tests/UnitTests/Transactions/TransactionProcessorTests.cs
WalletWasabi.Tests/UnitTests/Transactions/TransactionProcessorTests.cs
@@ -792,7 +792,6 @@ namespace WalletWasabi.Tests.UnitTests.Transactions
Assert.Empty(matureTxs);
Assert.Contains(spentCoin, tx1.WalletInputs);
- Assert.Contains(spentCoin, tx2.WalletInputs);
}
[Fact]

MODIFY
WalletWasabi/Blockchain/TransactionProcessing/TransactionProcessor.cs
WalletWasabi/Blockchain/TransactionProcessing/TransactionProcessor.cs
@@ -126,6 +126,14 @@ namespace WalletWasabi.Blockchain.TransactionProcessing
uint256 txId = tx.GetHash();
+ // If we already have the transaction, then let's work on that.
+ if (TransactionStore.TryGetTransaction(txId, out var foundTx))
+ {
+ foundTx.TryUpdate(tx);
+ tx = foundTx;
+ result = new ProcessedResult(tx);
+ }
+
// Performance ToDo: txids could be cached in a hashset here by the AllCoinsView and then the contains would be fast.
if (!tx.Transaction.IsCoinBase && !Coins.AsAllCoinsView().CreatedBy(txId).Any()) // Transactions we already have and processed would be "double spends" but they shouldn't.
{

Fix label show in history for >1 & <100 confirmations