diff --git a/CRM/Admin/Form/ScheduleReminders.php b/CRM/Admin/Form/ScheduleReminders.php index a7b94e70f4117787835a3b5ce925b72f86849016..c881ef779387a483a93801c760b8f851937dff6b 100644 --- a/CRM/Admin/Form/ScheduleReminders.php +++ b/CRM/Admin/Form/ScheduleReminders.php @@ -481,7 +481,6 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form { 'subject', 'absolute_date', 'group_id', - 'record_activity', 'limit_to', 'mode', 'sms_provider_id', @@ -492,7 +491,10 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form { $params[$key] = $values[$key] ?? NULL; } - $params['is_repeat'] = CRM_Utils_Array::value('is_repeat', $values, 0); + // set boolean fields to false if not set. + foreach (['record_activity', 'is_repeat', 'is_active'] as $boolFieldName) { + $params[$boolFieldName] = $values[$boolFieldName] ?? 0; + } $moreKeys = [ 'start_action_offset', @@ -559,8 +561,6 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form { $params['entity_status'] = implode(CRM_Core_DAO::VALUE_SEPARATOR, $entity_status); } - $params['is_active'] = CRM_Utils_Array::value('is_active', $values, 0); - if (empty($values['is_repeat'])) { $params['repetition_frequency_unit'] = 'null'; $params['repetition_frequency_interval'] = 'null';