Stop setting unused key
authorEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 20 Nov 2023 01:50:18 +0000 (14:50 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 20 Nov 2023 01:50:18 +0000 (14:50 +1300)
Since this function does not use this key & params is not passed by reference the value is set & discarded

CRM/Contribute/Form/Contribution/Main.php

index 90d939c4b15ae2ed8dd6536921be081c0f7fd6f7..8cc5bec5f570eaa7b502d027d660fa185b20bf58 100644 (file)
@@ -1166,11 +1166,6 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
         $amountID = $params['amount'] ?? NULL;
 
         if ($amountID) {
-          // @todo - stop setting amount level in this function & call the CRM_Price_BAO_PriceSet::getAmountLevel
-          // function to get correct amount level consistently. Remove setting of the amount level in
-          // CRM_Price_BAO_PriceSet::processAmount. Extend the unit tests in CRM_Price_BAO_PriceSetTest
-          // to cover all variants.
-          $params['amount_level'] = $formValues[$amountID]['label'] ?? NULL;
           $amount = $formValues[$amountID]['value'] ?? NULL;
         }
       }