simplewallet: rename duplicate amount headers for clarity
What the developer wrote
Authored by reemuru
simplewallet: rename duplicate amount headers for clarity
There are duplicate 'amount' headers when using export_transfers all. Column five represents the transaction total while column eleven represents the amount transferred per destination.
Rename the columns without changing their order or values.
(cherry picked from commit 30ab49d5cb5f7d5b089d20919cf4f02b063f2e6b)
What changed, and why it matters
This commit only renames two CSV column headers in the Monero wallet's export_transfers output from 'amount' to 'transaction amount' and 'destination amount' to make them less confusing. No numbers, logic, or behavior changed.
No security action needed. Treat as a normal non-security UI/labeling improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In src/simplewallet/simplewallet.cpp, the export_transfers CSV header string is updated: the fifth column header changes from tr(“amount”) to tr(“transaction amount”) and the eleventh column header changes from tr(“amount”) to tr(“destination amount”). The column order, formatting widths, and all data values remain identical. This is a cosmetic clarity fix.
Changed components
src/simplewallet/simplewallet.cpp export_transfers CSV headerInspect captured patch +1 / −1
diff --git a/src/simplewallet/simplewallet.cpp b/src/simplewallet/simplewallet.cpp
index 294b358..1806128 100644
--- a/src/simplewallet/simplewallet.cpp
+++ b/src/simplewallet/simplewallet.cpp
@@ -8681,7 +8681,7 @@ bool simple_wallet::export_transfers(const std::vector<std::string>& args_)
// header
file <<
boost::format("%8.8s,%9.9s,%8.8s,%25.25s,%20.20s,%20.20s,%64.64s,%16.16s,%14.14s,%106.106s,%20.20s,%s,%s,%s") %
- tr("block") % tr("direction") % tr("unlocked") % tr("timestamp") % tr("amount") % tr("running balance") % tr("hash") % tr("payment ID") % tr("fee") % tr("destination") % tr("amount") % tr("index") % tr("note") % tr("tx key")
+ tr("block") % tr("direction") % tr("unlocked") % tr("timestamp") % tr("transaction amount") % tr("running balance") % tr("hash") % tr("payment ID") % tr("fee") % tr("destination") % tr("destination amount") % tr("index") % tr("note") % tr("tx key")
<< std::endl;
uint64_t running_balance = 0;
Why this scored 15/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.