Remove inLedger from API version 2

XRPLF · Oct 25, 2023 · c4a83d3 · c4a83d3
1 parent aa7d8ca
commit c4a83d3
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 12 deletions.
diff --git a/src/ripple/app/misc/Transaction.h b/src/ripple/app/misc/Transaction.h
@@ -27,7 +27,7 @@
 #include <ripple/protocol/STTx.h>
 #include <ripple/protocol/TER.h>
 #include <ripple/protocol/TxMeta.h>
-#include <boost/optional.hpp>
+
 #include <optional>
 #include <variant>
 
@@ -310,6 +310,7 @@ class Transaction : public std::enable_shared_from_this<Transaction>,
     getJson(
         JsonOptions options,
         bool binary = false,
+        bool showInLedger = false,
         std::optional<std::reference_wrapper<std::string>> hash = {}) const;
 
     // Information used to locate a transaction.

diff --git a/src/ripple/app/misc/impl/Transaction.cpp b/src/ripple/app/misc/impl/Transaction.cpp
@@ -168,13 +168,16 @@ Json::Value
 Transaction::getJson(
     JsonOptions options,
     bool binary,
+    bool showInLedger,
     std::optional<std::reference_wrapper<std::string>> hash) const
 {
     Json::Value ret(mTransaction->getJson(JsonOptions::none, binary, hash));
 
     if (mInLedger)
     {
-        ret[jss::inLedger] = mInLedger;  // Deprecated.
+        if (showInLedger)
+            ret[jss::inLedger] = mInLedger;  // Deprecated.
+
         ret[jss::ledger_index] = mInLedger;
 
         if (options == JsonOptions::include_date)

diff --git a/src/ripple/rpc/handlers/AccountTx.cpp b/src/ripple/rpc/handlers/AccountTx.cpp
@@ -332,12 +332,15 @@ populateJsonResponse(
                     {
                         std::string hash;
                         jvObj[json_tx] = txn->getJson(
-                            JsonOptions::include_date, false, {std::ref(hash)});
+                            JsonOptions::include_date,
+                            false,
+                            false,
+                            {std::ref(hash)});
                         jvObj[jss::hash] = hash;
                     }
                     else
-                        jvObj[json_tx] =
-                            txn->getJson(JsonOptions::include_date);
+                        jvObj[json_tx] = txn->getJson(
+                            JsonOptions::include_date, false, true);
 
                     auto const& sttx = txn->getSTransaction();
                     RPC::insertDeliverMax(

diff --git a/src/ripple/rpc/handlers/Tx.cpp b/src/ripple/rpc/handlers/Tx.cpp
@@ -328,11 +328,11 @@ populateJsonResponse(
             std::string hash;
             if (args.binary)
                 response[jss::tx_blob] = result.txn->getJson(
-                    JsonOptions::include_date, true, {std::ref(hash)});
+                    JsonOptions::include_date, true, false, {std::ref(hash)});
             else
             {
                 response[jss::tx_json] = result.txn->getJson(
-                    JsonOptions::include_date, false, {std::ref(hash)});
+                    JsonOptions::include_date, false, false, {std::ref(hash)});
                 RPC::insertDeliverMax(
                     response[jss::tx_json],
                     sttx->getTxnType(),
@@ -345,8 +345,8 @@ populateJsonResponse(
         }
         else
         {
-            response =
-                result.txn->getJson(JsonOptions::include_date, args.binary);
+            response = result.txn->getJson(
+                JsonOptions::include_date, args.binary, true);
             if (!args.binary)
                 RPC::insertDeliverMax(
                     response, sttx->getTxnType(), context.apiVersion);

diff --git a/src/ripple/rpc/impl/TransactionSign.cpp b/src/ripple/rpc/impl/TransactionSign.cpp
@@ -653,12 +653,13 @@ transactionFormatResultImpl(Transaction::pointer tpTrans, unsigned apiVersion)
         if (apiVersion > 1)
         {
             std::string hash = {};
-            jvResult[jss::tx_json] =
-                tpTrans->getJson(JsonOptions::none, false, {std::ref(hash)});
+            jvResult[jss::tx_json] = tpTrans->getJson(
+                JsonOptions::none, false, false, {std::ref(hash)});
             jvResult[jss::hash] = hash;
         }
         else
-            jvResult[jss::tx_json] = tpTrans->getJson(JsonOptions::none);
+            jvResult[jss::tx_json] =
+                tpTrans->getJson(JsonOptions::none, false, true);
 
         jvResult[jss::tx_blob] =
             strHex(tpTrans->getSTransaction()->getSerializer().peekData());