[NONEVM-3400] [CL86-18] Unnecessary Use Of replaceAndGetPrevious() #494

Author: patricios-space

contracts/contracts/ccip/fee_quoter/contract.tolk

@@ -134,7 +134,7 @@ fun updateTokenTransferFeeConfigs(mutate st: Storage, msg: FeeQuoter_UpdateToken
                 destChainConfig.tokenTransferFeeConfigs.delete(tokenAddr);
             }
 
-            st.destChainConfigs.replaceAndGetPrevious(destChainSelector, destChainConfig);
+            st.destChainConfigs.replaceIfExists(destChainSelector, destChainConfig);
         }
         entry = msg.updates.iterateNext(entry);
     }
@@ -197,7 +197,7 @@ fun updatePrices(mutate st: Storage, priceUpdates: PriceUpdates) {
                 dataAvailabilityGasPrice: update.dataAvailabilityGasPrice,
                 timestamp
             }.toCell();
-            st.destChainConfigs.replaceAndGetPrevious(update.destChainSelector, destChainConfig);
+            st.destChainConfigs.replaceIfExists(update.destChainSelector, destChainConfig);
 
             emit(USD_PER_UNIT_GAS_UPDATED, UsdPerUnitGasUpdated {
                 destChainSelector: update.destChainSelector,

contracts/contracts/ccip/onramp/contract.tolk

@@ -394,7 +394,7 @@ fun onApplyDestChainConfigUpdates(mutate st: OnRamp_Storage, updates: SnakedCell
             config = entry.loadValue();
             config.router = update.router;
             config.allowlistEnabled = update.allowlistEnabled;
-            st.destChainConfigs.replaceAndGetPrevious(update.destChainSelector, config);
+            st.destChainConfigs.replaceIfExists(update.destChainSelector, config);
         } else {
             config = OnRamp_DestChainConfig {
                 router: update.router,