Minor fixes

XRPLF · Oct 30, 2023 · ef1276c · ef1276c
1 parent c81bece
commit ef1276c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/ripple/app/ledger/impl/LedgerMaster.cpp b/src/ripple/app/ledger/impl/LedgerMaster.cpp
@@ -607,7 +607,7 @@ bool
 LedgerMaster::isValidated(ReadView const& ledger)
 {
     if (app_.config().reporting())
-        return haveLedger(ledger.seq());  // TODO Is this correct ?
+        return true;  // Reporting mode only supports validated ledger
 
     if (ledger.open())
         return false;

diff --git a/src/ripple/app/misc/NetworkOPs.cpp b/src/ripple/app/misc/NetworkOPs.cpp
@@ -3124,8 +3124,7 @@ NetworkOPsImp::transJson(
         jvObj[jss::transaction][jss::date] =
             ledger->info().closeTime.time_since_epoch().count();
         jvObj[jss::validated] = true;
-        if (auto close_time = m_ledgerMaster.getCloseTimeBySeq(ledger->seq()))
-            jvObj[jss::close_time_iso] = to_string_iso(*close_time);
+        jvObj[jss::close_time_iso] = to_string_iso(ledger->info().closeTime);
 
         // WRITEME: Put the account next seq here
     }
@@ -3174,10 +3173,11 @@ NetworkOPsImp::transJson(
         assert(index < MultiApiJson::size);
         if (index != lastIndex)
         {
+            lastIndex = index;
+
             Json::Value& jvTx = multiObj.val[index];
             RPC::insertDeliverMax(
                 jvTx[jss::transaction], transaction->getTxnType(), apiVersion);
-            lastIndex = index;
 
             if (apiVersion > 1)
             {