From 7770bc511c8f4b1b68513133ed1e4cf4b5ef7860 Mon Sep 17 00:00:00 2001
From: Chenna Keshava <ckeshavabs@gmail.com>
Date: Thu, 2 Nov 2023 13:16:44 -0700
Subject: [PATCH] simplify the extraction of transactionID from Transaction
 object

---
 src/ripple/rpc/handlers/AccountTx.cpp   | 3 +--
 src/ripple/rpc/handlers/Tx.cpp          | 3 +--
 src/ripple/rpc/impl/TransactionSign.cpp | 3 +--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/ripple/rpc/handlers/AccountTx.cpp b/src/ripple/rpc/handlers/AccountTx.cpp
index 85faf02e918..005d2053b99 100644
--- a/src/ripple/rpc/handlers/AccountTx.cpp
+++ b/src/ripple/rpc/handlers/AccountTx.cpp
@@ -333,8 +333,7 @@ populateJsonResponse(
                             JsonOptions::include_date |
                                 JsonOptions::disable_API_prior_V2,
                             false);
-                        jvObj[jss::hash] = to_string(
-                            txn->getSTransaction()->getTransactionID());
+                        jvObj[jss::hash] = to_string(txn->getID());
                         jvObj[jss::ledger_index] = txn->getLedger();
                         jvObj[jss::ledger_hash] =
                             to_string(context.ledgerMaster.getHashBySeq(
diff --git a/src/ripple/rpc/handlers/Tx.cpp b/src/ripple/rpc/handlers/Tx.cpp
index 1f885af2f3d..0237fef22ac 100644
--- a/src/ripple/rpc/handlers/Tx.cpp
+++ b/src/ripple/rpc/handlers/Tx.cpp
@@ -349,8 +349,7 @@ populateJsonResponse(
             if (result.ledgerHash)
                 response[jss::ledger_hash] = to_string(*result.ledgerHash);
 
-            response[jss::hash] =
-                to_string(result.txn->getSTransaction()->getTransactionID());
+            response[jss::hash] = to_string(result.txn->getID());
             if (result.validated)
             {
                 response[jss::ledger_index] = result.txn->getLedger();
diff --git a/src/ripple/rpc/impl/TransactionSign.cpp b/src/ripple/rpc/impl/TransactionSign.cpp
index 7c619290d7b..48a9c66d81c 100644
--- a/src/ripple/rpc/impl/TransactionSign.cpp
+++ b/src/ripple/rpc/impl/TransactionSign.cpp
@@ -654,8 +654,7 @@ transactionFormatResultImpl(Transaction::pointer tpTrans, unsigned apiVersion)
         {
             jvResult[jss::tx_json] =
                 tpTrans->getJson(JsonOptions::disable_API_prior_V2);
-            jvResult[jss::hash] =
-                to_string(tpTrans->getSTransaction()->getTransactionID());
+            jvResult[jss::hash] = to_string(tpTrans->getID());
         }
         else
             jvResult[jss::tx_json] = tpTrans->getJson(JsonOptions::none);