diff --git a/libraries/wallet/wallet.cpp b/libraries/wallet/wallet.cpp index 5810985c..71f6bc32 100644 --- a/libraries/wallet/wallet.cpp +++ b/libraries/wallet/wallet.cpp @@ -749,7 +749,7 @@ public: { std::string account_id = account_id_to_string(id); auto rec = _remote_db->get_accounts({account_id}).front(); - FC_ASSERT(rec); + FC_ASSERT(rec, "Accout id: ${account_id} doesn't exist", ("account_id", account_id)); return *rec; } account_object get_account(string account_name_or_id) const @@ -762,7 +762,7 @@ public: return get_account(*id); } else { auto rec = _remote_db->lookup_account_names({account_name_or_id}).front(); - FC_ASSERT( rec && rec->name == account_name_or_id ); + FC_ASSERT( rec && rec->name == account_name_or_id, "Account name or id: ${account_name_or_id} doesn't exist", ("account_name_or_id",account_name_or_id ) ); return *rec; } } @@ -4514,6 +4514,7 @@ asset wallet_api::get_lottery_balance( asset_id_type lottery_id )const vector wallet_api::get_account_history(string name, int limit) const { + FC_ASSERT(my->get_account(name).name == name, "Account name: ${account_name} doesn't exist", ("account_name", name)); vector result; while (limit > 0) @@ -4568,6 +4569,8 @@ vector wallet_api::get_relative_account_history(string name, u { FC_ASSERT( start > 0 || limit <= 100 ); + FC_ASSERT(my->get_account(name).name == name, "Account name: ${account_name} doesn't exist", ("account_name", name)); + vector result; @@ -4743,7 +4746,7 @@ account_object wallet_api::get_account(string account_name_or_id) const asset_object wallet_api::get_asset(string asset_name_or_id) const { auto a = my->find_asset(asset_name_or_id); - FC_ASSERT(a); + FC_ASSERT(a, "Asset name or id: ${asset_name_or_id} doesn't exist", ("asset_name_or_id", asset_name_or_id) ); return *a; } @@ -4766,7 +4769,7 @@ asset_id_type wallet_api::get_asset_id(string asset_symbol_or_id) const bool wallet_api::import_key(string account_name_or_id, string wif_key) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); // backup wallet fc::optional optional_private_key = wif_to_key(wif_key); if (!optional_private_key) @@ -4785,7 +4788,7 @@ bool wallet_api::import_key(string account_name_or_id, string wif_key) map wallet_api::import_accounts( string filename, string password ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); FC_ASSERT( fc::exists( filename ) ); const auto imported_keys = fc::json::from_file( filename ); @@ -4855,7 +4858,7 @@ map wallet_api::import_accounts( string filename, string password bool wallet_api::import_account_keys( string filename, string password, string src_account_name, string dest_account_name ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); FC_ASSERT( fc::exists( filename ) ); bool is_my_account = false; @@ -5404,13 +5407,13 @@ operation wallet_api::get_prototype_operation(string operation_name) void wallet_api::dbg_make_uia(string creator, string symbol) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); my->dbg_make_uia(creator, symbol); } void wallet_api::dbg_make_mia(string creator, string symbol) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); my->dbg_make_mia(creator, symbol); } @@ -5446,7 +5449,7 @@ vector< variant > wallet_api::network_get_connected_peers() void wallet_api::flood_network(string prefix, uint32_t number_of_transactions) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); my->flood_network(prefix, number_of_transactions); } @@ -5925,13 +5928,13 @@ signed_transaction wallet_api::buy( string buyer_account, signed_transaction wallet_api::borrow_asset(string seller_name, string amount_to_sell, string asset_symbol, string amount_of_collateral, bool broadcast) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); return my->borrow_asset(seller_name, amount_to_sell, asset_symbol, amount_of_collateral, broadcast); } signed_transaction wallet_api::cancel_order(object_id_type order_id, bool broadcast) { - FC_ASSERT(!is_locked()); + FC_ASSERT(!is_locked(), "Wallet is locked, please unlock to get all operations available"); return my->cancel_order(order_id, broadcast); } @@ -5981,7 +5984,7 @@ map wallet_api::get_blind_accounts()const } map wallet_api::get_my_blind_accounts()const { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); map result; for( const auto& item : my->_wallet.labeled_keys ) { @@ -5993,7 +5996,7 @@ map wallet_api::get_my_blind_accounts()const public_key_type wallet_api::create_blind_account( string label, string brain_key ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); auto label_itr = my->_wallet.labeled_keys.get().find(label); if( label_itr != my->_wallet.labeled_keys.get().end() ) @@ -6121,7 +6124,7 @@ blind_confirmation wallet_api::blind_transfer_help( string from_key_or_label, blind_confirmation confirm; try { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); public_key_type from_key = get_public_key(from_key_or_label); public_key_type to_key = get_public_key(to_key_or_label); @@ -6290,7 +6293,7 @@ blind_confirmation wallet_api::transfer_to_blind( string from_account_id_or_name vector> to_amounts, bool broadcast ) { try { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); idump((to_amounts)); blind_confirmation confirm; @@ -6370,7 +6373,7 @@ blind_confirmation wallet_api::transfer_to_blind( string from_account_id_or_name blind_receipt wallet_api::receive_blind_transfer( string confirmation_receipt, string opt_from, string opt_memo ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); stealth_confirmation conf(confirmation_receipt); FC_ASSERT( conf.to ); @@ -6512,7 +6515,7 @@ signed_transaction wallet_api::propose_create_sport( internationalized_string_type name, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; sport_create_operation sport_create_op; @@ -6540,7 +6543,7 @@ signed_transaction wallet_api::propose_update_sport( fc::optional name, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; sport_update_operation sport_update_op; @@ -6568,7 +6571,7 @@ signed_transaction wallet_api::propose_delete_sport( sport_id_type sport_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; sport_delete_operation sport_delete_op; @@ -6596,7 +6599,7 @@ signed_transaction wallet_api::propose_create_event_group( sport_id_type sport_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; event_group_create_operation event_group_create_op; @@ -6626,7 +6629,7 @@ signed_transaction wallet_api::propose_update_event_group( fc::optional name, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; event_group_update_operation event_group_update_op; @@ -6655,7 +6658,7 @@ signed_transaction wallet_api::propose_delete_event_group( event_group_id_type event_group, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; event_group_delete_operation event_group_delete_op; @@ -6685,7 +6688,7 @@ signed_transaction wallet_api::propose_create_event( event_group_id_type event_group_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; event_create_operation event_create_op; @@ -6720,7 +6723,7 @@ signed_transaction wallet_api::propose_update_event( fc::optional start_time, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; event_update_operation event_update_op; @@ -6753,7 +6756,7 @@ signed_transaction wallet_api::propose_create_betting_market_rules( internationalized_string_type description, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_rules_create_operation betting_market_rules_create_op; @@ -6783,7 +6786,7 @@ signed_transaction wallet_api::propose_update_betting_market_rules( fc::optional description, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_rules_update_operation betting_market_rules_update_op; @@ -6815,7 +6818,7 @@ signed_transaction wallet_api::propose_create_betting_market_group( asset_id_type asset_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_group_create_operation betting_market_group_create_op; @@ -6848,7 +6851,7 @@ signed_transaction wallet_api::propose_update_betting_market_group( fc::optional status, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_group_update_operation betting_market_group_update_op; @@ -6880,7 +6883,7 @@ signed_transaction wallet_api::propose_create_betting_market( internationalized_string_type payout_condition, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_create_operation betting_market_create_op; @@ -6912,7 +6915,7 @@ signed_transaction wallet_api::propose_update_betting_market( fc::optional payout_condition, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked() , "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_update_operation betting_market_update_op; @@ -6944,7 +6947,7 @@ signed_transaction wallet_api::place_bet(string betting_account, double backer_multiplier, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); fc::optional asset_obj = get_asset(asset_symbol); FC_ASSERT(asset_obj, "Could not find asset matching ${asset}", ("asset", asset_symbol)); @@ -6969,7 +6972,7 @@ signed_transaction wallet_api::cancel_bet(string betting_account, bet_id_type bet_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; @@ -6992,7 +6995,7 @@ signed_transaction wallet_api::propose_resolve_betting_market_group( const std::map& resolutions, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_group_resolve_operation betting_market_group_resolve_op; @@ -7020,7 +7023,7 @@ signed_transaction wallet_api::propose_cancel_betting_market_group( betting_market_group_id_type betting_market_group_id, bool broadcast /*= false*/) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); const chain_parameters& current_params = get_global_properties().parameters; betting_market_group_cancel_unmatched_bets_operation betting_market_group_cancel_unmatched_bets_op; @@ -7043,7 +7046,7 @@ signed_transaction wallet_api::propose_cancel_betting_market_group( signed_transaction wallet_api::tournament_create( string creator, tournament_options options, bool broadcast ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); account_object creator_account_obj = get_account(creator); signed_transaction tx; @@ -7064,7 +7067,7 @@ signed_transaction wallet_api::tournament_join( string payer_account, string buy_in_asset_symbol, bool broadcast ) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); account_object payer_account_obj = get_account(payer_account); account_object player_account_obj = get_account(player_account); //graphene::chain::tournament_object tournament_obj = my->get_object(tournament_id); @@ -7091,7 +7094,7 @@ signed_transaction wallet_api::tournament_leave( string canceling_account, tournament_id_type tournament_id, bool broadcast) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); account_object player_account_obj = get_account(player_account); account_object canceling_account_obj = get_account(canceling_account); //graphene::chain::tournament_object tournament_obj = my->get_object(tournament_id); @@ -7136,7 +7139,7 @@ signed_transaction wallet_api::rps_throw(game_id_type game_id, rock_paper_scissors_gesture gesture, bool broadcast) { - FC_ASSERT( !is_locked() ); + FC_ASSERT( !is_locked(), "Wallet is locked, please unlock to get all operations available" ); // check whether the gesture is appropriate for the game we're playing graphene::chain::game_object game_obj = my->get_object(game_id);