store is optional during close and defaults to true; except during descruction

This commit is contained in:
m2049r 2017-09-23 00:52:09 +02:00
parent 1f9058187c
commit 6ee1116d75
5 changed files with 16 additions and 14 deletions

View file

@ -303,7 +303,7 @@ WalletImpl::~WalletImpl()
// Pause refresh thread - prevents refresh from starting again // Pause refresh thread - prevents refresh from starting again
pauseRefresh(); pauseRefresh();
// Close wallet - stores cache and stops ongoing refresh operation // Close wallet - stores cache and stops ongoing refresh operation
close(); close(false); // do not store wallet as part of the closing activities
// Stop refresh thread // Stop refresh thread
stopRefresh(); stopRefresh();
delete m_wallet2Callback; delete m_wallet2Callback;
@ -566,19 +566,21 @@ bool WalletImpl::recover(const std::string &path, const std::string &seed)
return m_status == Status_Ok; return m_status == Status_Ok;
} }
bool WalletImpl::close() bool WalletImpl::close(bool store)
{ {
bool result = false; bool result = false;
LOG_PRINT_L1("closing wallet..."); LOG_PRINT_L1("closing wallet...");
try { try {
// Do not store wallet with invalid status if (store) {
// Status Critical refers to errors on opening or creating wallets. // Do not store wallet with invalid status
if (status() != Status_Critical) // Status Critical refers to errors on opening or creating wallets.
m_wallet->store(); if (status() != Status_Critical)
else m_wallet->store();
LOG_ERROR("Status_Critical - not storing wallet"); else
LOG_PRINT_L1("wallet::store done"); LOG_ERROR("Status_Critical - not storing wallet");
LOG_PRINT_L1("wallet::store done");
}
LOG_PRINT_L1("Calling wallet::stop..."); LOG_PRINT_L1("Calling wallet::stop...");
m_wallet->stop(); m_wallet->stop();
LOG_PRINT_L1("wallet::stop done"); LOG_PRINT_L1("wallet::stop done");

View file

@ -63,7 +63,7 @@ public:
const std::string &address_string, const std::string &address_string,
const std::string &viewkey_string, const std::string &viewkey_string,
const std::string &spendkey_string = ""); const std::string &spendkey_string = "");
bool close(); bool close(bool store = true);
std::string seed() const; std::string seed() const;
std::string getSeedLanguage() const; std::string getSeedLanguage() const;
void setSeedLanguage(const std::string &arg); void setSeedLanguage(const std::string &arg);

View file

@ -102,10 +102,10 @@ Wallet *WalletManagerImpl::createWalletFromKeys(const std::string &path,
return wallet; return wallet;
} }
bool WalletManagerImpl::closeWallet(Wallet *wallet) bool WalletManagerImpl::closeWallet(Wallet *wallet, bool store)
{ {
WalletImpl * wallet_ = dynamic_cast<WalletImpl*>(wallet); WalletImpl * wallet_ = dynamic_cast<WalletImpl*>(wallet);
bool result = wallet_->close(); bool result = wallet_->close(store);
if (!result) { if (!result) {
m_errorString = wallet_->errorString(); m_errorString = wallet_->errorString();
} else { } else {

View file

@ -48,7 +48,7 @@ public:
const std::string &addressString, const std::string &addressString,
const std::string &viewKeyString, const std::string &viewKeyString,
const std::string &spendKeyString = ""); const std::string &spendKeyString = "");
virtual bool closeWallet(Wallet *wallet); virtual bool closeWallet(Wallet *wallet, bool store = true);
bool walletExists(const std::string &path); bool walletExists(const std::string &path);
bool verifyWalletPassword(const std::string &keys_file_name, const std::string &password, bool watch_only) const; bool verifyWalletPassword(const std::string &keys_file_name, const std::string &password, bool watch_only) const;
std::vector<std::string> findWallets(const std::string &path); std::vector<std::string> findWallets(const std::string &path);

View file

@ -663,7 +663,7 @@ struct WalletManager
* \param wallet previously opened / created wallet instance * \param wallet previously opened / created wallet instance
* \return None * \return None
*/ */
virtual bool closeWallet(Wallet *wallet) = 0; virtual bool closeWallet(Wallet *wallet, bool store = true) = 0;
/* /*
* ! checks if wallet with the given name already exists * ! checks if wallet with the given name already exists