fix monero.com block explorer link (#3174)
What the developer wrote
Authored by malik1004x
fix monero.com block explorer link (#3174)
* fix monero.com block explorer link
* Rename transaction ID variable for consistency [skip ci]
---------
Co-authored-by: Omar Hatem <omarh.ismail1@gmail.com>
What changed, and why it matters
This is a tiny UI bug fix in the Cake Wallet app. It changes which transaction identifier is used when building a link to the Monero.com block explorer. Previously the wrong field was used, so the explorer link for Monero transactions may have been broken or pointed to the wrong transaction. There is no sign this could be exploited to steal funds or compromise the app.
No security action needed. Treat as a normal bug fix and verify the Monero.com explorer link now resolves to the correct transaction.
Security signals we found
UI-only fix with no cryptographic or network changes
No input validation, parsing, or trust-boundary changes
No memory-unsafe or injection-sensitive operations
Evidence from the diff
In lib/view_model/transaction_details_view_model.dart, the code that constructs an external block-explorer URL was reading transactionInfo.id instead of transactionInfo.txHash. The patch changes the variable source to transactionInfo.txHash and renames the local variable from txId to txId for consistency. This only affects the URL shown/opened when a user taps to view a transaction on a block explorer. It does not alter transaction construction, signing, wallet logic, or network calls.
Changed components
lib/view_model/transaction_details_view_model.dartMonero.com block explorer link generationInspect captured patch +1 / −1
diff --git a/lib/view_model/transaction_details_view_model.dart b/lib/view_model/transaction_details_view_model.dart
index ed2c45e3..cc289733 100644
--- a/lib/view_model/transaction_details_view_model.dart
+++ b/lib/view_model/transaction_details_view_model.dart
@@ -397,7 +397,7 @@ abstract class TransactionDetailsViewModelBase with Store {
String get _explorerUrl {
- final txId = transactionInfo.id;
+ final txId = transactionInfo.txHash;
if (wallet.chainId != null) {
final explorerUrl = evm!.getExplorerUrlForChainId(wallet.chainId!);
if (explorerUrl != null) return '$explorerUrl/tx/${txId}';
Why this scored 19/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.