Merge branch 'develop' of github.com:jedmccaleb/NewCoin into develop

This commit is contained in:
JoelKatz
2013-04-15 05:45:33 -07:00
6 changed files with 109 additions and 60 deletions

View File

@@ -421,12 +421,24 @@ Json::Value RPCParser::parseLogLevel(const Json::Value& jvParams)
// owner_info <seed>|<pass_phrase>|<key> [<ledfer>]
// account_info <account>|<nickname>|<account_public_key>
// account_info <seed>|<pass_phrase>|<key> [<ledger>]
// account_lines <account>|<nickname>|<account_public_key> [<ledger>]
// account_offers <account>|<nickname>|<account_public_key> [<ledger>]
Json::Value RPCParser::parseAccountItems(const Json::Value& jvParams)
{
return parseAccountRaw(jvParams, false);
}
// account_lines <account> <account>|"" [<ledger>]
Json::Value RPCParser::parseAccountLines(const Json::Value& jvParams)
{
return parseAccountRaw(jvParams, true);
}
// TODO: Get index from an alternate syntax: rXYZ:<index>
Json::Value RPCParser::parseAccountRaw(const Json::Value& jvParams, bool bPeer)
{
std::string strIdent = jvParams[0u].asString();
// TODO: Get index from an alternate syntax: rXYZ:<index>
std::string strPeer = bPeer && jvParams.size() >= 2 ? jvParams[1u].asString() : "";
int iIndex = 0;
// int iIndex = jvParams.size() >= 2 ? lexical_cast_s<int>(jvParams[1u].asString()) : 0;
@@ -443,7 +455,17 @@ Json::Value RPCParser::parseAccountItems(const Json::Value& jvParams)
if (iIndex)
jvRequest["account_index"] = iIndex;
if (jvParams.size() == 2 && !jvParseLedger(jvRequest, jvParams[1u].asString()))
if (!strPeer.empty())
{
RippleAddress raPeer;
if (!raPeer.setAccountPublic(strPeer) && !raPeer.setAccountID(strPeer) && !raPeer.setSeedGeneric(strPeer))
return rpcError(rpcACT_MALFORMED);
jvRequest["peer"] = strPeer;
}
if (jvParams.size() == (2+bPeer) && !jvParseLedger(jvRequest, jvParams[1u+bPeer].asString()))
return rpcError(rpcLGR_IDX_MALFORMED);
return jvRequest;
@@ -655,7 +677,7 @@ Json::Value RPCParser::parseCommand(std::string strMethod, Json::Value jvParams)
// - Returns an error, or the request.
// - To modify the method, provide a new method in the request.
{ "account_info", &RPCParser::parseAccountItems, 1, 2 },
{ "account_lines", &RPCParser::parseAccountItems, 1, 2 },
{ "account_lines", &RPCParser::parseAccountLines, 1, 3 },
{ "account_offers", &RPCParser::parseAccountItems, 1, 2 },
{ "account_tx", &RPCParser::parseAccountTransactions, 1, 8 },
{ "book_offers", &RPCParser::parseBookOffers, 2, 7 },

View File

@@ -10,7 +10,10 @@ class RPCParser
protected:
typedef Json::Value (RPCParser::*parseFuncPtr)(const Json::Value &jvParams);
Json::Value parseAccountRaw(const Json::Value& jvParams, bool bPeer);
Json::Value parseAccountItems(const Json::Value& jvParams);
Json::Value parseAccountLines(const Json::Value& jvParams);
Json::Value parseAccountTransactions(const Json::Value& jvParams);
Json::Value parseAsIs(const Json::Value& jvParams);
Json::Value parseBookOffers(const Json::Value& jvParams);

View File

@@ -899,11 +899,23 @@ Json::Value RPCHandler::doAccountLines(Json::Value jvRequest, int& cost)
if (!jvResult.empty())
return jvResult;
// Get info on account.
std::string strPeer = jvRequest.isMember("peer") ? jvRequest["peer"].asString() : "";
bool bPeerIndex = jvRequest.isMember("peer_index");
int iPeerIndex = bIndex ? jvRequest["peer_index"].asUInt() : 0;
jvResult["account"] = raAccount.humanAccountID();
if (bIndex)
jvResult["account_index"] = iIndex;
RippleAddress raPeer;
if (!strPeer.empty())
{
jvResult["peer"] = raAccount.humanAccountID();
if (bPeerIndex)
jvResult["peer_index"] = iPeerIndex;
jvResult = accountFromString(lpLedger, raPeer, bPeerIndex, strPeer, iPeerIndex, false);
if (!jvResult.empty())
return jvResult;
}
AccountState::pointer as = mNetOps->getAccountState(lpLedger, raAccount);
if (as)
@@ -912,7 +924,6 @@ Json::Value RPCHandler::doAccountLines(Json::Value jvRequest, int& cost)
jvResult["account"] = raAccount.humanAccountID();
// XXX This is wrong, we do access the current ledger and do need to worry about changes.
// We access a committed ledger and need not worry about changes.
@@ -922,23 +933,26 @@ Json::Value RPCHandler::doAccountLines(Json::Value jvRequest, int& cost)
{
RippleState* line=(RippleState*)item.get();
STAmount saBalance = line->getBalance();
STAmount saLimit = line->getLimit();
STAmount saLimitPeer = line->getLimitPeer();
if (!raPeer.isValid() || raPeer.getAccountID() == line->getAccountIDPeer())
{
STAmount saBalance = line->getBalance();
STAmount saLimit = line->getLimit();
STAmount saLimitPeer = line->getLimitPeer();
Json::Value jPeer = Json::Value(Json::objectValue);
Json::Value jPeer = Json::Value(Json::objectValue);
jPeer["account"] = RippleAddress::createHumanAccountID(line->getAccountIDPeer());
// Amount reported is positive if current account holds other account's IOUs.
// Amount reported is negative if other account holds current account's IOUs.
jPeer["balance"] = saBalance.getText();
jPeer["currency"] = saBalance.getHumanCurrency();
jPeer["limit"] = saLimit.getText();
jPeer["limit_peer"] = saLimitPeer.getText();
jPeer["quality_in"] = static_cast<Json::UInt>(line->getQualityIn());
jPeer["quality_out"] = static_cast<Json::UInt>(line->getQualityOut());
jPeer["account"] = RippleAddress::createHumanAccountID(line->getAccountIDPeer());
// Amount reported is positive if current account holds other account's IOUs.
// Amount reported is negative if other account holds current account's IOUs.
jPeer["balance"] = saBalance.getText();
jPeer["currency"] = saBalance.getHumanCurrency();
jPeer["limit"] = saLimit.getText();
jPeer["limit_peer"] = saLimitPeer.getText();
jPeer["quality_in"] = static_cast<Json::UInt>(line->getQualityIn());
jPeer["quality_out"] = static_cast<Json::UInt>(line->getQualityOut());
jsonLines.append(jPeer);
jsonLines.append(jPeer);
}
}
jvResult["lines"] = jsonLines;
}

