diff options
author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2017-03-18 11:43:13 +0000 |
---|---|---|
committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2017-03-18 11:44:31 +0000 |
commit | 1a666c301634ebf543fe23aa1dd51388e3f05edc (patch) | |
tree | fa8cff2feff8e1aa1c2e8bbe65e0d82b65432aa8 | |
parent | Merge pull request #1869 (diff) | |
download | monero-1a666c301634ebf543fe23aa1dd51388e3f05edc.tar.xz |
simplewallet: catch case where no transactions are made
This would otherwise be a silent noop, which is confusing.
This can happen if the daemon is started, but not yet ready
to service all requests, and this is a safe catch all.
-rw-r--r-- | src/simplewallet/simplewallet.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/simplewallet/simplewallet.cpp b/src/simplewallet/simplewallet.cpp index f5763648f..387c44545 100644 --- a/src/simplewallet/simplewallet.cpp +++ b/src/simplewallet/simplewallet.cpp @@ -2298,6 +2298,12 @@ bool simple_wallet::transfer_main(int transfer_type, const std::vector<std::stri break; } + if (ptx_vector.empty()) + { + fail_msg_writer() << tr("No outputs found, or daemon is not ready"); + return true; + } + // if more than one tx necessary, prompt user to confirm if (m_wallet->always_confirm_transfers() || ptx_vector.size() > 1) { @@ -2764,7 +2770,7 @@ bool simple_wallet::sweep_all(const std::vector<std::string> &args_) if (ptx_vector.empty()) { - fail_msg_writer() << tr("No outputs found"); + fail_msg_writer() << tr("No outputs found, or daemon is not ready"); return true; } |