diff --git a/core/includes/form.inc b/core/includes/form.inc index 53e7c6b266c8654f29d17447b0f6a9514f4e0faa..11da7bfa14518877c46b36bf148d97c33dc90aef 100644 --- a/core/includes/form.inc +++ b/core/includes/form.inc @@ -135,18 +135,6 @@ function form_execute_handlers($type, &$form, FormStateInterface $form_state) { } } -/** - * Files an error against a form element. - * - * @deprecated in Drupal 8.x-dev, will be removed before Drupal 8.0. - * Use $form_state->setErrorByName(). - * - * @see \Drupal\Core\Form\FormStateInterface::setErrorByName(). - */ -function form_set_error($name, FormStateInterface $form_state, $message = '') { - $form_state->setErrorByName($name, $message); -} - /** * Returns an associative array of all errors. * diff --git a/core/modules/action/src/Plugin/Action/EmailAction.php b/core/modules/action/src/Plugin/Action/EmailAction.php index 1c67123f25aa32ffe1c5df163e18753f9249bacd..58409d1fc4a047868b7599311a944125412595b3 100644 --- a/core/modules/action/src/Plugin/Action/EmailAction.php +++ b/core/modules/action/src/Plugin/Action/EmailAction.php @@ -159,7 +159,7 @@ public function buildConfigurationForm(array $form, FormStateInterface $form_sta public function validateConfigurationForm(array &$form, FormStateInterface $form_state) { if (!valid_email_address($form_state->getValue('recipient')) && strpos($form_state->getValue('recipient'), ':mail') === FALSE) { // We want the literal %author placeholder to be emphasized in the error message. - form_set_error('recipient', $form_state, t('Enter a valid email address or use a token email address such as %author.', array('%author' => '[node:author:mail]'))); + $form_state->setErrorByName('recipient', t('Enter a valid email address or use a token email address such as %author.', array('%author' => '[node:author:mail]'))); } } diff --git a/core/modules/color/color.module b/core/modules/color/color.module index ae63364e9b50edf1ce2a28efa8b76ef6521cdde3..44a2a7acdf113ec069838589a00d0ff43e2e08ee 100644 --- a/core/modules/color/color.module +++ b/core/modules/color/color.module @@ -334,7 +334,7 @@ function color_scheme_form_validate($form, FormStateInterface $form_state) { // Only accept hexadecimal CSS color strings to avoid XSS upon use. foreach ($form_state->getValue('palette') as $key => $color) { if (!color_valid_hexadecimal_string($color)) { - form_set_error('palette][' . $key, $form_state, t('You must enter a valid hexadecimal color value for %name.', array('%name' => $form['color']['palette'][$key]['#title']))); + $form_state->setErrorByName('palette][' . $key, t('You must enter a valid hexadecimal color value for %name.', array('%name' => $form['color']['palette'][$key]['#title']))); } } } diff --git a/core/modules/content_translation/content_translation.admin.inc b/core/modules/content_translation/content_translation.admin.inc index 768b89169a0d47c0e931337d779026ae65cec8fe..36ac92e825689354467a6db5d209d9de6d5f5610 100644 --- a/core/modules/content_translation/content_translation.admin.inc +++ b/core/modules/content_translation/content_translation.admin.inc @@ -264,7 +264,7 @@ function content_translation_form_language_content_settings_validate(array $form $translatable_fields = isset($settings[$entity_type][$bundle]['fields']) ? array_filter($settings[$entity_type][$bundle]['fields']) : FALSE; if (empty($translatable_fields)) { $t_args = array('%bundle' => $form['settings'][$entity_type][$bundle]['settings']['#label']); - form_set_error($name, $form_state, t('At least one field needs to be translatable to enable %bundle for translation.', $t_args)); + $form_state->setErrorByName($name, t('At least one field needs to be translatable to enable %bundle for translation.', $t_args)); } $values = $bundle_settings['settings']['language']; @@ -272,7 +272,7 @@ function content_translation_form_language_content_settings_validate(array $form foreach (\Drupal::languageManager()->getLanguages(LanguageInterface::STATE_LOCKED) as $language) { $locked_languages[] = $language->name; } - form_set_error($name, $form_state, t('Translation is not supported if language is always one of: @locked_languages', array('@locked_languages' => implode(', ', $locked_languages)))); + $form_state->setErrorByName($name, t('Translation is not supported if language is always one of: @locked_languages', array('@locked_languages' => implode(', ', $locked_languages)))); } } } diff --git a/core/modules/content_translation/content_translation.module b/core/modules/content_translation/content_translation.module index d8b281952b885afa49254dff03d4d0c53d19d5ae..fe9042d15c69119e84649392a8b2792d99d9c0d7 100644 --- a/core/modules/content_translation/content_translation.module +++ b/core/modules/content_translation/content_translation.module @@ -753,7 +753,7 @@ function content_translation_language_configuration_element_validate($element, F // @todo Set the correct form element name as soon as the element parents // are correctly set. We should be using NestedArray::getValue() but for // now we cannot. - form_set_error('', $form_state, t('"Show language selector" is not compatible with translating content that has default language: %choice. Either do not hide the language selector or pick a specific language.', array('%choice' => $locked_languages[$values['langcode']]))); + $form_state->setErrorByName('', t('"Show language selector" is not compatible with translating content that has default language: %choice. Either do not hide the language selector or pick a specific language.', array('%choice' => $locked_languages[$values['langcode']]))); } } diff --git a/core/modules/content_translation/src/ContentTranslationHandler.php b/core/modules/content_translation/src/ContentTranslationHandler.php index cba41ae4e60898f72df2a0af38ee3cef64f6dcaf..c1f638f84aac05ca30877d34bea6dfdf923c6df5 100644 --- a/core/modules/content_translation/src/ContentTranslationHandler.php +++ b/core/modules/content_translation/src/ContentTranslationHandler.php @@ -420,11 +420,11 @@ function entityFormValidate($form, FormStateInterface $form_state) { $translation = $form_state->getValue('content_translation'); // Validate the "authored by" field. if (!empty($translation['name']) && !($account = user_load_by_name($translation['name']))) { - form_set_error('content_translation][name', $form_state, t('The translation authoring username %name does not exist.', array('%name' => $translation['name']))); + $form_state->setErrorByName('content_translation][name', t('The translation authoring username %name does not exist.', array('%name' => $translation['name']))); } // Validate the "authored on" field. if (!empty($translation['created']) && strtotime($translation['created']) === FALSE) { - form_set_error('content_translation][created', $form_state, t('You have to specify a valid translation authoring date.')); + $form_state->setErrorByName('content_translation][created', t('You have to specify a valid translation authoring date.')); } } } diff --git a/core/modules/editor/editor.module b/core/modules/editor/editor.module index 3e2804dcfe771d538b54e8bc7b6f46973243b178..ce3b9fa7d88cebfc7edfd13f96a7cd8b49edfaeb 100644 --- a/core/modules/editor/editor.module +++ b/core/modules/editor/editor.module @@ -207,7 +207,7 @@ function editor_form_filter_admin_format_validate($form, FormStateInterface $for // selected a text editor and has then clicked 'Save configuration', we should // point out that the user must still configure the text editor. if ($form_state->getValue(array('editor', 'editor')) !== '' && empty($form_state['editor'])) { - form_set_error('editor][editor', $form_state, t('You must configure the selected text editor.')); + $form_state->setErrorByName('editor][editor', t('You must configure the selected text editor.')); } } diff --git a/core/modules/file/file.module b/core/modules/file/file.module index 825594e6cd7bb028d865fc1f885d9566d1b809f3..af6ccab67e41916517edd8033906e075deb4c06f 100644 --- a/core/modules/file/file.module +++ b/core/modules/file/file.module @@ -1446,7 +1446,7 @@ function file_managed_file_save_upload($element, FormStateInterface $form_state) $destination = isset($element['#upload_location']) ? $element['#upload_location'] : NULL; if (isset($destination) && !file_prepare_directory($destination, FILE_CREATE_DIRECTORY)) { \Drupal::logger('file')->notice('The upload directory %directory for the file field !name could not be created or is not accessible. A newly uploaded file could not be saved in this directory as a consequence, and the upload was canceled.', array('%directory' => $destination, '!name' => $element['#field_name'])); - form_set_error($upload_name, $form_state, t('The file could not be uploaded.')); + $form_state->setErrorByName($upload_name, t('The file could not be uploaded.')); return FALSE; } @@ -1456,7 +1456,7 @@ function file_managed_file_save_upload($element, FormStateInterface $form_state) if ($files_uploaded) { if (!$files = file_save_upload($upload_name, $element['#upload_validators'], $destination)) { \Drupal::logger('file')->notice('The file upload failed. %upload', array('%upload' => $upload_name)); - form_set_error($upload_name, $form_state, t('Files in the !name field were unable to be uploaded.', array('!name' => $element['#title']))); + $form_state->setErrorByName($upload_name, t('Files in the !name field were unable to be uploaded.', array('!name' => $element['#title']))); return array(); } diff --git a/core/modules/forum/forum.module b/core/modules/forum/forum.module index d30c85b73ebef92bc9009110116ecf812d55120e..ed2f05a3d566c44d6ec341db57e2e4ca543a66e5 100644 --- a/core/modules/forum/forum.module +++ b/core/modules/forum/forum.module @@ -213,7 +213,7 @@ function forum_node_validate(EntityInterface $node, $form, FormStateInterface $f } $term = $item->entity; if (!$term) { - form_set_error('taxonomy_forums', $form_state, t('Select a forum.')); + $form_state->setErrorByName('taxonomy_forums', t('Select a forum.')); continue; } $used = \Drupal::entityQuery('taxonomy_term') @@ -223,7 +223,7 @@ function forum_node_validate(EntityInterface $node, $form, FormStateInterface $f ->count() ->execute(); if ($used && !empty($term->forum_container->value)) { - form_set_error('taxonomy_forums', $form_state, t('The item %forum is a forum container, not a forum. Select one of the forums below instead.', array('%forum' => $term->getName()))); + $form_state->setErrorByName('taxonomy_forums', t('The item %forum is a forum container, not a forum. Select one of the forums below instead.', array('%forum' => $term->getName()))); } } } diff --git a/core/modules/image/src/Plugin/ImageEffect/RotateImageEffect.php b/core/modules/image/src/Plugin/ImageEffect/RotateImageEffect.php index 078dbf6e993f9fc9078be17d58a60ce60a2ef039..399501147b783118e3b5e851df10aaecb2a10755 100644 --- a/core/modules/image/src/Plugin/ImageEffect/RotateImageEffect.php +++ b/core/modules/image/src/Plugin/ImageEffect/RotateImageEffect.php @@ -129,7 +129,7 @@ public function buildConfigurationForm(array $form, FormStateInterface $form_sta */ public function validateConfigurationForm(array &$form, FormStateInterface $form_state) { if (!$form_state->isValueEmpty('bgcolor') && !Color::validateHex($form_state->getValue('bgcolor'))) { - form_set_error('bgcolor', $form_state, $this->t('Background color must be a hexadecimal color value.')); + $form_state->setErrorByName('bgcolor', $this->t('Background color must be a hexadecimal color value.')); } } diff --git a/core/modules/image/src/Plugin/ImageEffect/ScaleImageEffect.php b/core/modules/image/src/Plugin/ImageEffect/ScaleImageEffect.php index 338be49eb89214e5ae2763b86b05ef47a22cbd24..0b9bece5a9e677f2dc297522403323437746beb0 100644 --- a/core/modules/image/src/Plugin/ImageEffect/ScaleImageEffect.php +++ b/core/modules/image/src/Plugin/ImageEffect/ScaleImageEffect.php @@ -86,7 +86,7 @@ public function buildConfigurationForm(array $form, FormStateInterface $form_sta public function validateConfigurationForm(array &$form, FormStateInterface $form_state) { parent::validateConfigurationForm($form, $form_state); if ($form_state->isValueEmpty('width') && $form_state->isValueEmpty('height')) { - form_set_error('data', $form_state, $this->t('Width and height can not both be blank.')); + $form_state->setErrorByName('data', $this->t('Width and height can not both be blank.')); } } diff --git a/core/modules/menu_ui/menu_ui.module b/core/modules/menu_ui/menu_ui.module index 76aee080a624db55841cfd13140b20161d04ca0f..e9a381ec9bdf06e99aba6a30e82f663a85a018b0 100644 --- a/core/modules/menu_ui/menu_ui.module +++ b/core/modules/menu_ui/menu_ui.module @@ -501,7 +501,7 @@ function menu_ui_form_node_type_form_validate(&$form, FormStateInterface $form_s if (count($available_menus)) { $menu_item_id_parts = explode(':', $form_state->getValue('menu_parent')); if (!in_array($menu_item_id_parts[0], $available_menus)) { - form_set_error('menu_parent', $form_state, t('The selected menu item is not under one of the selected menus.')); + $form_state->setErrorByName('menu_parent', t('The selected menu item is not under one of the selected menus.')); } } else { diff --git a/core/modules/node/node.api.php b/core/modules/node/node.api.php index 7238abe7263879989e035a8edce776455fab7242..6fb311cfa2c37d2858cce0eda4ee1010fe4709f9 100644 --- a/core/modules/node/node.api.php +++ b/core/modules/node/node.api.php @@ -405,7 +405,7 @@ function hook_node_update_index(\Drupal\node\NodeInterface $node, $langcode) { * finished editing the node and is previewing or submitting it. It is invoked * at the end of all the standard validation steps. * - * To indicate a validation error, use form_set_error(). + * To indicate a validation error, use $form_state->setErrorByName(). * * Note: Changes made to the $node object within your hook implementation will * have no effect. The preferred method to change a node's content is to use @@ -424,7 +424,7 @@ function hook_node_update_index(\Drupal\node\NodeInterface $node, $langcode) { function hook_node_validate(\Drupal\node\NodeInterface $node, $form, \Drupal\Core\Form\FormStateInterface $form_state) { if (isset($node->end) && isset($node->start)) { if ($node->start > $node->end) { - form_set_error('time', $form_state, t('An event may not end before it starts.')); + $form_state->setErrorByName('time', t('An event may not end before it starts.')); } } } diff --git a/core/modules/node/src/Plugin/Action/AssignOwnerNode.php b/core/modules/node/src/Plugin/Action/AssignOwnerNode.php index 8618fd68630b8a74ac879d8c5480f4183d9bbb17..17e62d1bd618881da9f8b7c350a8ac6ff2634316 100644 --- a/core/modules/node/src/Plugin/Action/AssignOwnerNode.php +++ b/core/modules/node/src/Plugin/Action/AssignOwnerNode.php @@ -121,7 +121,7 @@ public function buildConfigurationForm(array $form, FormStateInterface $form_sta public function validateConfigurationForm(array &$form, FormStateInterface $form_state) { $exists = (bool) $this->connection->queryRange('SELECT 1 FROM {users_field_data} WHERE name = :name AND default_langcode = 1', 0, 1, array(':name' => $form_state->getValue('owner_name')))->fetchField(); if (!$exists) { - form_set_error('owner_name', $form_state, t('Enter a valid username.')); + $form_state->setErrorByName('owner_name', t('Enter a valid username.')); } } diff --git a/core/modules/rest/src/Plugin/views/row/DataFieldRow.php b/core/modules/rest/src/Plugin/views/row/DataFieldRow.php index 821ed48424e258d1db72537c220916210c2b8db8..6f6a6296684e2b28e6b3a45d627727aaa3ffc1b0 100644 --- a/core/modules/rest/src/Plugin/views/row/DataFieldRow.php +++ b/core/modules/rest/src/Plugin/views/row/DataFieldRow.php @@ -128,7 +128,7 @@ public function validateOptionsForm(&$form, FormStateInterface $form_state) { // If array filter returns empty, no values have been entered. Unique keys // should only be validated if we have some. if (($filtered = array_filter($aliases)) && (array_unique($filtered) !== $filtered)) { - form_set_error('aliases', $form_state, t('All field aliases must be unique')); + $form_state->setErrorByName('aliases', t('All field aliases must be unique')); } } diff --git a/core/modules/search/src/Plugin/views/filter/Search.php b/core/modules/search/src/Plugin/views/filter/Search.php index 20d60b65dc1da32a13f17e7820197504dbe8eb4e..763512565215d444cec0412c1c4322dade25dd27 100644 --- a/core/modules/search/src/Plugin/views/filter/Search.php +++ b/core/modules/search/src/Plugin/views/filter/Search.php @@ -108,7 +108,7 @@ public function validateExposed(&$form, FormStateInterface $form_state) { if (!$form_state->isValueEmpty($key)) { $this->queryParseSearchExpression($form_state->getValue($key)); if (count($this->searchQuery->words()) == 0) { - form_set_error($key, $form_state, format_plural(\Drupal::config('search.settings')->get('index.minimum_word_size'), 'You must include at least one positive keyword with 1 character or more.', 'You must include at least one positive keyword with @count characters or more.')); + $form_state->setErrorByName($key, format_plural(\Drupal::config('search.settings')->get('index.minimum_word_size'), 'You must include at least one positive keyword with 1 character or more.', 'You must include at least one positive keyword with @count characters or more.')); } } } diff --git a/core/modules/system/src/Plugin/views/field/BulkForm.php b/core/modules/system/src/Plugin/views/field/BulkForm.php index 130dc33485e46677a8a6e3fc3bd650ba37ca8433..bae235d57e8383051db9221089b9bb73bfaf79a7 100644 --- a/core/modules/system/src/Plugin/views/field/BulkForm.php +++ b/core/modules/system/src/Plugin/views/field/BulkForm.php @@ -294,7 +294,7 @@ protected function emptySelectedMessage() { public function viewsFormValidate(&$form, FormStateInterface $form_state) { $selected = array_filter($form_state->getValue($this->options['id'])); if (empty($selected)) { - form_set_error('', $form_state, $this->emptySelectedMessage()); + $form_state->setErrorByName('', $this->emptySelectedMessage()); } } diff --git a/core/modules/system/system.module b/core/modules/system/system.module index aceb805d9dcbea934ec0b1edadb7be60b55d6485..c956e4df3e9d6ecdd19b71ac97e50b09a52070b9 100644 --- a/core/modules/system/system.module +++ b/core/modules/system/system.module @@ -8,6 +8,7 @@ use Drupal\Core\Cache\Cache; use Drupal\Core\Extension\Extension; use Drupal\Core\Extension\ExtensionDiscovery; +use Drupal\Core\Form\FormState; use Drupal\Core\Form\FormStateInterface; use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\StringTranslation\TranslationWrapper; @@ -1090,8 +1091,10 @@ function system_preprocess_block(&$variables) { * * @param $form_element * The form element containing the name of the directory to check. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. */ -function system_check_directory($form_element) { +function system_check_directory($form_element, FormStateInterface $form_state) { $directory = $form_element['#value']; if (strlen($directory) == 0) { return $form_element; @@ -1100,13 +1103,13 @@ function system_check_directory($form_element) { $logger = \Drupal::logger('file system'); if (!is_dir($directory) && !drupal_mkdir($directory, NULL, TRUE)) { // If the directory does not exists and cannot be created. - form_set_error($form_element['#parents'][0], $form_state, t('The directory %directory does not exist and could not be created.', array('%directory' => $directory))); + $form_state->setErrorByName($form_element['#parents'][0], t('The directory %directory does not exist and could not be created.', array('%directory' => $directory))); $logger->error('The directory %directory does not exist and could not be created.', array('%directory' => $directory)); } if (is_dir($directory) && !is_writable($directory) && !drupal_chmod($directory)) { // If the directory is not writable and cannot be made so. - form_set_error($form_element['#parents'][0], $form_state, t('The directory %directory exists but is not writable and could not be made writable.', array('%directory' => $directory))); + $form_state->setErrorByName($form_element['#parents'][0], t('The directory %directory exists but is not writable and could not be made writable.', array('%directory' => $directory))); $logger->error('The directory %directory exists but is not writable and could not be made writable.', array('%directory' => $directory)); } elseif (is_dir($directory)) { diff --git a/core/modules/system/tests/modules/form_test/src/Callbacks.php b/core/modules/system/tests/modules/form_test/src/Callbacks.php index bababe6259575cf301bdd251738250fca1663d19..ed26dd953748811135cd449feeea460e8dc6634b 100644 --- a/core/modules/system/tests/modules/form_test/src/Callbacks.php +++ b/core/modules/system/tests/modules/form_test/src/Callbacks.php @@ -46,7 +46,7 @@ public function validateName(&$element, FormStateInterface $form_state) { drupal_set_message(t('@label value: @value', array('@label' => $element['#title'], '@value' => $form_state->getValue('name')))); // Trigger a form validation error to see our changes. - form_set_error('', $form_state); + $form_state->setErrorByName(''); } } diff --git a/core/modules/system/tests/modules/form_test/src/Form/FormTestProgrammaticForm.php b/core/modules/system/tests/modules/form_test/src/Form/FormTestProgrammaticForm.php index d782ef5121e21739ebc356717946fe5f9a49ae9a..f7d615022b86ab2387374a34f21d7463a448f4ed 100644 --- a/core/modules/system/tests/modules/form_test/src/Form/FormTestProgrammaticForm.php +++ b/core/modules/system/tests/modules/form_test/src/Form/FormTestProgrammaticForm.php @@ -90,7 +90,7 @@ public function buildForm(array $form, FormStateInterface $form_state) { */ public function validateForm(array &$form, FormStateInterface $form_state) { if ($form_state->isValueEmpty('textfield')) { - form_set_error('textfield', $form_state, t('Textfield is required.')); + $form_state->setErrorByName('textfield', t('Textfield is required.')); } } diff --git a/core/modules/system/tests/modules/form_test/src/Form/FormTestValidateForm.php b/core/modules/system/tests/modules/form_test/src/Form/FormTestValidateForm.php index 27f31edebe35f27950a57f60db20439945323493..d9293b45c4b52ddb0b83e99885de5de9ed643866 100644 --- a/core/modules/system/tests/modules/form_test/src/Form/FormTestValidateForm.php +++ b/core/modules/system/tests/modules/form_test/src/Form/FormTestValidateForm.php @@ -69,7 +69,7 @@ public function validateForm(array &$form, FormStateInterface $form_state) { drupal_set_message(t('@label value: @value', array('@label' => $form['name']['#title'], '@value' => $form_state->getValue('name')))); // Trigger a form validation error to see our changes. - form_set_error('', $form_state); + $form_state->setErrorByName(''); } } diff --git a/core/modules/user/user.module b/core/modules/user/user.module index 8516792c488ea1b9c6458605c582976bf42d3467..f2b1984a962a38822d5a98a9e686fcd4748217e7 100644 --- a/core/modules/user/user.module +++ b/core/modules/user/user.module @@ -507,8 +507,8 @@ function user_validate_current_pass(&$form, FormStateInterface $form_state) { if ((strlen(trim($form_state->getValue($key))) > 0) && ($form_state->getValue($key) != $current_value)) { $current_pass_failed = $form_state->isValueEmpty('current_pass') || !\Drupal::service('password')->check($form_state->getValue('current_pass'), $account); if ($current_pass_failed) { - form_set_error('current_pass', $form_state, t("Your current password is missing or incorrect; it's required to change the %name.", array('%name' => $name))); - form_set_error($key, $form_state); + $form_state->setErrorByName('current_pass', t("Your current password is missing or incorrect; it's required to change the %name.", array('%name' => $name))); + $form_state->setErrorByName($key); } // We only need to check the password once. break; diff --git a/core/modules/views/src/Plugin/views/filter/BooleanOperator.php b/core/modules/views/src/Plugin/views/filter/BooleanOperator.php index c635eb6f795d82c3143be9941ec71fccf26f0231..3db2233c2da65a2f851bdf55abe3ba10b9695f45 100644 --- a/core/modules/views/src/Plugin/views/filter/BooleanOperator.php +++ b/core/modules/views/src/Plugin/views/filter/BooleanOperator.php @@ -165,7 +165,7 @@ protected function valueForm(&$form, FormStateInterface $form_state) { protected function valueValidate($form, FormStateInterface $form_state) { if ($form_state->getValue(array('options', 'value')) == 'All' && !$form_state->isValueEmpty(array('options', 'expose', 'required'))) { - form_set_error('value', $form_state, t('You must select a value unless this is an non-required exposed filter.')); + $form_state->setErrorByName('value', t('You must select a value unless this is an non-required exposed filter.')); } } diff --git a/core/modules/views/src/Plugin/views/filter/Combine.php b/core/modules/views/src/Plugin/views/filter/Combine.php index 12e6f22e940cc6372b2813b8ecaaddb17f7318f5..7751027ec4da7696061c2cc994bb5c76a592784c 100644 --- a/core/modules/views/src/Plugin/views/filter/Combine.php +++ b/core/modules/views/src/Plugin/views/filter/Combine.php @@ -51,7 +51,7 @@ public function buildOptionsForm(&$form, FormStateInterface $form_state) { ); } else { - form_set_error('', $form_state, t('You have to add some fields to be able to use this filter.')); + $form_state->setErrorByName('', t('You have to add some fields to be able to use this filter.')); } } } diff --git a/core/modules/views/src/Plugin/views/pager/SqlBase.php b/core/modules/views/src/Plugin/views/pager/SqlBase.php index 1fe68bcec9470ed16f722d5b9de967c671402a6a..8a4a8e54d6f00df4e3f6a0b4086133d3f8a2a0d0 100644 --- a/core/modules/views/src/Plugin/views/pager/SqlBase.php +++ b/core/modules/views/src/Plugin/views/pager/SqlBase.php @@ -197,14 +197,14 @@ public function validateOptionsForm(&$form, FormStateInterface $form_state) { $error = TRUE; } if ($error) { - form_set_error('pager_options][expose][items_per_page_options', $form_state, t('Insert a list of integer numeric values separated by commas: e.g: 10, 20, 50, 100')); + $form_state->setErrorByName('pager_options][expose][items_per_page_options', t('Insert a list of integer numeric values separated by commas: e.g: 10, 20, 50, 100')); } // Make sure that the items_per_page is part of the expose settings. if (!$form_state->isValueEmpty(array('pager_options', 'expose', 'items_per_page')) && !$form_state->isValueEmpty(array('pager_options', 'items_per_page'))) { $items_per_page = $form_state->getValue(array('pager_options', 'items_per_page')); if (array_search($items_per_page, $options) === FALSE) { - form_set_error('pager_options][expose][items_per_page_options', $form_state, t("The Exposed items per page field's options must include the value from the Items per page field (@items_per_page).", + $form_state->setErrorByName('pager_options][expose][items_per_page_options', t("The Exposed items per page field's options must include the value from the Items per page field (@items_per_page).", array('@items_per_page' => $items_per_page)) ); } @@ -371,7 +371,7 @@ public function exposedFormAlter(&$form, FormStateInterface $form_state) { public function exposedFormValidate(&$form, FormStateInterface $form_state) { if (!$form_state->isValueEmpty('offset') && trim($form_state->getValue('offset'))) { if (!is_numeric($form_state->getValue('offset')) || $form_state->getValue('offset') < 0) { - form_set_error('offset', $form_state, t('Offset must be an number greather or equal than 0.')); + $form_state->setErrorByName('offset', t('Offset must be an number greather or equal than 0.')); } } }