Set version to 0.30.1-hf1

This reverts commit 9fea06ad84.

src/ripple/app/paths/PathState.cpp

@@ -799,6 +799,19 @@ TER PathState::checkNoRipple (
             if (terStatus != tesSUCCESS)
                 return terStatus;
         }
+
+        if (!nodes_[i - 1].isAccount() &&
+            nodes_[i].isAccount() &&
+            nodes_[i + 1].isAccount() &&
+            nodes_[i -1].issue_.account != nodes_[i].account_)
+        { // offer -> account -> account
+            auto const& currencyID = nodes_[i].issue_.currency;
+            terStatus = checkNoRipple (
+                nodes_[i-1].issue_.account, nodes_[i].account_, nodes_[i+1].account_,
+                currencyID);
+            if (terStatus != tesSUCCESS)
+                return terStatus;
+        }
     }
 
     return tesSUCCESS;

src/ripple/app/paths/RippleCalc.cpp

@@ -32,7 +32,7 @@ namespace {
 static
 TER
 deleteOffers (ApplyView& view,
-    OfferSet const& offers, beast::Journal j)
+    std::set<uint256> const& offers, beast::Journal j)
 {
     for (auto& e: offers)
         if (TER r = offerDelete (view,
@@ -201,7 +201,7 @@ TER RippleCalc::rippleCalculate ()
             getRate (saDstAmountReq_, saMaxAmountReq_) : 0;
 
     // Offers that became unfunded.
-    OfferSet unfundedOffersFromBestPaths;
+    std::set<uint256> unfundedOffersFromBestPaths;
 
     int iPass = 0;
 

src/ripple/app/paths/RippleCalc.h

@@ -106,10 +106,10 @@ public:
     PaymentSandbox& view;
 
     // If the transaction fails to meet some constraint, still need to delete
-    // unfunded offers.
+    // unfunded offers in a deterministic order (hence the ordered container).
     //
     // Offers that were found unfunded.
-    path::OfferSet permanentlyUnfundedOffers_;
+    std::set<uint256> permanentlyUnfundedOffers_;
 
     // First time working in reverse a funding source was mentioned.  Source may
     // only be used there.

src/ripple/app/paths/Types.h

@@ -30,8 +30,6 @@ namespace path {
 
 using NodeIndex = unsigned int;
 
-using OfferSet = hash_set <uint256>;
-
 }
 
 using AccountIssueToNodeIndex = hash_map <AccountIssue, path::NodeIndex>;

src/ripple/protocol/impl/BuildInfo.cpp

@@ -35,7 +35,7 @@ char const* getRawVersionString ()
     //
     //  The build version number (edit this for each release)
     //
-        "0.30.1-rc2"
+        "0.30.1-hf1"
     //
     //  Must follow the format described here:
     //

src/ripple/rpc/handlers/AccountTx.cpp

@@ -163,7 +163,7 @@ Json::Value doAccountTx (RPC::Context& context)
                 {
                     auto meta = it.second->getJson (1);
                     addPaymentDeliveredAmount (meta, context, it.first, it.second);
-                    jvObj[jss::meta] = meta;
+                    jvObj[jss::meta] = std::move(meta);
 
                     std::uint32_t uLedgerIndex = it.second->getLgrSeq ();
 

src/ripple/rpc/handlers/AccountTxOld.cpp

@@ -29,6 +29,7 @@
 #include <ripple/resource/Fees.h>
 #include <ripple/rpc/Context.h>
 #include <ripple/rpc/impl/LookupLedger.h>
+#include <ripple/rpc/impl/Utilities.h>
 #include <ripple/server/Role.h>
 
 namespace ripple {
@@ -183,7 +184,10 @@ Json::Value doAccountTxOld (RPC::Context& context)
                 {
                     std::uint32_t uLedgerIndex = it->second->getLgrSeq ();
 
-                    jvObj[jss::meta]           = it->second->getJson (0);
+                    auto meta = it->second->getJson(0);
+                    addPaymentDeliveredAmount(meta, context, it->first, it->second);
+                    jvObj[jss::meta] = std::move(meta);
+
                     jvObj[jss::validated]
                             = bValidated
                             && uValidatedMin <= uLedgerIndex

src/ripple/rpc/impl/Utilities.cpp

@@ -39,20 +39,29 @@ addPaymentDeliveredAmount (
 {
     // We only want to add a "delivered_amount" field if the transaction
     // succeeded - otherwise nothing could have been delivered.
-    if (!transaction || transaction->getResult () != tesSUCCESS)
+    if (! transaction)
         return;
 
     auto serializedTx = transaction->getSTransaction ();
-
-    if (!serializedTx || serializedTx->getTxnType () != ttPAYMENT)
+    if (! serializedTx || serializedTx->getTxnType () != ttPAYMENT)
         return;
 
-    // If the transaction explicitly specifies a DeliveredAmount in the
-    // metadata then we use it.
-    if (transactionMeta && transactionMeta->hasDeliveredAmount ())
+    if (transactionMeta)
+    {
+        if (transactionMeta->getResultTER() != tesSUCCESS)
+            return;
+
+        // If the transaction explicitly specifies a DeliveredAmount in the
+        // metadata then we use it.
+        if (transactionMeta->hasDeliveredAmount ())
+        {
+            meta[jss::delivered_amount] =
+                transactionMeta->getDeliveredAmount ().getJson (1);
+            return;
+        }
+    }
+    else if (transaction->getResult() != tesSUCCESS)
     {
-        meta[jss::delivered_amount] =
-            transactionMeta->getDeliveredAmount ().getJson (1);
         return;
     }