CRM-19126 further fix - less aggressive return

diff --git a/CRM/Contribute/BAO/Contribution.php b/CRM/Contribute/BAO/Contribution.php
index b6c0afa3b145f781dc26cb537079087764515879..68bd16345905a4e5cfb91b002e73b979cb976550 100644 (file)
--- a/CRM/Contribute/BAO/Contribution.php
+++ b/CRM/Contribute/BAO/Contribution.php
@@ -4125,26 +4125,22 @@ WHERE eft.financial_trxn_id IN ({$trxnId}, {$baseTrxnId['financialTrxnId']})
       if (empty($params['prevContribution'])) {
         $params['prevContribution'] = self::getOriginalContribution($params['id']);
       }
-      $isRequireTaxCalculation = FALSE;
-      foreach (array('total_amount', 'financial_type_id', 'fee_amount', 'tax_amount') as $field) {
+
+      foreach (array('total_amount', 'financial_type_id', 'fee_amount') as $field) {
         if (!isset($params[$field])) {
           if ($field == 'total_amount' && $params['prevContribution']->tax_amount) {
             // Tax amount gets added back on later....
             $params['total_amount'] = $params['prevContribution']->total_amount -
               $params['prevContribution']->tax_amount;
-            $isRequireTaxCalculation = TRUE;
           }
           else {
             $params[$field] = $params['prevContribution']->$field;
             if ($params[$field] != $params['prevContribution']->$field) {
-              $isRequireTaxCalculation = TRUE;
             }
           }
         }
       }
-      if (!$isRequireTaxCalculation) {
-        return $params;
-      }
+
       self::calculateMissingAmountParams($params, $params['id']);
       if (!array_key_exists($params['financial_type_id'], $taxRates)) {
         // Assign tax Amount on update of contribution

diff --git a/tests/phpunit/api/v3/ContributionTest.php b/tests/phpunit/api/v3/ContributionTest.php
index 9617a2ea611e797d16ad5adf6139855a8ffa694e..cad444a6b96365998f22e3efe06d62777380a68f 100644 (file)
--- a/tests/phpunit/api/v3/ContributionTest.php
+++ b/tests/phpunit/api/v3/ContributionTest.php
@@ -1786,7 +1786,7 @@ class api_v3_ContributionTest extends CiviUnitTestCase {
       'account_relationship' => 10,
       'financial_account_id' => $financialAccountId,
     );
-    $financialRelation = CRM_Financial_BAO_FinancialTypeAccount::add($financialAccountParams);
+    CRM_Financial_BAO_FinancialTypeAccount::add($financialAccountParams);
     $taxRates = CRM_Core_PseudoConstant::getTaxRates();
     $params = array_merge($this->_params, array('contribution_status_id' => 2, 'financial_type_id' => $financialTypeId));
     $contribution = $this->callAPISuccess('contribution', 'create', $params);

diff --git a/tests/phpunit/api/v3/TaxContributionPageTest.php b/tests/phpunit/api/v3/TaxContributionPageTest.php
index c6d307899a07e4348db0a23214d5aebb0f472965..c897e8bb5b595ae1676ba6299b286fc5294c26b0 100644 (file)
--- a/tests/phpunit/api/v3/TaxContributionPageTest.php
+++ b/tests/phpunit/api/v3/TaxContributionPageTest.php
@@ -361,7 +361,8 @@ class api_v3_TaxContributionPageTest extends CiviUnitTestCase {
   }
 
   /**
-   * Updation of contrbution.
+   * Update a contribution.
+   *
    * Function tests that line items, financial records are updated when contribution amount is changed
    */
   public function testCreateUpdateContributionChangeTotal() {
@@ -389,7 +390,7 @@ class api_v3_TaxContributionPageTest extends CiviUnitTestCase {
       'financial_type_id' => 1, // without tax rate i.e Donation
       'total_amount' => '300',
     );
-    $contribution = $this->callAPISuccess('contribution', 'update', $newParams);
+    $contribution = $this->callAPISuccess('contribution', 'create', $newParams);
 
     $lineItems = $this->callAPISuccess('line_item', 'getvalue', array(
       'entity_id' => $contribution['id'],