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.'));
}
}
}