Merge pull request #17473 from eileenmcnaughton/anet

[civicrm-core.git] / CRM / Admin / Form / ScheduleReminders.php

diff --git a/CRM/Admin/Form/ScheduleReminders.php b/CRM/Admin/Form/ScheduleReminders.php
index a7b94e70f4117787835a3b5ce925b72f86849016..c881ef779387a483a93801c760b8f851937dff6b 100644 (file)
--- 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';