diff -u b/core/modules/aggregator/src/FeedForm.php b/core/modules/aggregator/src/FeedForm.php --- b/core/modules/aggregator/src/FeedForm.php +++ b/core/modules/aggregator/src/FeedForm.php @@ -39,3 +39,2 @@ - } diff -u b/core/modules/search/src/Form/SearchPageAddForm.php b/core/modules/search/src/Form/SearchPageAddForm.php --- b/core/modules/search/src/Form/SearchPageAddForm.php +++ b/core/modules/search/src/Form/SearchPageAddForm.php @@ -42,7 +42,7 @@ $this->searchPageRepository->setDefaultSearchPage($this->entity); } - $status = $this->entity->save(); + $status = parent::save($form, $form_state); drupal_set_message($this->t('The %label search page has been added.', array('%label' => $this->entity->label()))); return $status; diff -u b/core/modules/search/src/Form/SearchPageEditForm.php b/core/modules/search/src/Form/SearchPageEditForm.php --- b/core/modules/search/src/Form/SearchPageEditForm.php +++ b/core/modules/search/src/Form/SearchPageEditForm.php @@ -27,7 +27,7 @@ * {@inheritdoc} */ public function save(array $form, FormStateInterface $form_state) { - $status = $this->entity->save(); + $status = parent::save($form, $form_state); drupal_set_message($this->t('The %label search page has been updated.', array('%label' => $this->entity->label()))); return $status; diff -u b/core/modules/system/tests/modules/entity_test/src/EntityTestForm.php b/core/modules/system/tests/modules/entity_test/src/EntityTestForm.php --- b/core/modules/system/tests/modules/entity_test/src/EntityTestForm.php +++ b/core/modules/system/tests/modules/entity_test/src/EntityTestForm.php @@ -71,19 +71,12 @@ } drupal_set_message($message); - if ($entity->id()) { - $entity_type = $entity->getEntityTypeId(); - $form_state->setRedirect( - "entity.$entity_type.edit_form", - array($entity_type => $entity->id()) - ); - return $status; - } - else { - // Error on save. - drupal_set_message(t('The entity could not be saved.'), 'error'); - $form_state->setRebuild(); - } + $entity_type = $entity->getEntityTypeId(); + $form_state->setRedirect( + "entity.$entity_type.edit_form", + array($entity_type => $entity->id()) + ); + return $status; } catch (\Exception $e) { \Drupal::state()->set('entity_test.form.save.exception', get_class($e) . ': ' . $e->getMessage()); only in patch2: unchanged: --- a/core/modules/system/tests/modules/entity_test/src/Entity/EntityTestMulChanged.php +++ b/core/modules/system/tests/modules/entity_test/src/Entity/EntityTestMulChanged.php @@ -70,7 +70,7 @@ public static function baseFieldDefinitions(EntityTypeInterface $entity_type) { public function save() { // Ensure a new timestamp. sleep(1); - parent::save(); + return parent::save(); } /**