[FOLD] Address @seelabs feedback

* Remove comments
* Refactor amendment changed code

Author: gregtatcam

src/ripple/app/misc/impl/AMMUtils.cpp

@@ -138,7 +138,6 @@ std::uint16_t
 getTradingFee(ReadView const& view, SLE const& ammSle, AccountID const& account)
 {
     using namespace std::chrono;
-    // should not happen
     assert(
         !view.rules().enabled(fixInnerObjTemplate) ||
         ammSle.isFieldPresent(sfAuctionSlot));
@@ -304,12 +303,22 @@ initializeFeeAuctionVote(
     // AMM creator gets the auction slot for free.
     // AuctionSlot is created on AMMCreate and updated on AMMDeposit
     // when AMM is in an empty state
-    if (!ammSle->isFieldPresent(sfAuctionSlot))
-    {
-        STObject auctionSlot = STObject::makeInnerObject(sfAuctionSlot, rules);
-        ammSle->set(&auctionSlot);
-    }
-    STObject& auctionSlot = ammSle->peekFieldObject(sfAuctionSlot);
+    STObject& auctionSlot = [&]() -> STObject& {
+        if (!rules.enabled(fixInnerObjTemplate))
+        {
+            return ammSle->peekFieldObject(sfAuctionSlot);
+        }
+        else
+        {
+            if (!ammSle->isFieldPresent(sfAuctionSlot))
+            {
+                STObject auctionSlot =
+                    STObject::makeInnerObject(sfAuctionSlot, rules);
+                ammSle->set(&auctionSlot);
+            }
+            return ammSle->peekFieldObject(sfAuctionSlot);
+        }
+    }();
     auctionSlot.setAccountID(sfAccount, account);
     // current + sec in 24h
     auto const expiration = std::chrono::duration_cast<std::chrono::seconds>(

src/ripple/app/tx/impl/AMMBid.cpp

@@ -173,15 +173,17 @@ applyBid(
         return {tecINTERNAL, false};
     STAmount const lptAMMBalance = (*ammSle)[sfLPTokenBalance];
     auto const lpTokens = ammLPHolds(sb, *ammSle, account_, ctx_.journal);
-    // should not happen
-    assert(
-        !ctx_.view().rules().enabled(fixInnerObjTemplate) ||
-        ammSle->isFieldPresent(sfAuctionSlot));
-    if (!ammSle->isFieldPresent(sfAuctionSlot))
+    auto const& rules = ctx_.view().rules();
+    if (!rules.enabled(fixInnerObjTemplate))
     {
-        STObject auctionSlot =
-            STObject::makeInnerObject(sfAuctionSlot, ctx_.view().rules());
-        ammSle->set(&auctionSlot);
+        if (!ammSle->isFieldPresent(sfAuctionSlot))
+            ammSle->makeFieldPresent(sfAuctionSlot);
+    }
+    else
+    {
+        assert(ammSle->isFieldPresent(sfAuctionSlot));
+        if (!ammSle->isFieldPresent(sfAuctionSlot))
+            return {tecINTERNAL, false};
     }
     auto& auctionSlot = ammSle->peekFieldObject(sfAuctionSlot);
     auto const current =

src/ripple/app/tx/impl/AMMVote.cpp

@@ -200,7 +200,6 @@ applyVote(
         }
     }
 
-    // should not happen
     assert(
         !ctx_.view().rules().enabled(fixInnerObjTemplate) ||
         ammSle->isFieldPresent(sfAuctionSlot));

src/ripple/rpc/handlers/AMMInfo.cpp

@@ -200,7 +200,6 @@ doAMMInfo(RPC::JsonContext& context)
     }
     if (voteSlots.size() > 0)
         ammResult[jss::vote_slots] = std::move(voteSlots);
-    // should not happen
     assert(
         !ledger->rules().enabled(fixInnerObjTemplate) ||
         amm->isFieldPresent(sfAuctionSlot));