From e62b3a8f51e8396f5658313954cc1884567bce6b Mon Sep 17 00:00:00 2001 From: Eileen McNaughton Date: Mon, 18 Dec 2023 10:38:00 +1300 Subject: [PATCH] Fix regression on frequency unit handling in 5.69 --- CRM/Core/Payment/Dummy.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/CRM/Core/Payment/Dummy.php b/CRM/Core/Payment/Dummy.php index 3bd2318dc7..57a963cbc8 100644 --- a/CRM/Core/Payment/Dummy.php +++ b/CRM/Core/Payment/Dummy.php @@ -223,7 +223,9 @@ class CRM_Core_Payment_Dummy extends CRM_Core_Payment { // does this, it needs to invoke this hook after it has done translation, // but before it actually starts talking to its proprietary back-end. if ($propertyBag->getIsRecur()) { - $throwAnENoticeIfNotSetAsTheseAreRequired = $propertyBag->getRecurFrequencyInterval() . $propertyBag->getRecurFrequencyUnit(); + if (empty($params['frequency_interval']) || empty($params['frequency_unit'])) { + CRM_Core_Error::deprecatedWarning('contracted frequency params not passed'); + } } // no translation in Dummy processor CRM_Utils_Hook::alterPaymentProcessorParams($this, $params, $propertyBag); -- 2.25.1