diff options
| author | Wladimir J. van der Laan <[email protected]> | 2015-07-18 07:42:23 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2015-07-18 09:03:59 +0200 |
| commit | e061e2778d592826970483e0844308c4e9a12626 (patch) | |
| tree | 9ef8f2de0cd1fb78a22675c24fa26d751117daae /src/rpcserver.cpp | |
| parent | Merge pull request #6372 (diff) | |
| download | discoin-e061e2778d592826970483e0844308c4e9a12626.tar.xz discoin-e061e2778d592826970483e0844308c4e9a12626.zip | |
rpc: Make ValueFromAmount always return 8 decimals
This is the format that was always returned to JSON clients.
The difference was not noticed before, because VREAL values
are post-processed by univalue.
By implementing the functionality directly it breaks the dependency
of rpcserver on utilmoneystr. FormatMoney is now only used for debugging
purposes.
To test, port over the formatting tests from util_tests.cpp to
rpc_tests.cpp.
Diffstat (limited to 'src/rpcserver.cpp')
| -rw-r--r-- | src/rpcserver.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/rpcserver.cpp b/src/rpcserver.cpp index bcad06a0c..c5402e0df 100644 --- a/src/rpcserver.cpp +++ b/src/rpcserver.cpp @@ -11,7 +11,6 @@ #include "sync.h" #include "ui_interface.h" #include "util.h" -#include "utilmoneystr.h" #include "utilstrencodings.h" #ifdef ENABLE_WALLET #include "wallet/wallet.h" @@ -133,7 +132,12 @@ CAmount AmountFromValue(const UniValue& value) UniValue ValueFromAmount(const CAmount& amount) { - return UniValue(UniValue::VREAL, FormatMoney(amount)); + bool sign = amount < 0; + int64_t n_abs = (sign ? -amount : amount); + int64_t quotient = n_abs / COIN; + int64_t remainder = n_abs % COIN; + return UniValue(UniValue::VNUM, + strprintf("%s%d.%08d", sign ? "-" : "", quotient, remainder)); } uint256 ParseHashV(const UniValue& v, string strName) |