View File

@@ -30,39 +30,14 @@ std::size_t hash_value(const CBase58Data& b58)
return seed;
}
RippleAddress::RippleAddress()
RippleAddress::RippleAddress() : mIsValid(false)
{
nVersion = VER_NONE;
}
bool RippleAddress::isValid() const
{
bool bValid = false;
if (!vchData.empty())
{
CKey key;
switch (nVersion) {
case VER_NODE_PUBLIC:
bValid = key.SetPubKey(getNodePublic());
break;
case VER_ACCOUNT_PUBLIC:
bValid = key.SetPubKey(getAccountPublic());
break;
case VER_ACCOUNT_PRIVATE:
bValid = key.SetPrivateKeyU(getAccountPrivate());
break;
default:
bValid = true;
break;
}
}
return bValid;
return mIsValid;
}
void RippleAddress::clear()
@@ -170,11 +145,15 @@ std::string RippleAddress::humanNodePublic() const
bool RippleAddress::setNodePublic(const std::string& strPublic)
{
return SetString(strPublic.c_str(), VER_NODE_PUBLIC);
mIsValid = SetString(strPublic.c_str(), VER_NODE_PUBLIC);
return mIsValid;
}
void RippleAddress::setNodePublic(const std::vector<unsigned char>& vPublic)
{
mIsValid = true;
SetData(VER_NODE_PUBLIC, vPublic);
}
@@ -264,15 +243,22 @@ std::string RippleAddress::humanNodePrivate() const
bool RippleAddress::setNodePrivate(const std::string& strPrivate)
{
return SetString(strPrivate.c_str(), VER_NODE_PRIVATE);
mIsValid = SetString(strPrivate.c_str(), VER_NODE_PRIVATE);
return mIsValid;
}
void RippleAddress::setNodePrivate(const std::vector<unsigned char>& vPrivate) {
void RippleAddress::setNodePrivate(const std::vector<unsigned char>& vPrivate)
{
mIsValid = true;
SetData(VER_NODE_PRIVATE, vPrivate);
}
void RippleAddress::setNodePrivate(uint256 hash256)
{
mIsValid = true;
SetData(VER_NODE_PRIVATE, hash256.begin(), 32);
}
@@ -348,16 +334,20 @@ bool RippleAddress::setAccountID(const std::string& strAccountID)
{
setAccountID(uint160());
return true;
mIsValid = true;
}
else
{
return SetString(strAccountID.c_str(), VER_ACCOUNT_ID);
mIsValid = SetString(strAccountID.c_str(), VER_ACCOUNT_ID);
}
return mIsValid;
}
void RippleAddress::setAccountID(const uint160& hash160)
{
mIsValid = true;
SetData(VER_ACCOUNT_ID, hash160.begin(), 20);
}
@@ -412,11 +402,15 @@ std::string RippleAddress::humanAccountPublic() const
bool RippleAddress::setAccountPublic(const std::string& strPublic)
{
return SetString(strPublic.c_str(), VER_ACCOUNT_PUBLIC);
mIsValid = SetString(strPublic.c_str(), VER_ACCOUNT_PUBLIC);
return mIsValid;
}
void RippleAddress::setAccountPublic(const std::vector<unsigned char>& vPublic)
{
mIsValid = true;
SetData(VER_ACCOUNT_PUBLIC, vPublic);
}
@@ -498,16 +492,22 @@ std::string RippleAddress::humanAccountPrivate() const
bool RippleAddress::setAccountPrivate(const std::string& strPrivate)
{
return SetString(strPrivate.c_str(), VER_ACCOUNT_PRIVATE);
mIsValid = SetString(strPrivate.c_str(), VER_ACCOUNT_PRIVATE);
return mIsValid;
}
void RippleAddress::setAccountPrivate(const std::vector<unsigned char>& vPrivate)
{
mIsValid = true;
SetData(VER_ACCOUNT_PRIVATE, vPrivate);
}
void RippleAddress::setAccountPrivate(uint256 hash256)
{
mIsValid = true;
SetData(VER_ACCOUNT_PRIVATE, hash256.begin(), 32);
}
@@ -678,11 +678,15 @@ std::string RippleAddress::humanGenerator() const
bool RippleAddress::setGenerator(const std::string& strGenerator)
{
return SetString(strGenerator.c_str(), VER_FAMILY_GENERATOR);
mIsValid = SetString(strGenerator.c_str(), VER_FAMILY_GENERATOR);
return mIsValid;
}
void RippleAddress::setGenerator(const std::vector<unsigned char>& vPublic)
{
mIsValid = true;
SetData(VER_FAMILY_GENERATOR, vPublic);
}
@@ -773,7 +777,9 @@ int RippleAddress::setSeed1751(const std::string& strHuman1751)
bool RippleAddress::setSeed(const std::string& strSeed)
{
return SetString(strSeed.c_str(), VER_FAMILY_SEED);
mIsValid = SetString(strSeed.c_str(), VER_FAMILY_SEED);
return mIsValid;
}
extern const char *ALPHABET;
@@ -817,6 +823,8 @@ bool RippleAddress::setSeedGeneric(const std::string& strText)
}
void RippleAddress::setSeed(uint128 hash128) {
mIsValid = true;
SetData(VER_FAMILY_SEED, hash128.begin(), 16);
}

View File

@@ -22,6 +22,8 @@ private:
VER_FAMILY_SEED = 33,
} VersionEncoding;
bool mIsValid;
public:
RippleAddress();

View File

@@ -71,7 +71,7 @@ void printHelp(const po::options_description& desc)
cerr << "Commands: " << endl;
cerr << " account_info <account>|<nickname>|<seed>|<pass_phrase>|<key> [<ledger>]" << endl;
cerr << " account_lines <account>|<nickname>|<account_public_key> [<ledger>]" << endl;
cerr << " account_lines <account> <account>|\"\" [<ledger>]" << endl;
cerr << " account_offers <account>|<nickname>|<account_public_key> [<ledger>]" << endl;
cerr << " account_tx accountID [ledger_min [ledger_max [limit [offset]]]] [binary] [count] [descending]" << endl;
cerr << " book_offers <taker_pays> <taker_gets> [<taker [<ledger> [<limit> [<proof> [<marker>]]]]]" << endl;