Issue #3291737 by mxh: Adjust event implementations to new version of ECA

src/Event/VboExecuteEvent.php

@@ -4,13 +4,13 @@ namespace Drupal\eca_vbo\Event;
 
 use Drupal\Core\Action\ActionInterface;
 use Drupal\Core\Entity\EntityInterface;
-use Drupal\eca\Event\ContentEntityEventInterface;
+use Drupal\eca\Event\EntityEventInterface;
 use Drupal\views\ViewExecutable;
 
 /**
  * Dispatches when executing a "eca_vbo_execute" action.
  */
-class VboExecuteEvent extends VboEventBase implements ContentEntityEventInterface {
+class VboExecuteEvent extends VboEventBase implements EntityEventInterface {
 
   /**
    * The main entity of the Views row.

src/Plugin/ECA/Event/VboEvent.php

@@ -2,6 +2,7 @@
 
 namespace Drupal\eca_vbo\Plugin\ECA\Event;
 
+use Drupal\Core\Form\FormStateInterface;
 use Drupal\eca\Entity\Objects\EcaEvent;
 use Drupal\eca\Event\Tag;
 use Drupal\eca\Plugin\ECA\Event\EventBase;
@@ -63,35 +64,58 @@ class VboEvent extends EventBase {
   /**
    * {@inheritdoc}
    */
-  public function fields(): array {
-    $fields = [];
-    $fields[] = [
-      'name' => 'operation_name',
-      'label' => 'Operation name',
-      'type' => 'String',
-      'description' => '',
+  public function defaultConfiguration(): array {
+    return [
+      'operation_name' => '',
+      'view_id' => '',
+      'display_id' => '',
+    ];
+  }
+
+  /**
+   * {@inheritdoc}
+   */
+  public function buildConfigurationForm(array $form, FormStateInterface $form_state): array {
+    $form['operation_name'] = [
+      '#type' => 'textfield',
+      '#title' => $this->t('Operation name'),
+      '#default_value' => $this->configuration['operation_name'],
+      '#required' => TRUE,
+      '#weight' => 10,
     ];
-    end($fields);
-    $field = &$fields[key($fields)];
     if ($this->eventClass() === VboExecuteEvent::class) {
-      $field['description'] = 'The operation name identifies this process and will show up in the bulk operations configuration form as selectable action. If you need custom access handling using the operation name, then make sure that the Views configuration is using the <em>ECA bulk operations</em> Views field plugin.';
+      $form['operation_name']['#description'] = $this->t('The operation name identifies this process and will show up in the bulk operations configuration form as selectable action. If you need custom access handling using the operation name, then make sure that the Views configuration is using the <em>ECA bulk operations</em> Views field plugin.');
     }
     elseif ($this->eventClass() === VboCustomAccessEvent::class) {
-      $field['description'] = 'Important note: The operation name is only available, when the operation got executed from the <em>ECA bulk operations</em> Views field plugin.';
+      $form['operation_name']['#description'] = $this->t('Important note: The operation name is only available, when the operation got executed from the <em>ECA bulk operations</em> Views field plugin.');
     }
-    $fields[] = [
-      'name' => 'view_id',
-      'label' => 'Restrict by view ID',
-      'type' => 'String',
-      'description' => '',
+
+    $form['view_id'] = [
+      '#type' => 'textfield',
+      '#title' => $this->t('Restrict by view ID'),
+      '#default_value' => $this->configuration['view_id'],
+      '#required' => TRUE,
+      '#weight' => 20,
     ];
-    $fields[] = [
-      'name' => 'display_id',
-      'label' => 'Restrict by view display ID',
-      'type' => 'String',
-      'description' => '',
+
+    $form['display_id'] = [
+      '#type' => 'textfield',
+      '#title' => $this->t('Restrict by view display ID'),
+      '#default_value' => $this->configuration['display_id'],
+      '#required' => TRUE,
+      '#weight' => 30,
     ];
-    return $fields;
+
+    return $form;
+  }
+
+  /**
+   * {@inheritdoc}
+   */
+  public function submitConfigurationForm(array &$form, FormStateInterface $form_state): void {
+    $this->configuration['operation_name'] = $form_state->getValue('operation_name');
+    $this->configuration['view_id'] = $form_state->getValue('view_id');
+    $this->configuration['display_id'] = $form_state->getValue('display_id');
   }
 
   /**