bitshares / bitshares1-core

Software to run the old chain (before 2015-10-13). Code for current chain is https://github.com/bitshares/bitshares-core
https://bitshares.org/
The Unlicense
219 stars 174 forks source link

blockchain_market_list_covers command broken: default base_symbol should be BTS but not XTS #1536

Open abitmore opened 9 years ago

abitmore commented 9 years ago
>> blockchain_market_list_covers USD

10 assert_exception: Assert Exception
arec.valid(): 
    {}
    bitshares  chain_database.cpp:2093 bts::blockchain::chain_database::get_asset_id

    {"symbol":"XTS"}
    bitshares  chain_database.cpp:2095 bts::blockchain::chain_database::get_asset_id

    {"quote_symbol":"USD","limit":4294967295}
    bitshares  chain_database.cpp:2884 bts::blockchain::chain_database::get_market_covers

    {}
    bitshares  common_api_client.cpp:2208 bts::rpc_stubs::common_api_client::blockchain_market_list_covers

    {"command":"blockchain_market_list_covers"}
    bitshares  cli.cpp:626 bts::cli::detail::cli_impl::execute_command
>> help blockchain_market_list_covers

Usage:
blockchain_market_list_covers <quote_symbol> [base_symbol] [limit]                                    Returns the covers side of the order book for a given market
Returns the covers side of the order book for a given market

Parameters:
  quote_symbol (asset_symbol, required): the symbol name the market is quoted in
  base_symbol (asset_symbol, optional, defaults to "XTS"): the symbol name the market is collateralized in
  limit (uint32_t, optional, defaults to "-1"): the maximum number of items to return, -1 for all

Returns:
  market_order_array

aliases: market_covers
vikramrajkumar commented 9 years ago

base_symbol should in fact be removed since only BTS can be used as collateral.