Skip to content
Snippets Groups Projects
Commit cb14170d authored by eileen's avatar eileen
Browse files

Make mapping_id & start_action_unit conditionally required

The fact they are not set for repeating events means they are not truly required
parent e9b99ff2
Branches
Tags
No related merge requests found
......@@ -27,10 +27,11 @@ class ActionScheduleSpecProvider extends \Civi\Core\Service\AutoService implemen
if ($spec->getAction() === 'create') {
$spec->getFieldByName('title')->setRequiredIf('empty($values.name)');
$spec->getFieldByName('name')->setRequired(FALSE);
$spec->getFieldByName('mapping_id')->setRequired(TRUE);
$spec->getFieldByName('entity_value')->setRequired(TRUE);
// Repeat events do not require mapping_id or start_action_unit - although
// we don't have that level of nuance available so we make them optional for all events.
$spec->getFieldByName('mapping_id')->setRequiredIf('empty($values.used_for) || $values.used_for !== "civicrm_event"');
$spec->getFieldByName('start_action_unit')->setRequiredIf('empty($values.absolute_date) && (empty($values.used_for) || $values.used_for !== "civicrm_event")');$spec->getFieldByName('entity_value')->setRequired(TRUE);
$spec->getFieldByName('start_action_offset')->setRequiredIf('empty($values.absolute_date)');
$spec->getFieldByName('start_action_unit')->setRequiredIf('empty($values.absolute_date)');
$spec->getFieldByName('start_action_condition')->setRequiredIf('empty($values.absolute_date)');
$spec->getFieldByName('start_action_date')->setRequiredIf('empty($values.absolute_date)');
$spec->getFieldByName('absolute_date')->setRequiredIf('empty($values.start_action_date)');
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment