From 19393e516538d2c21c99987d73aa1a40becbbf14 Mon Sep 17 00:00:00 2001 From: Pradeep Nayak Date: Fri, 11 Sep 2015 19:59:23 +0530 Subject: [PATCH] CRM-17208, moved $params before pre hook so that it can be altered in pre hook ---------------------------------------- * CRM-17208: Allow $params to be changed in pre hook https://issues.civicrm.org/jira/browse/CRM-17208 --- CRM/Contribute/BAO/Contribution.php | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/CRM/Contribute/BAO/Contribution.php b/CRM/Contribute/BAO/Contribution.php index 0305ee83c2..c4f871113b 100644 --- a/CRM/Contribute/BAO/Contribution.php +++ b/CRM/Contribute/BAO/Contribution.php @@ -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 -- 2.25.1