CRM-17208, moved $params before pre hook so that it can be altered in pre hook
authorPradeep Nayak <pradpnayak@gmail.com>
Fri, 11 Sep 2015 14:29:23 +0000 (19:59 +0530)
committerPradeep Nayak <pradpnayak@gmail.com>
Fri, 11 Sep 2015 14:29:23 +0000 (19:59 +0530)
----------------------------------------
* CRM-17208: Allow $params to be changed in pre hook
  https://issues.civicrm.org/jira/browse/CRM-17208

CRM/Contribute/BAO/Contribution.php

index 0305ee83c250dd0c9b145a79cddbc721d49dfbe2..c4f871113bebffbe6d7b84ebf2e8cea63c5cec56 100644 (file)
@@ -160,6 +160,9 @@ class CRM_Contribute_BAO_Contribution extends CRM_Contribute_DAO_Contribution {
         $setPrevContribution = FALSE;
       }
     }
+    if ($contributionID && $setPrevContribution) {
+      $params['prevContribution'] = self::getValues(array('id' => $contributionID), CRM_Core_DAO::$_nullArray, CRM_Core_DAO::$_nullArray);
+    }
 
     if ($contributionID) {
       CRM_Utils_Hook::pre('edit', 'Contribution', $contributionID, $params);
@@ -177,10 +180,6 @@ class CRM_Contribute_BAO_Contribution extends CRM_Contribute_DAO_Contribution {
       $contribution->currency = $config->defaultCurrency;
     }
 
-    if ($contributionID && $setPrevContribution) {
-      $params['prevContribution'] = self::getValues(array('id' => $contributionID), CRM_Core_DAO::$_nullArray, CRM_Core_DAO::$_nullArray);
-    }
-
     $result = $contribution->save();
 
     // Add financial_trxn details as part of fix for CRM-4724