Skip to content

Commit ec3b040

Browse files
committed
Add per event-type message templates / defaults dpi#166
1 parent 5b53d43 commit ec3b040

File tree

3 files changed

+13
-17
lines changed

3 files changed

+13
-17
lines changed

src/Entity/EventType.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
* "edit" = "Drupal\rng\Form\EventTypeForm",
2525
* "delete" = "Drupal\rng\Form\EventTypeDeleteForm",
2626
* "event_access_defaults" = "Drupal\rng\Form\EventTypeAccessDefaultsForm",
27-
* "event_default_messages" = "Drupal\rng\Form\EventTypeDefaultMessagesForm",
27+
* "event_default_messages" = "Drupal\rng\Form\EventTypeDefaultMessagesListForm",
2828
* "field_mapping" = "Drupal\rng\Form\EventTypeFieldMappingForm",
2929
* }
3030
* },

src/Form/EventTypeDefaultMessagesAddForm.php

-6
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,6 @@ public function buildForm(array $form, FormStateInterface $form_state) {
5555
],
5656
];
5757

58-
$form['label'] = [
59-
'#type' => 'textfield',
60-
'#title' => $this->t('Label'),
61-
'#required' => TRUE,
62-
];
6358
$form['trigger'] = [
6459
'#type' => 'select',
6560
'#options' => $triggers,
@@ -96,7 +91,6 @@ public function buildForm(array $form, FormStateInterface $form_state) {
9691
public function submitForm(array &$form, FormStateInterface $form_state) {
9792
$messages = $this->eventType->getDefaultMessages();
9893
array_push($messages, [
99-
'label' => $form_state->getValue('label'),
10094
'trigger' => $form_state->getValue('trigger'),
10195
'status' => $form_state->getValue('status'),
10296
'subject' => $form_state->getValue('subject'),

src/Form/EventTypeDefaultMessagesForm.php renamed to src/Form/EventTypeDefaultMessagesListForm.php

+12-10
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
/**
1818
* Form for event type default message.
1919
*/
20-
class EventTypeDefaultMessagesForm extends EntityForm {
20+
class EventTypeDefaultMessagesListForm extends EntityForm {
2121

2222
/**
2323
* The redirect destination service.
@@ -112,13 +112,18 @@ public function buildForm(array $form, FormStateInterface $form_state) {
112112

113113
// @TODO : Move this and other occurences into a common place?.
114114
// @see EventTypeDefaultMessagesAddForm::buildForm.
115-
$triggers = [
115+
$trigger_options = [
116116
'rng:custom:date' => $this->t('To all registrations, on a date.'),
117117
(string) $this->t('Registrations') => [
118118
'entity:registration:new' => $this->t('To a single registration, when it is created.'),
119119
'entity:registration:update' => $this->t('To a single registration, when it is updated.'),
120120
],
121121
];
122+
$trigger_labels = [
123+
'entity:registration:new' => $this->t('Registration creation'),
124+
'entity:registration:update' => $this->t('Registration updated'),
125+
'rng:custom:date' => $this->t('Send on a date'),
126+
];
122127

123128
$form['messages'] = [
124129
'#type' => 'container',
@@ -131,17 +136,14 @@ public function buildForm(array $form, FormStateInterface $form_state) {
131136
$form['messages'][$key] = [
132137
'#type' => 'details',
133138
'#tree' => TRUE,
134-
'#title' => $message['label'],
135-
];
136-
$form['messages'][$key]['label'] = [
137-
'#type' => 'textfield',
138-
'#title' => $this->t('Label'),
139-
'#default_value' => $message['label'],
140-
'#required' => TRUE,
139+
'#title' => $this->t('@label (@status)', [
140+
'@label' => isset($trigger_labels[$message['trigger']]) ? $trigger_labels[$message['trigger']] : $message['trigger'],
141+
'@status' => $message['status'] ? $this->t('active') : $this->t('disabled'),
142+
]),
141143
];
142144
$form['messages'][$key]['trigger'] = [
143145
'#type' => 'select',
144-
'#options' => $triggers,
146+
'#options' => $trigger_options,
145147
'#title' => $this->t('Trigger'),
146148
'#default_value' => $message['trigger'],
147149
];

0 commit comments

Comments
 (0)