From 8536e5a08fafdc38356ea583c9b8c1cc9dd3dc44 Mon Sep 17 00:00:00 2001 From: eileen Date: Thu, 27 Oct 2016 21:15:10 +1300 Subject: [PATCH] CRM-19478 Handle contribution_page_id = 'null' oddity. I can't quite figure out the full flow here but this fix has been tested by Stoob & confirmed --- CRM/Contribute/BAO/Contribution.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/CRM/Contribute/BAO/Contribution.php b/CRM/Contribute/BAO/Contribution.php index 1237d2576f..3f6fc427ca 100644 --- a/CRM/Contribute/BAO/Contribution.php +++ b/CRM/Contribute/BAO/Contribution.php @@ -2148,7 +2148,8 @@ LEFT JOIN civicrm_contribution contribution ON ( componentPayment.contribution_ //CRM-18805 -- Contribution page not recorded on recurring transactions, Recurring contribution payments //do not create CC or BCC emails or profile notifications. //The if is just to be safe. Not sure if we can ever arrive with this unset - if (isset($contribution->contribution_page_id)) { + // but per CRM-19478 it seems it can be 'null' + if (isset($contribution->contribution_page_id) && is_numeric($contribution->contribution_page_id)) { $contributionParams['contribution_page_id'] = $contribution->contribution_page_id; } -- 2.25.1