Fix regression on frequency unit handling in 5.69
authorEileen McNaughton <emcnaughton@wikimedia.org>
Sun, 17 Dec 2023 21:38:00 +0000 (10:38 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Tue, 19 Dec 2023 01:18:33 +0000 (14:18 +1300)
CRM/Core/Payment/Dummy.php

index 3bd2318dc797a7efc7f505649385976c4be6b9e3..57a963cbc8196174298ddd7298d9091d424f8d16 100644 (file)
@@ -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);