Skip to content
Snippets Groups Projects
Commit 23116573 authored by catch's avatar catch
Browse files

Issue #3181173 by longwave, pefferen, JD_1, larowlan: Drupal 8.9.7 breaks Book...

Issue #3181173 by longwave, pefferen, JD_1, larowlan: Drupal 8.9.7 breaks Book Settings Form Validation
parent 58db3cd8
No related branches found
No related tags found
No related merge requests found
......@@ -55,7 +55,7 @@ public function buildForm(array $form, FormStateInterface $form_state) {
* {@inheritdoc}
*/
public function validateForm(array &$form, FormStateInterface $form_state) {
$child_type = array_filter($form_state->getValue('book_child_type'));
$child_type = $form_state->getValue('book_child_type');
if ($form_state->isValueEmpty(['book_allowed_types', $child_type])) {
$form_state->setErrorByName('book_child_type', $this->t('The content type for the %add-child link must be one of those selected as an allowed book outline type.', ['%add-child' => $this->t('Add child page')]));
}
......@@ -75,7 +75,7 @@ public function submitForm(array &$form, FormStateInterface $form_state) {
$this->config('book.settings')
// Remove unchecked types.
->set('allowed_types', $allowed_types)
->set('child_type', array_filter($form_state->getValue('book_child_type')))
->set('child_type', $form_state->getValue('book_child_type'))
->save();
parent::submitForm($form, $form_state);
......
......@@ -712,4 +712,12 @@ public function testBookNavigationBlockOnUnpublishedBook() {
$this->assertText($this->book->label(), 'Unpublished book with "Show block only on book pages" book navigation settings.');
}
/**
* Tests that the book settings form can be saved without error.
*/
public function testSettingsForm() {
$this->drupalLogin($this->adminUser);
$this->drupalPostForm('admin/structure/book/settings', [], 'Save configuration');
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment