Issue #2459753 follow-up by tim.plunkett, alexpott: EntityForm::validate()...

Issue #2459753 follow-up by tim.plunkett, alexpott: EntityForm::validate() should be able to modify the form structure

core/lib/Drupal/Core/Entity/ContentEntityConfirmFormBase.php

@@ -121,7 +121,7 @@ public function delete(array $form, FormStateInterface $form_state) {}
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     // Override the default validation implementation as it is not necessary
     // nor possible to validate an entity in a confirmation form.
   }

core/lib/Drupal/Core/Entity/ContentEntityForm.php

@@ -74,7 +74,7 @@ public function form(array $form, FormStateInterface $form_state) {
    * For more information about entity validation, see
    * https://www.drupal.org/node/2015613.
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     $entity = $this->buildEntity($form, $form_state);
     $this->getFormDisplay($form_state)->validateFormValues($entity, $form, $form_state);
 

core/lib/Drupal/Core/Entity/EntityForm.php

@@ -247,7 +247,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     // @todo Remove this.
     // Execute legacy global validation handlers.
     $form_state->setValidateHandlers([]);

core/lib/Drupal/Core/Entity/EntityFormInterface.php

@@ -105,7 +105,7 @@ public function buildEntity(array $form, FormStateInterface $form_state);
    * @return \Drupal\Core\Entity\ContentEntityTypeInterface
    *   The built entity.
    */
-  public function validate(array $form, FormStateInterface $form_state);
+  public function validate(array &$form, FormStateInterface $form_state);
 
   /**
    * Form submission handler for the 'save' action.

core/modules/action/src/ActionFormBase.php

@@ -123,7 +123,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     if ($this->plugin instanceof PluginFormInterface) {

core/modules/block/src/BlockForm.php

@@ -273,7 +273,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // The Block Entity form puts all block plugin form elements in the

core/modules/block_content/src/BlockContentForm.php

@@ -223,7 +223,7 @@ public function save(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validateForm(array &$form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     if ($this->entity->isNew()) {
       $exists = $this->blockContentStorage->loadByProperties(array('info' => $form_state->getValue(['info', 0, 'value'])));
       if (!empty($exists)) {

core/modules/comment/src/CommentForm.php

@@ -314,7 +314,7 @@ public function buildEntity(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     $comment = parent::validate($form, $form_state);
 
     // Customly trigger validation of manually added fields and add in

core/modules/contact/src/ContactFormEditForm.php

@@ -112,7 +112,7 @@ public function form(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // Validate and each email recipient.

core/modules/contact/src/MessageForm.php

@@ -187,7 +187,7 @@ public function preview(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     $message = $this->entity;

core/modules/field_ui/src/Form/EntityDisplayModeAddForm.php

@@ -38,7 +38,7 @@ public function buildForm(array $form, FormStateInterface $form_state, $entity_t
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     $form_state->setValueForElement($form['id'], $this->targetEntityTypeId . '.' . $form_state->getValue('id'));

core/modules/field_ui/src/Form/FieldConfigEditForm.php

@@ -150,7 +150,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     if (isset($form['default_value'])) {

core/modules/field_ui/src/Form/FieldStorageConfigEditForm.php

@@ -146,7 +146,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // Validate field cardinality.

core/modules/filter/src/FilterFormatFormBase.php

@@ -204,7 +204,7 @@ public function exists($format_id) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // @todo Move trimming upstream.

core/modules/node/src/NodeForm.php

@@ -286,7 +286,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     $node = parent::validate($form, $form_state);
 
     if ($node->id() && (node_last_changed($node->id(), $this->getFormLangcode($form_state)) > $node->getChangedTime())) {

core/modules/node/src/NodeTypeForm.php

@@ -204,7 +204,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     $id = trim($form_state->getValue('type'));

core/modules/responsive_image/src/ResponsiveImageStyleForm.php

@@ -133,7 +133,7 @@ public function form(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     // Only validate on edit.
     if ($form_state->hasValue('keyed_styles')) {
       // Check if another breakpoint group is selected.

core/modules/search/src/Form/SearchPageFormBase.php

@@ -144,7 +144,7 @@ public function exists($id) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // Ensure each path is unique.

core/modules/system/src/Form/DateFormatFormBase.php

@@ -155,7 +155,7 @@ public function form(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // The machine name field should already check to see if the requested

core/modules/taxonomy/src/TermForm.php

@@ -94,7 +94,7 @@ public function form(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     // Ensure numeric values.

core/modules/user/src/AccountForm.php

@@ -354,7 +354,7 @@ public function buildEntity(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     /** @var \Drupal\user\UserInterface $account */
     $account = parent::validate($form, $form_state);
 

core/modules/views_ui/src/ViewAddForm.php

@@ -162,7 +162,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     $wizard_type = $form_state->getValue(array('show', 'wizard_key'));
     $wizard_instance = $this->wizardManager->createInstance($wizard_type);
     $form_state->set('wizard', $wizard_instance->getPluginDefinition());

core/modules/views_ui/src/ViewEditForm.php

@@ -258,7 +258,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   /**
    * {@inheritdoc}
    */
-  public function validate(array $form, FormStateInterface $form_state) {
+  public function validate(array &$form, FormStateInterface $form_state) {
     parent::validate($form, $form_state);
 
     $view = $this->entity;

core/tests/Drupal/Tests/Core/Entity/EntityFormTest.php

@@ -9,6 +9,7 @@
 
 use Drupal\Core\Entity\EntityForm;
 use Drupal\Core\Form\FormState;
+use Drupal\Core\Form\FormStateInterface;
 use Drupal\Tests\UnitTestCase;
 
 /**
@@ -123,4 +124,20 @@ public function testCopyFormValuesToEntity() {
     $this->assertNull($result->get('key_controlled_by_plugin_collection'));
   }
 
+  /**
+   * @covers ::validate
+   */
+  public function testValidate() {
+    $form_object = $this->getMock('Drupal\Core\Entity\EntityFormInterface');
+    $form_object->expects($this->once())
+      ->method('validate')
+      ->willReturnCallback(function (array &$form, FormStateInterface $form_state) {
+        $form['foo'] = 'bar';
+      });
+    $form['baz'] = 'bim';
+    $form_state = new FormState();
+    $form_object->validate($form, $form_state);
+    $this->assertArrayHasKey('foo', $form);
+  }
+
 }