From 4054a23fe5731dc734d5b36778df913c057df411 Mon Sep 17 00:00:00 2001 From: David Reedy Jr Date: Tue, 10 Jan 2017 21:26:58 -0600 Subject: [PATCH] CRM-19853 replace interval controls with numeric --- CRM/Admin/Form/ScheduleReminders.php | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/CRM/Admin/Form/ScheduleReminders.php b/CRM/Admin/Form/ScheduleReminders.php index 9df937354d..bda05dee48 100644 --- a/CRM/Admin/Form/ScheduleReminders.php +++ b/CRM/Admin/Form/ScheduleReminders.php @@ -161,10 +161,10 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form { //get the frequency units. $this->_freqUnits = CRM_Core_SelectValues::getRecurringFrequencyUnits(); - $numericOptions = CRM_Core_SelectValues::getNumericOptions(0, 30); - //reminder_interval - $this->add('select', 'start_action_offset', ts('When'), $numericOptions); + $this->add('number', 'start_action_offset', ts('When'), array('style' => 'width:5em')); + $this->addRule('start_action_offset', ts('Value should be a positive number'), 'positiveInteger'); + $isActive = ts('Send email'); $recordActivity = ts('Record activity for automated email'); if ($providersCount) { @@ -208,9 +208,13 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form { ); $this->add('select', 'repetition_frequency_unit', ts('every'), $freqUnitsDisplay); - $this->add('select', 'repetition_frequency_interval', ts('every'), $numericOptions); + $this->add('number', 'repetition_frequency_interval', ts('every'), array('style' => 'width:5em')); + $this->addRule('repetition_frequency_interval', ts('Value should be a positive number'), 'positiveInteger'); + $this->add('select', 'end_frequency_unit', ts('until'), $freqUnitsDisplay); - $this->add('select', 'end_frequency_interval', ts('until'), $numericOptions); + $this->add('number', 'end_frequency_interval', ts('until'), array('style' => 'width:5em')); + $this->addRule('end_frequency_interval', ts('Value should be a positive number'), 'positiveInteger'); + $this->add('select', 'end_action', ts('Repetition Condition'), $condition, TRUE); $this->add('select', 'end_date', ts('Date Field'), $selectedMapping->getDateFields(), TRUE); -- 2.25.1