simplewallet: single out 0 amount destinations as dummy ones

Avoids surprising the user with "sending 0 to..."
This commit is contained in:
moneromooo-monero 2018-01-15 18:47:11 +00:00
parent c1d19f3c84
commit 5ae617d5ae
No known key found for this signature in database
GPG Key ID: 686F07454D6CEFC3
1 changed files with 14 additions and 2 deletions

View File

@ -4669,12 +4669,24 @@ bool simple_wallet::accept_loaded_tx(const std::function<size_t()> get_num_txes,
payment_id_string = "no payment ID"; payment_id_string = "no payment ID";
std::string dest_string; std::string dest_string;
size_t n_dummy_outputs = 0;
for (auto i = dests.begin(); i != dests.end(); ) for (auto i = dests.begin(); i != dests.end(); )
{ {
dest_string += (boost::format(tr("sending %s to %s")) % print_money(i->second.second) % i->second.first).str(); if (i->second.second > 0)
{
if (!dest_string.empty())
dest_string += ", ";
dest_string += (boost::format(tr("sending %s to %s")) % print_money(i->second.second) % i->second.first).str();
}
else
++n_dummy_outputs;
++i; ++i;
if (i != dests.end()) }
if (n_dummy_outputs > 0)
{
if (!dest_string.empty())
dest_string += ", "; dest_string += ", ";
dest_string += std::to_string(n_dummy_outputs) + tr(" dummy output(s)");
} }
if (dest_string.empty()) if (dest_string.empty())
dest_string = tr("with no destinations"); dest_string = tr("with no destinations");