Refactor RPC wallet_accounts.

This commit is contained in:
Arthur Britto
2012-12-03 16:12:32 -08:00
parent dc977d1bfd
commit f8a906ce9a
4 changed files with 22 additions and 11 deletions

View File

@@ -228,6 +228,17 @@ Json::Value RPCParser::parseUnlDelete(const Json::Value& jvParams)
return jvRequest; return jvRequest;
} }
// wallet_accounts <seed>
Json::Value RPCParser::parseWalletAccounts(const Json::Value& jvParams)
{
Json::Value jvRequest;
jvRequest["seed"] = jvParams[0u].asString();
return jvRequest;
}
// //
// parseCommand // parseCommand
// //
@@ -288,7 +299,7 @@ Json::Value RPCParser::parseCommand(std::string strMethod, Json::Value jvParams)
// { "validation_create", &RPCParser::doValidationCreate, 0, 1, false, false, optNone }, // { "validation_create", &RPCParser::doValidationCreate, 0, 1, false, false, optNone },
// { "validation_seed", &RPCParser::doValidationSeed, 0, 1, false, false, optNone }, // { "validation_seed", &RPCParser::doValidationSeed, 0, 1, false, false, optNone },
// { "wallet_accounts", &RPCParser::doWalletAccounts, 1, 1, false, false, optCurrent }, { "wallet_accounts", &RPCParser::parseWalletAccounts, 1, 1 },
// { "wallet_propose", &RPCParser::doWalletPropose, 0, 1, false, false, optNone }, // { "wallet_propose", &RPCParser::doWalletPropose, 0, 1, false, false, optNone },
// { "wallet_seed", &RPCParser::doWalletSeed, 0, 1, false, false, optNone }, // { "wallet_seed", &RPCParser::doWalletSeed, 0, 1, false, false, optNone },
// //

View File

@@ -17,6 +17,7 @@ protected:
Json::Value parseConnect(const Json::Value& jvParams); Json::Value parseConnect(const Json::Value& jvParams);
Json::Value parseEvented(const Json::Value& jvParams); Json::Value parseEvented(const Json::Value& jvParams);
Json::Value parseLedger(const Json::Value& jvParams); Json::Value parseLedger(const Json::Value& jvParams);
Json::Value parseWalletAccounts(const Json::Value& jvParams);
Json::Value parseRippleLinesGet(const Json::Value& jvParams); Json::Value parseRippleLinesGet(const Json::Value& jvParams);
Json::Value parseSubmit(const Json::Value& jvParams); Json::Value parseSubmit(const Json::Value& jvParams);
Json::Value parseUnlAdd(const Json::Value& jvParams); Json::Value parseUnlAdd(const Json::Value& jvParams);

View File

@@ -1325,12 +1325,14 @@ Json::Value RPCHandler::accounts(const uint256& uLedger, const RippleAddress& na
return jsonAccounts; return jsonAccounts;
} }
// wallet_accounts <seed> // {
Json::Value RPCHandler::doWalletAccounts(Json::Value params) // seed: <string>
// }
Json::Value RPCHandler::doWalletAccounts(Json::Value jvParams)
{ {
RippleAddress naSeed; RippleAddress naSeed;
if (!naSeed.setSeedGeneric(params[0u].asString())) if (!jvParams.isMember("seed") || !naSeed.setSeedGeneric(jvParams["seed"].asString()))
{ {
return rpcError(rpcBAD_SEED); return rpcError(rpcBAD_SEED);
} }
@@ -2214,7 +2216,7 @@ Json::Value RPCHandler::doCommand(Json::Value& jvParams, int iRole)
{ "validation_create", &RPCHandler::doValidationCreate, 0, 1, false, false, optNone }, { "validation_create", &RPCHandler::doValidationCreate, 0, 1, false, false, optNone },
{ "validation_seed", &RPCHandler::doValidationSeed, 0, 1, false, false, optNone }, { "validation_seed", &RPCHandler::doValidationSeed, 0, 1, false, false, optNone },
{ "wallet_accounts", &RPCHandler::doWalletAccounts, 1, 1, false, false, optCurrent }, { "wallet_accounts", &RPCHandler::doWalletAccounts, -1, -1, false, false, optCurrent },
{ "wallet_propose", &RPCHandler::doWalletPropose, 0, 1, false, false, optNone }, { "wallet_propose", &RPCHandler::doWalletPropose, 0, 1, false, false, optNone },
{ "wallet_seed", &RPCHandler::doWalletSeed, 0, 1, false, false, optNone }, { "wallet_seed", &RPCHandler::doWalletSeed, 0, 1, false, false, optNone },

View File

@@ -712,14 +712,12 @@ Remote.prototype.request_ripple_lines_get = function (accountID, index) {
return request; return request;
}; };
Remote.prototype.request_wallet_accounts = function (key) { Remote.prototype.request_wallet_accounts = function (seed) {
// XXX Does this require the server to be trusted? assert(this.trusted); // Don't send secrets.
//assert(this.trusted);
var request = new Request(this, 'wallet_accounts'); var request = new Request(this, 'wallet_accounts');
// XXX Convert API call to JSON request.message.seed = seed;
request.message.params = [key];
return request; return request;
}; };
@@ -730,7 +728,6 @@ Remote.prototype.request_account_tx = function (accountID, ledger_min, ledger_ma
var request = new Request(this, 'account_tx'); var request = new Request(this, 'account_tx');
// XXX Convert API call to JSON
request.message.account = accountID; request.message.account = accountID;
if (ledger_min === ledger_max) { if (ledger_min === ledger_max) {