summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Pott2015-06-12 13:54:11 (GMT)
committerAlex Pott2015-06-12 13:54:11 (GMT)
commit215b967a441320b04eec085b6f10d7c3552cde2d (patch)
tree1e31e57d2bdc379348a7f7ce8b04445ec4a1556a
parent3dd383704dc55fa271e99ed0ea2b7a436edf72a1 (diff)
Issue #1493324 by tim.plunkett, dmsmidt, mgifford, bleen18, davidhernandez, crasx, mparker17, stefan.r, YesCT, joelpittet, tstoeckler, larowlan, vijaycs85, swentel, rpayanm, Bojhan, LewisNyman, emma.maria, BarisW, njbarrett, rteijeiro, nod_, sun, joshtaylor, mrjmd, webchick, marcvangend, kattekrab, SKAUGHT, bowersox, andrewmacpherson, Manjit.Singh, RavindraSingh, Wim Leers, BLadwin, aspilicious, mortendk, mausolos, jessebeach, Gábor Hojtsy, anandps, falcon03, franz, andypost, rooby, rootwork, Cottser, Xano: Inline form errors for accessibility and UX
-rw-r--r--core/core.services.yml5
-rw-r--r--core/includes/form.inc15
-rw-r--r--core/includes/theme.inc6
-rw-r--r--core/lib/Drupal/Core/Datetime/Element/Datelist.php1
-rw-r--r--core/lib/Drupal/Core/Datetime/Element/Datetime.php2
-rw-r--r--core/lib/Drupal/Core/Form/FormElementHelper.php68
-rw-r--r--core/lib/Drupal/Core/Form/FormErrorHandler.php137
-rw-r--r--core/lib/Drupal/Core/Form/FormErrorHandlerInterface.php27
-rw-r--r--core/lib/Drupal/Core/Form/FormState.php14
-rw-r--r--core/lib/Drupal/Core/Form/FormStateInterface.php3
-rw-r--r--core/lib/Drupal/Core/Form/FormValidator.php39
-rw-r--r--core/lib/Drupal/Core/Render/Element.php15
-rw-r--r--core/lib/Drupal/Core/Render/Element/Checkboxes.php2
-rw-r--r--core/lib/Drupal/Core/Render/Element/PasswordConfirm.php3
-rw-r--r--core/lib/Drupal/Core/Render/Element/Radios.php2
-rw-r--r--core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php2
-rw-r--r--core/modules/file/file.module4
-rw-r--r--core/modules/file/src/Element/ManagedFile.php3
-rw-r--r--core/modules/file/src/Tests/FileFieldValidateTest.php6
-rw-r--r--core/modules/shortcut/src/Tests/ShortcutSetsTest.php6
-rw-r--r--core/modules/system/css/system.theme.css11
-rw-r--r--core/modules/system/src/Tests/Form/FormTest.php2
-rw-r--r--core/modules/system/src/Tests/Form/TriggeringElementProgrammedUnitTest.php12
-rw-r--r--core/modules/system/src/Tests/Form/ValidationTest.php67
-rw-r--r--core/modules/system/templates/datetime-wrapper.html.twig5
-rw-r--r--core/modules/system/templates/fieldset.html.twig6
-rw-r--r--core/modules/system/templates/form-element.html.twig7
-rw-r--r--core/modules/user/src/Tests/UserBlocksTest.php11
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php8
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormElementHelperTest.php179
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormErrorHandlerTest.php125
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormStateTest.php23
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormTestBase.php5
-rw-r--r--core/tests/Drupal/Tests/Core/Form/FormValidatorTest.php105
-rw-r--r--core/tests/Drupal/Tests/UnitTestCase.php6
-rw-r--r--core/themes/bartik/css/components/form.css10
-rw-r--r--core/themes/classy/templates/form/datetime-wrapper.html.twig5
-rw-r--r--core/themes/classy/templates/form/fieldset.html.twig6
-rw-r--r--core/themes/classy/templates/form/form-element.html.twig7
-rw-r--r--core/themes/seven/css/components/form.css31
40 files changed, 844 insertions, 147 deletions
diff --git a/core/core.services.yml b/core/core.services.yml
index 589c5f6..9f83cec 100644
--- a/core/core.services.yml
+++ b/core/core.services.yml
@@ -285,10 +285,13 @@ services:
arguments: ['@form_validator', '@form_submitter', '@form_cache', '@module_handler', '@event_dispatcher', '@request_stack', '@class_resolver', '@element_info', '@theme.manager', '@?csrf_token']
form_validator:
class: Drupal\Core\Form\FormValidator
- arguments: ['@request_stack', '@string_translation', '@csrf_token', '@logger.channel.form']
+ arguments: ['@request_stack', '@string_translation', '@csrf_token', '@logger.channel.form', '@form_error_handler']
form_submitter:
class: Drupal\Core\Form\FormSubmitter
arguments: ['@request_stack', '@url_generator']
+ form_error_handler:
+ class: Drupal\Core\Form\FormErrorHandler
+ arguments: ['@string_translation', '@link_generator']
form_cache:
class: Drupal\Core\Form\FormCache
arguments: ['@app.root', '@keyvalue.expirable', '@module_handler', '@current_user', '@csrf_token', '@logger.channel.form', '@request_stack', '@page_cache_request_policy']
diff --git a/core/includes/form.inc b/core/includes/form.inc
index 01d9068..8170820 100644
--- a/core/includes/form.inc
+++ b/core/includes/form.inc
@@ -11,6 +11,7 @@ use Drupal\Component\Utility\UrlHelper;
use Drupal\Component\Utility\Xss;
use Drupal\Core\Database\Database;
use Drupal\Core\Form\FormStateInterface;
+use Drupal\Core\Form\FormElementHelper;
use Drupal\Core\Form\OptGroup;
use Drupal\Core\Render\Element;
use Drupal\Core\Template\Attribute;
@@ -203,6 +204,12 @@ function template_preprocess_fieldset(&$variables) {
// Add the description's id to the fieldset aria attributes.
$variables['attributes']['aria-describedby'] = $description_id;
}
+
+ // Display any error messages.
+ $variables['errors'] = NULL;
+ if (!empty($element['#errors']) && empty($element['#error_no_message'])) {
+ $variables['errors'] = $element['#errors'];
+ }
}
/**
@@ -407,7 +414,7 @@ function template_preprocess_form_element(&$variables) {
);
$variables['attributes'] = $element['#wrapper_attributes'];
- // Add element #id for #type 'item'.
+ // Add element #id for #type 'item' and 'password_confirm'.
if (isset($element['#markup']) && !empty($element['#id'])) {
$variables['attributes']['id'] = $element['#id'];
}
@@ -423,6 +430,12 @@ function template_preprocess_form_element(&$variables) {
// Pass elements disabled status to template.
$variables['disabled'] = !empty($element['#attributes']['disabled']) ? $element['#attributes']['disabled'] : NULL;
+ // Display any error messages.
+ $variables['errors'] = NULL;
+ if (!empty($element['#errors']) && empty($element['#error_no_message'])) {
+ $variables['errors'] = $element['#errors'];
+ }
+
// If #title is not set, we don't display any label.
if (!isset($element['#title'])) {
$element['#title_display'] = 'none';
diff --git a/core/includes/theme.inc b/core/includes/theme.inc
index 121da17..9ea8f05 100644
--- a/core/includes/theme.inc
+++ b/core/includes/theme.inc
@@ -500,6 +500,12 @@ function template_preprocess_datetime_wrapper(&$variables) {
$variables['title'] = $element['#title'];
}
+ // Display any error messages.
+ $variables['errors'] = NULL;
+ if (!empty($element['#errors']) && empty($element['#error_no_message'])) {
+ $variables['errors'] = $element['#errors'];
+ }
+
if (!empty($element['#description'])) {
$variables['description'] = $element['#description'];
}
diff --git a/core/lib/Drupal/Core/Datetime/Element/Datelist.php b/core/lib/Drupal/Core/Datetime/Element/Datelist.php
index c330a4f..a0060ed 100644
--- a/core/lib/Drupal/Core/Datetime/Element/Datelist.php
+++ b/core/lib/Drupal/Core/Datetime/Element/Datelist.php
@@ -265,6 +265,7 @@ class Datelist extends DateElementBase {
'#attributes' => $element['#attributes'],
'#options' => $options,
'#required' => $element['#required'],
+ '#error_no_message' => TRUE,
);
}
diff --git a/core/lib/Drupal/Core/Datetime/Element/Datetime.php b/core/lib/Drupal/Core/Datetime/Element/Datetime.php
index d298e21..b8e4b69 100644
--- a/core/lib/Drupal/Core/Datetime/Element/Datetime.php
+++ b/core/lib/Drupal/Core/Datetime/Element/Datetime.php
@@ -267,6 +267,7 @@ class Datetime extends DateElementBase {
'#attributes' => $element['#attributes'] + $extra_attributes,
'#required' => $element['#required'],
'#size' => max(12, strlen($element['#value']['date'])),
+ '#error_no_message' => TRUE,
);
// Allows custom callbacks to alter the element.
@@ -298,6 +299,7 @@ class Datetime extends DateElementBase {
'#attributes' => $element['#attributes'] + $extra_attributes,
'#required' => $element['#required'],
'#size' => 12,
+ '#error_no_message' => TRUE,
);
// Allows custom callbacks to alter the element.
diff --git a/core/lib/Drupal/Core/Form/FormElementHelper.php b/core/lib/Drupal/Core/Form/FormElementHelper.php
new file mode 100644
index 0000000..013eaa1
--- /dev/null
+++ b/core/lib/Drupal/Core/Form/FormElementHelper.php
@@ -0,0 +1,68 @@
+<?php
+
+/**
+ * @file
+ * Contains \Drupal\Core\Form\FormElementHelper.
+ */
+
+namespace Drupal\Core\Form;
+
+use Drupal\Core\Render\Element;
+
+/**
+ * Provides common functionality for form elements.
+ */
+class FormElementHelper {
+
+ /**
+ * Retrieves a form element.
+ *
+ * @param string $name
+ * The name of the form element. If the #parents property of your form
+ * element is ['foo', 'bar', 'baz'] then the name is 'foo][bar][baz'.
+ * @param array $form
+ * An associative array containing the structure of the form.
+ *
+ * @return array
+ * The form element.
+ */
+ public static function getElementByName($name, array $form) {
+ foreach (Element::children($form) as $key) {
+ if (implode('][', $form[$key]['#parents']) === $name) {
+ return $form[$key];
+ }
+ elseif ($element = static::getElementByName($name, $form[$key])) {
+ return $element;
+ }
+ }
+ return [];
+ }
+
+ /**
+ * Returns the title for the element.
+ *
+ * If the element has no title, this will recurse through all children of the
+ * element until a title is found.
+ *
+ * @param array $element
+ * An associative array containing the properties of the form element.
+ *
+ * @return string
+ * The title of the element, or an empty string if none is found.
+ */
+ public static function getElementTitle(array $element) {
+ $title = '';
+ if (isset($element['#title'])) {
+ $title = $element['#title'];
+ }
+ else {
+ foreach (Element::children($element) as $key) {
+ if ($title = static::getElementTitle($element[$key])) {
+ break;
+ }
+ }
+ }
+ return $title;
+ }
+
+}
diff --git a/core/lib/Drupal/Core/Form/FormErrorHandler.php b/core/lib/Drupal/Core/Form/FormErrorHandler.php
new file mode 100644
index 0000000..df736b0
--- /dev/null
+++ b/core/lib/Drupal/Core/Form/FormErrorHandler.php
@@ -0,0 +1,137 @@
+<?php
+
+/**
+ * @file
+ * Contains \Drupal\Core\Form\FormErrorHandler.
+ */
+
+namespace Drupal\Core\Form;
+
+use Drupal\Component\Utility\SafeMarkup;
+use Drupal\Core\Render\Element;
+use Drupal\Core\Routing\LinkGeneratorTrait;
+use Drupal\Core\StringTranslation\StringTranslationTrait;
+use Drupal\Core\StringTranslation\TranslationInterface;
+use Drupal\Core\Url;
+use Drupal\Core\Utility\LinkGeneratorInterface;
+
+/**
+ * Handles form errors.
+ */
+class FormErrorHandler implements FormErrorHandlerInterface {
+
+ use StringTranslationTrait;
+ use LinkGeneratorTrait;
+
+ /**
+ * Constructs a new FormErrorHandler.
+ *
+ * @param \Drupal\Core\StringTranslation\TranslationInterface $string_translation
+ * The string translation service.
+ * @param \Drupal\Core\Utility\LinkGeneratorInterface $link_generator
+ * The link generation service.
+ */
+ public function __construct(TranslationInterface $string_translation, LinkGeneratorInterface $link_generator) {
+ $this->stringTranslation = $string_translation;
+ $this->linkGenerator = $link_generator;
+ }
+
+ /**
+ * {@inheritdoc}
+ */
+ public function handleFormErrors(array &$form, FormStateInterface $form_state) {
+ // After validation check if there are errors.
+ if ($errors = $form_state->getErrors()) {
+ // Display error messages for each element.
+ $this->displayErrorMessages($form, $form_state);
+
+ // Loop through and assign each element its errors.
+ $this->setElementErrorsFromFormState($form, $form_state);
+ }
+
+ return $this;
+ }
+
+ /**
+ * Loops through and displays all form errors.
+ *
+ * @param array $form
+ * An associative array containing the structure of the form.
+ * @param \Drupal\Core\Form\FormStateInterface $form_state
+ * The current state of the form.
+ */
+ protected function displayErrorMessages(array $form, FormStateInterface $form_state) {
+ $error_links = [];
+ $errors = $form_state->getErrors();
+ // Loop through all form errors and check if we need to display a link.
+ foreach ($errors as $name => $error) {
+ $form_element = FormElementHelper::getElementByName($name, $form);
+ $title = FormElementHelper::getElementTitle($form_element);
+
+ // Only show links to erroneous elements that are visible.
+ $is_visible_element = Element::isVisibleElement($form_element);
+ // Only show links for elements that have a title themselves or have
+ // children with a title.
+ $has_title = !empty($title);
+ // Only show links for elements with an ID.
+ $has_id = !empty($form_element['#id']);
+
+ // Do not show links to elements with suppressed messages. Most often
+ // their parent element is used for inline errors.
+ if (!empty($form_element['#error_no_message'])) {
+ unset($errors[$name]);
+ }
+ elseif ($is_visible_element && $has_title && $has_id) {
+ // We need to pass this through SafeMarkup::escape() so
+ // drupal_set_message() does not encode the links.
+ $error_links[] = SafeMarkup::escape($this->l($title, Url::fromRoute('<none>', [], ['fragment' => $form_element['#id'], 'external' => TRUE])));
+ unset($errors[$name]);
+ }
+ }
+
+ // Set normal error messages for all remaining errors.
+ foreach ($errors as $error) {
+ $this->drupalSetMessage($error, 'error');
+ }
+
+ if (!empty($error_links)) {
+ $message = $this->formatPlural(count($error_links), '1 error has been found: !errors', '@count errors have been found: !errors', [
+ '!errors' => SafeMarkup::set(implode(', ', $error_links)),
+ ]);
+ $this->drupalSetMessage($message, 'error');
+ }
+ }
+
+ /**
+ * Stores the errors of each element directly on the element.
+ *
+ * We must provide a way for non-form functions to check the errors for a
+ * specific element. The most common usage of this is a #pre_render callback.
+ *
+ * @param array $elements
+ * An associative array containing the structure of a form element.
+ * @param \Drupal\Core\Form\FormStateInterface $form_state
+ * The current state of the form.
+ */
+ protected function setElementErrorsFromFormState(array &$elements, FormStateInterface &$form_state) {
+ // Recurse through all children.
+ foreach (Element::children($elements) as $key) {
+ if (isset($elements[$key]) && $elements[$key]) {
+ $this->setElementErrorsFromFormState($elements[$key], $form_state);
+ }
+ }
+
+ // Store the errors for this element on the element directly.
+ $elements['#errors'] = $form_state->getError($elements);
+ }
+
+ /**
+ * Wraps drupal_set_message().
+ *
+ * @codeCoverageIgnore
+ */
+ protected function drupalSetMessage($message = NULL, $type = 'status', $repeat = FALSE) {
+ drupal_set_message($message, $type, $repeat);
+ }
+
+}
diff --git a/core/lib/Drupal/Core/Form/FormErrorHandlerInterface.php b/core/lib/Drupal/Core/Form/FormErrorHandlerInterface.php
new file mode 100644
index 0000000..161505f
--- /dev/null
+++ b/core/lib/Drupal/Core/Form/FormErrorHandlerInterface.php
@@ -0,0 +1,27 @@
+<?php
+
+/**
+ * @file
+ * Contains \Drupal\Core\Form\FormErrorHandlerInterface.
+ */
+
+namespace Drupal\Core\Form;
+
+/**
+ * Provides an interface for handling form errors.
+ */
+interface FormErrorHandlerInterface {
+
+ /**
+ * Handles form errors after form validation.
+ *
+ * @param array $form
+ * An associative array containing the structure of the form.
+ * @param \Drupal\Core\Form\FormStateInterface $form_state
+ * The current state of the form.
+ *
+ * @return $this
+ */
+ public function handleFormErrors(array &$form, FormStateInterface $form_state);
+
+}
diff --git a/core/lib/Drupal/Core/Form/FormState.php b/core/lib/Drupal/Core/Form/FormState.php
index 9d651cd..4a17d6b 100644
--- a/core/lib/Drupal/Core/Form/FormState.php
+++ b/core/lib/Drupal/Core/Form/FormState.php
@@ -1090,9 +1090,6 @@ class FormState implements FormStateInterface {
$errors[$name] = $message;
$this->errors = $errors;
static::setAnyErrors();
- if ($message) {
- $this->drupalSetMessage($message, 'error');
- }
}
}
@@ -1119,7 +1116,7 @@ class FormState implements FormStateInterface {
* {@inheritdoc}
*/
public function getError(array $element) {
- if ($errors = $this->getErrors($this)) {
+ if ($errors = $this->getErrors()) {
$parents = array();
foreach ($element['#parents'] as $parent) {
$parents[] = $parent;
@@ -1245,15 +1242,6 @@ class FormState implements FormStateInterface {
}
/**
- * Wraps drupal_set_message().
- *
- * @return array|null
- */
- protected function drupalSetMessage($message = NULL, $type = 'status', $repeat = FALSE) {
- return drupal_set_message($message, $type, $repeat);
- }
-
- /**
* Wraps ModuleHandler::loadInclude().
*/
protected function moduleLoadInclude($module, $type, $name = NULL) {
diff --git a/core/lib/Drupal/Core/Form/FormStateInterface.php b/core/lib/Drupal/Core/Form/FormStateInterface.php
index 5e44ecb..e0ca1c6 100644
--- a/core/lib/Drupal/Core/Form/FormStateInterface.php
+++ b/core/lib/Drupal/Core/Form/FormStateInterface.php
@@ -417,7 +417,8 @@ interface FormStateInterface {
* indicate which element needs to be changed and provide an error message.
* This causes the Form API to not execute the form submit handlers, and
* instead to re-display the form to the user with the corresponding elements
- * rendered with an 'error' CSS class (shown as red by default).
+ * rendered with an 'error' CSS class (shown as red by default) and the error
+ * message near the element.
*
* The standard behavior of this method can be changed if a button provides
* the #limit_validation_errors property. Multistep forms not wanting to
diff --git a/core/lib/Drupal/Core/Form/FormValidator.php b/core/lib/Drupal/Core/Form/FormValidator.php
index ee2da65..96f2320 100644
--- a/core/lib/Drupal/Core/Form/FormValidator.php
+++ b/core/lib/Drupal/Core/Form/FormValidator.php
@@ -45,6 +45,13 @@ class FormValidator implements FormValidatorInterface {
protected $logger;
/**
+ * The form error handler.
+ *
+ * @var \Drupal\Core\Form\FormErrorHandlerInterface
+ */
+ protected $formErrorHandler;
+
+ /**
* Constructs a new FormValidator.
*
* @param \Symfony\Component\HttpFoundation\RequestStack $request_stack
@@ -55,12 +62,15 @@ class FormValidator implements FormValidatorInterface {
* The CSRF token generator.
* @param \Psr\Log\LoggerInterface $logger
* A logger instance.
+ * @param \Drupal\Core\Form\FormErrorHandlerInterface $form_error_handler
+ * The form error handler.
*/
- public function __construct(RequestStack $request_stack, TranslationInterface $string_translation, CsrfTokenGenerator $csrf_token, LoggerInterface $logger) {
+ public function __construct(RequestStack $request_stack, TranslationInterface $string_translation, CsrfTokenGenerator $csrf_token, LoggerInterface $logger, FormErrorHandlerInterface $form_error_handler) {
$this->requestStack = $request_stack;
$this->stringTranslation = $string_translation;
$this->csrfToken = $csrf_token;
$this->logger = $logger;
+ $this->formErrorHandler = $form_error_handler;
}
/**
@@ -184,8 +194,9 @@ class FormValidator implements FormValidatorInterface {
* The unique string identifying the form.
*/
protected function finalizeValidation(&$form, FormStateInterface &$form_state, $form_id) {
- // After validation, loop through and assign each element its errors.
- $this->setElementErrorsFromFormState($form, $form_state);
+ // Delegate handling of form errors to a service.
+ $this->formErrorHandler->handleFormErrors($form, $form_state);
+
// Mark this form as validated.
$form_state->setValidationComplete();
}
@@ -394,26 +405,4 @@ class FormValidator implements FormValidatorInterface {
}
}
- /**
- * Stores the errors of each element directly on the element.
- *
- * We must provide a way for non-form functions to check the errors for a
- * specific element. The most common usage of this is a #pre_render callback.
- *
- * @param array $elements
- * An associative array containing the structure of a form element.
- * @param \Drupal\Core\Form\FormStateInterface $form_state
- * The current state of the form.
- */
- protected function setElementErrorsFromFormState(array &$elements, FormStateInterface &$form_state) {
- // Recurse through all children.
- foreach (Element::children($elements) as $key) {
- if (isset($elements[$key]) && $elements[$key]) {
- $this->setElementErrorsFromFormState($elements[$key], $form_state);
- }
- }
- // Store the errors for this element on the element directly.
- $elements['#errors'] = $form_state->getError($elements);
- }
-
}
diff --git a/core/lib/Drupal/Core/Render/Element.php b/core/lib/Drupal/Core/Render/Element.php
index c0d3a4a..67186f7 100644
--- a/core/lib/Drupal/Core/Render/Element.php
+++ b/core/lib/Drupal/Core/Render/Element.php
@@ -142,7 +142,7 @@ class Element {
}
// Skip value and hidden elements, since they are not rendered.
- if (isset($child['#type']) && in_array($child['#type'], array('value', 'hidden'))) {
+ if (!static::isVisibleElement($child)) {
continue;
}
@@ -153,6 +153,19 @@ class Element {
}
/**
+ * Determines if an element is visible.
+ *
+ * @param array $element
+ * The element to check for visibility.
+ *
+ * @return bool
+ * TRUE if the element is visible, otherwise FALSE.
+ */
+ public static function isVisibleElement($element) {
+ return (!isset($element['#type']) || !in_array($element['#type'], ['value', 'hidden', 'token'])) && (!isset($element['#access']) || $element['#access']);
+ }
+
+ /**
* Sets HTML attributes based on element properties.
*
* @param array $element
diff --git a/core/lib/Drupal/Core/Render/Element/Checkboxes.php b/core/lib/Drupal/Core/Render/Element/Checkboxes.php
index d931460..117f839 100644
--- a/core/lib/Drupal/Core/Render/Element/Checkboxes.php
+++ b/core/lib/Drupal/Core/Render/Element/Checkboxes.php
@@ -86,6 +86,8 @@ class Checkboxes extends FormElement {
'#default_value' => isset($value[$key]) ? $key : NULL,
'#attributes' => $element['#attributes'],
'#ajax' => isset($element['#ajax']) ? $element['#ajax'] : NULL,
+ // Errors should only be shown on the parent checkboxes element.
+ '#error_no_message' => TRUE,
'#weight' => $weight,
);
}
diff --git a/core/lib/Drupal/Core/Render/Element/PasswordConfirm.php b/core/lib/Drupal/Core/Render/Element/PasswordConfirm.php
index f7f6240..e7195d0 100644
--- a/core/lib/Drupal/Core/Render/Element/PasswordConfirm.php
+++ b/core/lib/Drupal/Core/Render/Element/PasswordConfirm.php
@@ -26,6 +26,7 @@ class PasswordConfirm extends FormElement {
$class = get_class($this);
return array(
'#input' => TRUE,
+ '#markup' => '',
'#process' => array(
array($class, 'processPasswordConfirm'),
),
@@ -53,6 +54,7 @@ class PasswordConfirm extends FormElement {
'#value' => empty($element['#value']) ? NULL : $element['#value']['pass1'],
'#required' => $element['#required'],
'#attributes' => array('class' => array('password-field', 'js-password-field')),
+ '#error_no_message' => TRUE,
);
$element['pass2'] = array(
'#type' => 'password',
@@ -60,6 +62,7 @@ class PasswordConfirm extends FormElement {
'#value' => empty($element['#value']) ? NULL : $element['#value']['pass2'],
'#required' => $element['#required'],
'#attributes' => array('class' => array('password-confirm', 'js-password-confirm')),
+ '#error_no_message' => TRUE,
);
$element['#element_validate'] = array(array(get_called_class(), 'validatePasswordConfirm'));
$element['#tree'] = TRUE;
diff --git a/core/lib/Drupal/Core/Render/Element/Radios.php b/core/lib/Drupal/Core/Render/Element/Radios.php
index 619c0b9..5423ef0 100644
--- a/core/lib/Drupal/Core/Render/Element/Radios.php
+++ b/core/lib/Drupal/Core/Render/Element/Radios.php
@@ -83,6 +83,8 @@ class Radios extends FormElement {
'#parents' => $element['#parents'],
'#id' => HtmlUtility::getUniqueId('edit-' . implode('-', $parents_for_id)),
'#ajax' => isset($element['#ajax']) ? $element['#ajax'] : NULL,
+ // Errors should only be shown on the parent radios element.
+ '#error_no_message' => TRUE,
'#weight' => $weight,
);
}
diff --git a/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php b/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php
index b613937..e524359 100644
--- a/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php
+++ b/core/modules/content_translation/src/Tests/ContentTranslationUITestBase.php
@@ -278,7 +278,7 @@ abstract class ContentTranslationUITestBase extends ContentTranslationTestBase {
'content_translation[created]' => '19/11/1978',
);
$this->drupalPostForm($entity->urlInfo('edit-form'), $edit, $this->getFormSubmitAction($entity, $langcode));
- $this->assertTrue($this->xpath('//div[contains(@class, "error")]//ul'), 'Invalid values generate a list of form errors.');
+ $this->assertTrue($this->xpath('//div[contains(concat(" ", normalize-space(@class), " "), :class)]', array(':class' => ' messages--error ')), 'Invalid values generate a form error message.');
$metadata = $this->manager->getTranslationMetadata($entity->getTranslation($langcode));
$this->assertEqual($metadata->getAuthor()->id(), $values[$langcode]['uid'], 'Translation author correctly kept.');
$this->assertEqual($metadata->getCreatedTime(), $values[$langcode]['created'], 'Translation date correctly kept.');
diff --git a/core/modules/file/file.module b/core/modules/file/file.module
index 94ddc49..7d36c58 100644
--- a/core/modules/file/file.module
+++ b/core/modules/file/file.module
@@ -1163,7 +1163,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_state->setErrorByName($upload_name, t('The file could not be uploaded.'));
+ $form_state->setError($element, t('The file could not be uploaded.'));
return FALSE;
}
@@ -1173,7 +1173,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_state->setErrorByName($upload_name, t('Files in the !name field were unable to be uploaded.', array('!name' => $element['#title'])));
+ $form_state->setError($element, t('Files in the !name field were unable to be uploaded.', array('!name' => $element['#title'])));
return array();
}
diff --git a/core/modules/file/src/Element/ManagedFile.php b/core/modules/file/src/Element/ManagedFile.php
index 40d0c31..afde5af 100644
--- a/core/modules/file/src/Element/ManagedFile.php
+++ b/core/modules/file/src/Element/ManagedFile.php
@@ -233,6 +233,7 @@ class ManagedFile extends FormElement {
'#multiple' => $element['#multiple'],
'#theme_wrappers' => [],
'#weight' => -10,
+ '#error_no_message' => TRUE,
];
if (!empty($fids) && $element['#files']) {
@@ -328,7 +329,7 @@ class ManagedFile extends FormElement {
// Check required property based on the FID.
if ($element['#required'] && empty($element['fids']['#value']) && !in_array($clicked_button, ['upload_button', 'remove_button'])) {
- $form_state->setError($element['upload'], t('!name field is required.', ['!name' => $element['#title']]));
+ $form_state->setError($element, t('!name is required.', ['!name' => $element['#title']]));
}
// Consolidate the array value of this field to array of FIDs.
diff --git a/core/modules/file/src/Tests/FileFieldValidateTest.php b/core/modules/file/src/Tests/FileFieldValidateTest.php
index 51d7402..3e35a00 100644
--- a/core/modules/file/src/Tests/FileFieldValidateTest.php
+++ b/core/modules/file/src/Tests/FileFieldValidateTest.php
@@ -34,7 +34,8 @@ class FileFieldValidateTest extends FileFieldTestBase {
$edit = array();
$edit['title[0][value]'] = $this->randomMachineName();
$this->drupalPostForm('node/add/' . $type_name, $edit, t('Save and publish'));
- $this->assertRaw(t('!title field is required.', array('!title' => $field->getLabel())), 'Node save failed when required file field was empty.');
+ $this->assertText('1 error has been found: ' . $field->label(), 'Node save failed when required file field was empty.');
+ $this->assertIdentical(1, count($this->xpath('//div[contains(concat(" ", normalize-space(@class), " "), :class)]//a', [':class' => ' messages--error '])), 'There is one link in the error message.');
// Create a new node with the uploaded file.
$nid = $this->uploadNodeFile($test_file, $field_name, $type_name);
@@ -55,7 +56,8 @@ class FileFieldValidateTest extends FileFieldTestBase {
$edit = array();
$edit['title[0][value]'] = $this->randomMachineName();
$this->drupalPostForm('node/add/' . $type_name, $edit, t('Save and publish'));
- $this->assertRaw(t('!title field is required.', array('!title' => $field->getLabel())), 'Node save failed when required multiple value file field was empty.');
+ $this->assertText('1 error has been found: ' . $field->label(), 'Node save failed when required multiple value file field was empty.');
+ $this->assertIdentical(1, count($this->xpath('//div[contains(concat(" ", normalize-space(@class), " "), :class)]//a', [':class' => ' messages--error '])), 'There is one link in the error message.');
// Create a new node with the uploaded file into the multivalue field.
$nid = $this->uploadNodeFile($test_file, $field_name, $type_name);
diff --git a/core/modules/shortcut/src/Tests/ShortcutSetsTest.php b/core/modules/shortcut/src/Tests/ShortcutSetsTest.php
index 56fce39..db98745 100644
--- a/core/modules/shortcut/src/Tests/ShortcutSetsTest.php
+++ b/core/modules/shortcut/src/Tests/ShortcutSetsTest.php
@@ -6,6 +6,8 @@
*/
namespace Drupal\shortcut\Tests;
+
+use Drupal\Component\Utility\SafeMarkup;
use Drupal\shortcut\Entity\ShortcutSet;
/**
@@ -131,7 +133,9 @@ class ShortcutSetsTest extends ShortcutTestBase {
function testShortcutSetSwitchNoSetName() {
$edit = array('set' => 'new');
$this->drupalPostForm('user/' . $this->adminUser->id() . '/shortcuts', $edit, t('Change set'));
- $this->assertText(t('The new set label is required.'));
+ $this->assertRaw(\Drupal::translation()->formatPlural(1, '1 error has been found: !errors', '@count errors have been found: !errors', [
+ '!errors' => SafeMarkup::set('<a href="#edit-label">Label</a>')
+ ]));
$current_set = shortcut_current_displayed_set($this->adminUser);
$this->assertEqual($current_set->id(), $this->set->id(), 'Attempting to switch to a new shortcut set without providing a set name does not succeed.');
$this->assertFieldByXPath("//input[@name='label' and contains(concat(' ', normalize-space(@class), ' '), ' error ')]", NULL, 'The new set label field has the error class');
diff --git a/core/modules/system/css/system.theme.css b/core/modules/system/css/system.theme.css
index c20d6ca..fa7a75b 100644
--- a/core/modules/system/css/system.theme.css
+++ b/core/modules/system/css/system.theme.css
@@ -113,6 +113,17 @@ abbr.ajax-changed {
margin-right: 0;
}
+/* Inline error messages. */
+.form-error-message:before {
+ content: '';
+ display: inline-block;
+ height: 14px;
+ width: 14px;
+ vertical-align: sub;
+ background: url(../../../misc/icons/ea2800/error.svg) no-repeat;
+ background-size: contain;
+}
+
/**
* Inline items.
*/
diff --git a/core/modules/system/src/Tests/Form/FormTest.php b/core/modules/system/src/Tests/Form/FormTest.php
index 321502d..be1fc26 100644
--- a/core/modules/system/src/Tests/Form/FormTest.php
+++ b/core/modules/system/src/Tests/Form/FormTest.php
@@ -188,7 +188,7 @@ class FormTest extends WebTestBase {
}
// Check the page for error messages.
- $errors = $this->xpath('//div[contains(@class, "error")]//li');
+ $errors = $this->xpath('//div[contains(@class, "form-error-message")]//strong');
foreach ($errors as $error) {
$expected_key = array_search($error[0], $expected);
// If the error message is not one of the expected messages, fail.
diff --git a/core/modules/system/src/Tests/Form/TriggeringElementProgrammedUnitTest.php b/core/modules/system/src/Tests/Form/TriggeringElementProgrammedUnitTest.php
index ae6b82c..f7cffa1 100644
--- a/core/modules/system/src/Tests/Form/TriggeringElementProgrammedUnitTest.php
+++ b/core/modules/system/src/Tests/Form/TriggeringElementProgrammedUnitTest.php
@@ -19,11 +19,23 @@ use Drupal\simpletest\KernelTestBase;
*/
class TriggeringElementProgrammedUnitTest extends KernelTestBase implements FormInterface {
+ /**
+ * {@inheritdoc}
+ */
public static $modules = array('system');
/**
* {@inheritdoc}
*/
+ protected function setUp() {
+ parent::setUp();
+ $this->installSchema('system', ['router']);
+ \Drupal::service('router.builder')->rebuild();
+ }
+
+ /**
+ * {@inheritdoc}
+ */
public function getFormId() {
return 'triggering_element_programmed_form';
}
diff --git a/core/modules/system/src/Tests/Form/ValidationTest.php b/core/modules/system/src/Tests/Form/ValidationTest.php
index 7b6c9e8..eda6dae 100644
--- a/core/modules/system/src/Tests/Form/ValidationTest.php
+++ b/core/modules/system/src/Tests/Form/ValidationTest.php
@@ -7,7 +7,9 @@
namespace Drupal\system\Tests\Form;
+use Drupal\Component\Utility\SafeMarkup;
use Drupal\Core\Render\Element;
+use Drupal\Core\Url;
use Drupal\simpletest\WebTestBase;
/**
@@ -206,17 +208,33 @@ class ValidationTest extends WebTestBase {
$edit = array();
$this->drupalPostForm('form-test/validate-required', $edit, 'Submit');
+ $messages = [];
foreach (Element::children($form) as $key) {
if (isset($form[$key]['#required_error'])) {
$this->assertNoText(t('!name field is required.', array('!name' => $form[$key]['#title'])));
- $this->assertText($form[$key]['#required_error']);
+ $messages[] = [
+ 'title' => $form[$key]['#title'],
+ 'message' => $form[$key]['#required_error'],
+ 'key' => $key,
+ ];
}
elseif (isset($form[$key]['#form_test_required_error'])) {
$this->assertNoText(t('!name field is required.', array('!name' => $form[$key]['#title'])));
- $this->assertText($form[$key]['#form_test_required_error']);
+ $messages[] = [
+ 'title' => $form[$key]['#title'],
+ 'message' => $form[$key]['#form_test_required_error'],
+ 'key' => $key,
+ ];
+ }
+ elseif (!empty($form[$key]['#required'])) {
+ $messages[] = [
+ 'title' => $form[$key]['#title'],
+ 'message' => t('!name field is required.', ['!name' => $form[$key]['#title']]),
+ 'key' => $key,
+ ];
}
}
- $this->assertNoText(t('An illegal choice has been detected. Please contact the site administrator.'));
+ $this->assertErrorMessages($messages);
// Verify that no custom validation error appears with valid values.
$edit = array(
@@ -226,6 +244,7 @@ class ValidationTest extends WebTestBase {
);
$this->drupalPostForm('form-test/validate-required', $edit, 'Submit');
+ $messages = [];
foreach (Element::children($form) as $key) {
if (isset($form[$key]['#required_error'])) {
$this->assertNoText(t('!name field is required.', array('!name' => $form[$key]['#title'])));
@@ -235,7 +254,49 @@ class ValidationTest extends WebTestBase {
$this->assertNoText(t('!name field is required.', array('!name' => $form[$key]['#title'])));
$this->assertNoText($form[$key]['#form_test_required_error']);
}
+ elseif (!empty($form[$key]['#required'])) {
+ $messages[] = [
+ 'title' => $form[$key]['#title'],
+ 'message' => t('!name field is required.', ['!name' => $form[$key]['#title']]),
+ 'key' => $key,
+ ];
+ }
+ }
+ $this->assertErrorMessages($messages);
+ }
+
+ /**
+ * Asserts that the given error messages are displayed.
+ *
+ * @param array $messages
+ * An associative array of error messages keyed by the order they appear on
+ * the page, with the following key-value pairs:
+ * - title: The human readable form element title.
+ * - message: The error message for this form element.
+ * - key: The key used for the form element.
+ */
+ protected function assertErrorMessages($messages) {
+ $element = $this->xpath('//div[@class = "form-error-message"]/strong');
+ $this->assertIdentical(count($messages), count($element));
+
+ $error_links = [];
+ foreach ($messages as $delta => $message) {
+ // Ensure the message appears in the correct place.
+ if (!isset($element[$delta])) {
+ $this->fail(format_string('The error message for the "@title" element with key "@key" was not found.', ['@title' => $message['title'], '@key' => $message['key']]));
+ }
+ else {
+ $this->assertIdentical($message['message'], (string) $element[$delta]);
+ }
+
+ // Gather the element for checking the jump link section.
+ $error_links[] = \Drupal::l($message['title'], Url::fromRoute('<none>', [], ['fragment' => 'edit-' . str_replace('_', '-', $message['key']), 'external' => TRUE]));
}
+ $top_message = \Drupal::translation()->formatPlural(count($error_links), '1 error has been found: !errors', '@count errors have been found: !errors', [
+ '!errors' => SafeMarkup::set(implode(', ', $error_links))
+ ]);
+ $this->assertRaw($top_message);
$this->assertNoText(t('An illegal choice has been detected. Please contact the site administrator.'));
}
+
}
diff --git a/core/modules/system/templates/datetime-wrapper.html.twig b/core/modules/system/templates/datetime-wrapper.html.twig
index d5418df..bb4b3aa 100644
--- a/core/modules/system/templates/datetime-wrapper.html.twig
+++ b/core/modules/system/templates/datetime-wrapper.html.twig
@@ -24,4 +24,9 @@
<h4{{ title_attributes.addClass(title_classes) }}>{{ title }}</h4>
{% endif %}
{{ content }}
+{% if errors %}
+ <div>
+ {{ errors }}
+ </div>
+{% endif %}
{{ description }}
diff --git a/core/modules/system/templates/fieldset.html.twig b/core/modules/system/templates/fieldset.html.twig
index ab6796c..aace950 100644
--- a/core/modules/system/templates/fieldset.html.twig
+++ b/core/modules/system/templates/fieldset.html.twig
@@ -5,6 +5,7 @@
*
* Available variables:
* - attributes: HTML attributes for the fieldset element.
+ * - errors: (optional) Any errors for this fieldset element, may not be set.
* - required: Boolean indicating whether the fieldeset element is required.
* - legend: The legend element containing the following properties:
* - title: Title of the fieldset, intended for use as the text of the legend.
@@ -33,6 +34,11 @@
<span{{ legend_span.attributes.addClass(legend_span_classes) }}>{{ legend.title }}</span>
</legend>
<div class="fieldset-wrapper">
+ {% if errors %}
+ <div>
+ {{ errors }}
+ </div>
+ {% endif %}
{% if prefix %}
<span class="field-prefix">{{ prefix }}</span>
{% endif %}
diff --git a/core/modules/system/templates/form-element.html.twig b/core/modules/system/templates/form-element.html.twig
index dbbb830..b1ca6e9 100644
--- a/core/modules/system/templates/form-element.html.twig
+++ b/core/modules/system/templates/form-element.html.twig
@@ -5,6 +5,7 @@
*
* Available variables:
* - attributes: HTML attributes for the containing element.
+ * - errors: (optional) Any errors for this form element, may not be set.
* - prefix: (optional) The form element prefix, may not be set.
* - suffix: (optional) The form element suffix, may not be set.
* - required: The required marker, or empty if the associated form element is
@@ -52,6 +53,7 @@
'form-item-' ~ name|clean_class,
title_display not in ['after', 'before'] ? 'form-no-label',
disabled == 'disabled' ? 'form-disabled',
+ errors ? 'form-error',
]
%}
{%
@@ -79,6 +81,11 @@
{% if label_display == 'after' %}
{{ label }}
{% endif %}
+ {% if errors %}
+ <div class="form-error-message">
+ {{ errors }}
+ </div>
+ {% endif %}
{% if description_display in ['after', 'invisible'] and description.content %}
<div{{ description.attributes.addClass(description_classes) }}>
{{ description.content }}
diff --git a/core/modules/user/src/Tests/UserBlocksTest.php b/core/modules/user/src/Tests/UserBlocksTest.php
index 1349264..366f519 100644
--- a/core/modules/user/src/Tests/UserBlocksTest.php
+++ b/core/modules/user/src/Tests/UserBlocksTest.php
@@ -7,6 +7,7 @@
namespace Drupal\user\Tests;
+use Drupal\Component\Utility\SafeMarkup;
use Drupal\simpletest\WebTestBase;
/**
@@ -43,6 +44,16 @@ class UserBlocksTest extends WebTestBase {
* Test the user login block.
*/
function testUserLoginBlock() {
+ // Make sure the validation error is displayed when try to login with
+ // invalid username/password.
+ $edit['name'] = $this->randomMachineName();
+ $edit['pass'] = $this->randomMachineName();
+ $this->drupalPostForm('node', $edit, t('Log in'));
+ $this->assertRaw(\Drupal::translation()->formatPlural(1, '1 error has been found: !errors', '@count errors have been found: !errors', [
+ '!errors' => SafeMarkup::set('<a href="#edit-name">Username</a>')
+ ]));
+ $this->assertText(t('Sorry, unrecognized username or password.'));
+
// Create a user with some permission that anonymous users lack.
$user = $this->drupalCreateUser(array('administer permissions'));
diff --git a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
index 825bd8a..77c8763 100644
--- a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
+++ b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php
@@ -370,15 +370,11 @@ class FormBuilderTest extends FormTestBase {
->method('buildForm')
->will($this->returnValue($expected_form));
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
+ $form_state = new FormState();
$form = $this->simulateFormSubmission($form_id, $form_arg, $form_state);
$this->assertSame('test-form-id', $form['#id']);
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
+ $form_state = new FormState();
$form = $this->simulateFormSubmission($form_id, $form_arg, $form_state);
$this->assertSame('test-form-id--2', $form['#id']);
}
diff --git a/core/tests/Drupal/Tests/Core/Form/FormElementHelperTest.php b/core/tests/Drupal/Tests/Core/Form/FormElementHelperTest.php
new file mode 100644
index 0000000..d25a6f5
--- /dev/null
+++ b/core/tests/Drupal/Tests/Core/Form/FormElementHelperTest.php
@@ -0,0 +1,179 @@
+<?php
+
+/**
+ * @file
+ * Contains \Drupal\Tests\Core\Form\FormElementHelperTest.
+ */
+
+namespace Drupal\Tests\Core\Form;
+
+use Drupal\Core\Form\FormElementHelper;
+use Drupal\Tests\UnitTestCase;
+
+/**
+ * Tests the form element helper.
+ *
+ * @group Drupal
+ * @group Form
+ *
+ * @coversDefaultClass \Drupal\Core\Form\FormElementHelper
+ */
+class FormElementHelperTest extends UnitTestCase {
+
+ /**
+ * Tests the getElementByName() method.
+ *
+ * @covers ::getElementByName
+ *
+ * @dataProvider getElementByNameProvider
+ */
+ public function testGetElementByName($name, $form, $expected) {
+ $this->assertSame($expected, FormElementHelper::getElementByName($name, $form));
+ }
+
+ /**
+ * Provides test data.
+ */
+ public function getElementByNameProvider() {
+ $data = [];
+ $data[] = ['id', [], []];
+ $data[] = [
+ 'id',
+ [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ ],
+ [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ ];
+ $data[] = [
+ 'id',
+ [
+ 'fieldset' => [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ ];
+ $data[] = [
+ 'fieldset',
+ [
+ 'fieldset' => [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ];
+ $data[] = [
+ 'fieldset][id',
+ [
+ 'fieldset' => [
+ '#tree' => TRUE,
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['fieldset', 'id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ [
+ '#title' => 'ID',
+ '#parents' => ['fieldset', 'id'],
+ ],
+ ];
+ return $data;
+ }
+
+ /**
+ * Tests the getElementTitle() method.
+ *
+ * @covers ::getElementTitle
+ *
+ * @dataProvider getElementTitleProvider
+ */
+ public function testGetElementTitle($name, $form, $expected) {
+ $element = FormElementHelper::getElementByName($name, $form);
+ $this->assertSame($expected, FormElementHelper::getElementTitle($element));
+ }
+
+ /**
+ * Provides test data.
+ */
+ public function getElementTitleProvider() {
+ $data = [];
+ $data[] = ['id', [], ''];
+ $data[] = [
+ 'id',
+ [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ ],
+ 'ID',
+ ];
+ $data[] = [
+ 'id',
+ [
+ 'fieldset' => [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ 'ID',
+ ];
+ $data[] = [
+ 'fieldset',
+ [
+ 'fieldset' => [
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ 'ID',
+ ];
+ $data[] = [
+ 'fieldset][id',
+ [
+ 'fieldset' => [
+ '#tree' => TRUE,
+ 'id' => [
+ '#title' => 'ID',
+ '#parents' => ['fieldset', 'id'],
+ ],
+ '#parents' => ['fieldset'],
+ ],
+ ],
+ 'ID',
+ ];
+ return $data;
+ }
+
+}
diff --git a/core/tests/Drupal/Tests/Core/Form/FormErrorHandlerTest.php b/core/tests/Drupal/Tests/Core/Form/FormErrorHandlerTest.php
new file mode 100644
index 0000000..53d7e76
--- /dev/null
+++ b/core/tests/Drupal/Tests/Core/Form/FormErrorHandlerTest.php
@@ -0,0 +1,125 @@
+<?php
+
+/**
+ * @file
+ * Contains \Drupal\Tests\Core\Form\FormErrorHandlerTest.
+ */
+
+namespace Drupal\Tests\Core\Form;
+
+use Drupal\Core\Form\FormState;
+use Drupal\Tests\UnitTestCase;
+
+/**
+ * @coversDefaultClass \Drupal\Core\Form\FormErrorHandler
+ * @group Form
+ */
+class FormErrorHandlerTest extends UnitTestCase {
+
+ /**
+ * @covers ::handleFormErrors
+ * @covers ::displayErrorMessages
+ */
+ public function testDisplayErrorMessages() {
+ $link_generator = $this->getMock('Drupal\Core\Utility\LinkGeneratorInterface');
+ $link_generator->expects($this->any())
+ ->method('generate')
+ ->willReturnArgument(0);
+ $form_error_handler = $this->getMockBuilder('Drupal\Core\Form\FormErrorHandler')
+ ->setConstructorArgs([$this->getStringTranslationStub(), $link_generator])
+ ->setMethods(['drupalSetMessage'])
+ ->getMock();
+
+ $form_error_handler->expects($this->at(0))
+ ->method('drupalSetMessage')
+ ->with('no title given', 'error');
+ $form_error_handler->expects($this->at(1))
+ ->method('drupalSetMessage')
+ ->with('element is invisible', 'error');
+ $form_error_handler->expects($this->at(2))
+ ->method('drupalSetMessage')
+ ->with('this missing element is invalid', 'error');
+ $form_error_handler->expects($this->at(3))
+ ->method('drupalSetMessage')
+ ->with('3 errors have been found: Test 1, Test 2 &amp; a half, Test 3', 'error');
+
+ $form = [
+ '#parents' => [],
+ ];
+ $form['test1'] = [
+ '#type' => 'textfield',
+ '#title' => 'Test 1',
+ '#parents' => ['test1'],
+ '#id' => 'edit-test1',
+ ];
+ $form['test2'] = [
+ '#type' => 'textfield',
+ '#title' => 'Test 2 & a half',
+ '#parents' => ['test2'],
+ '#id' => 'edit-test2',
+ ];
+ $form['fieldset'] = [
+ '#parents' => ['fieldset'],
+ 'test3' => [
+ '#type' => 'textfield',
+ '#title' => 'Test 3',
+ '#parents' => ['fieldset', 'test3'],
+ '#id' => 'edit-test3',
+ ],
+ ];
+ $form['test4'] = [
+ '#type' => 'textfield',
+ '#title' => 'Test 4',
+ '#parents' => ['test4'],
+ '#id' => 'edit-test4',
+ '#error_no_message' => TRUE,
+ ];
+ $form['test5'] = [
+ '#type' => 'textfield',
+ '#parents' => ['test5'],
+ '#id' => 'edit-test5',
+ ];
+ $form['test6'] = [
+ '#type' => 'value',
+ '#title' => 'Test 6',
+ '#parents' => ['test6'],
+ '#id' => 'edit-test6',
+ ];
+ $form_state = new FormState();
+ $form_state->setErrorByName('test1', 'invalid');
+ $form_state->setErrorByName('test2', 'invalid');
+ $form_state->setErrorByName('fieldset][test3', 'invalid');
+ $form_state->setErrorByName('test4', 'no error message');
+ $form_state->setErrorByName('test5', 'no title given');
+ $form_state->setErrorByName('test6', 'element is invisible');
+ $form_state->setErrorByName('missing_element', 'this missing element is invalid');
+ $form_error_handler->handleFormErrors($form, $form_state);
+ $this->assertSame('invalid', $form['test1']['#errors']);
+ }
+
+ /**
+ * @covers ::handleFormErrors
+ * @covers ::setElementErrorsFromFormState
+ */
+ public function testSetElementErrorsFromFormState() {
+ $form_error_handler = $this->getMockBuilder('Drupal\Core\Form\FormErrorHandler')
+ ->setConstructorArgs([$this->getStringTranslationStub(), $this->getMock('Drupal\Core\Utility\LinkGeneratorInterface')])
+ ->setMethods(['drupalSetMessage'])
+ ->getMock();
+
+ $form = [
+ '#parents' => [],
+ ];
+ $form['test'] = [
+ '#type' => 'textfield',
+ '#title' => 'Test',
+ '#parents' => ['test'],
+ '#id' => 'edit-test',
+ ];
+ $form_state = new FormState();
+ $form_state->setErrorByName('test', 'invalid');
+ $form_error_handler->handleFormErrors($form, $form_state);
+ $this->assertSame('invalid', $form['test']['#errors']);
+ }
+
+}
diff --git a/core/tests/Drupal/Tests/Core/Form/FormStateTest.php b/core/tests/Drupal/Tests/Core/Form/FormStateTest.php
index 24c4ddd..b92c5f7 100644
--- a/core/tests/Drupal/Tests/Core/Form/FormStateTest.php
+++ b/core/tests/Drupal/Tests/Core/Form/FormStateTest.php
@@ -62,15 +62,10 @@ class FormStateTest extends UnitTestCase {
* @covers ::setError
*/
public function testSetError() {
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
- $form_state->expects($this->once())
- ->method('drupalSetMessage')
- ->willReturn('Fail');
-
+ $form_state = new FormState();
$element['#parents'] = array('foo', 'bar');
$form_state->setError($element, 'Fail');
+ $this->assertSame(['foo][bar' => 'Fail'], $form_state->getErrors());
}
/**
@@ -108,14 +103,10 @@ class FormStateTest extends UnitTestCase {
*
* @dataProvider providerTestSetErrorByName
*/
- public function testSetErrorByName($limit_validation_errors, $expected_errors, $set_message = FALSE) {
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
+ public function testSetErrorByName($limit_validation_errors, $expected_errors) {
+ $form_state = new FormState();
$form_state->setLimitValidationErrors($limit_validation_errors);
$form_state->clearErrors();
- $form_state->expects($set_message ? $this->once() : $this->never())
- ->method('drupalSetMessage');
$form_state->setErrorByName('test', 'Fail 1');
$form_state->setErrorByName('test', 'Fail 2');
@@ -131,7 +122,7 @@ class FormStateTest extends UnitTestCase {
array(array(array('options')), array('options' => '')),
// Do not limit an validation, and, ensuring the first error is returned
// for the 'test' element.
- array(NULL, array('test' => 'Fail 1', 'options' => ''), TRUE),
+ [NULL, ['test' => 'Fail 1', 'options' => '']],
// Limit all validation.
array(array(), array()),
);
@@ -146,9 +137,7 @@ class FormStateTest extends UnitTestCase {
* @expectedExceptionMessage Form errors cannot be set after form validation has finished.
*/
public function testFormErrorsDuringSubmission() {
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
+ $form_state = new FormState();
$form_state->setValidationComplete();
$form_state->setErrorByName('test', 'message');
}
diff --git a/core/tests/Drupal/Tests/Core/Form/FormTestBase.php b/core/tests/Drupal/Tests/Core/Form/FormTestBase.php
index 12b3260..b6fe9dd 100644
--- a/core/tests/Drupal/Tests/Core/Form/FormTestBase.php
+++ b/core/tests/Drupal/Tests/Core/Form/FormTestBase.php
@@ -178,9 +178,10 @@ abstract class FormTestBase extends UnitTestCase {
$this->requestStack = new RequestStack();
$this->requestStack->push($this->request);
$this->logger = $this->getMock('Drupal\Core\Logger\LoggerChannelInterface');
+ $form_error_handler = $this->getMock('Drupal\Core\Form\FormErrorHandlerInterface');
$this->formValidator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->setConstructorArgs(array($this->requestStack, $this->getStringTranslationStub(), $this->csrfToken, $this->logger))
- ->setMethods(array('drupalSetMessage'))
+ ->setConstructorArgs([$this->requestStack, $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $form_error_handler])
+ ->setMethods(NULL)
->getMock();
$this->formSubmitter = $this->getMockBuilder('Drupal\Core\Form\FormSubmitter')
->setConstructorArgs(array($this->requestStack, $this->urlGenerator))
diff --git a/core/tests/Drupal/Tests/Core/Form/FormValidatorTest.php b/core/tests/Drupal/Tests/Core/Form/FormValidatorTest.php
index 5165c0c..d701991 100644
--- a/core/tests/Drupal/Tests/Core/Form/FormValidatorTest.php
+++ b/core/tests/Drupal/Tests/Core/Form/FormValidatorTest.php
@@ -20,6 +20,39 @@ use Symfony\Component\HttpFoundation\RequestStack;
class FormValidatorTest extends UnitTestCase {
/**
+ * A logger instance.
+ *
+ * @var \Psr\Log\LoggerInterface|\PHPUnit_Framework_MockObject_MockObject
+ */
+ protected $logger;
+
+ /**
+ * The CSRF token generator to validate the form token.
+ *
+ * @var \Drupal\Core\Access\CsrfTokenGenerator|\PHPUnit_Framework_MockObject_MockObject
+ */
+ protected $csrfToken;
+
+ /**
+ * The form error handler.
+ *
+ * @var \Drupal\Core\Form\FormErrorHandlerInterface|\PHPUnit_Framework_MockObject_MockObject
+ */
+ protected $formErrorHandler;
+
+ /**
+ * {@inheritdoc}
+ */
+ protected function setUp() {
+ parent::setUp();
+ $this->logger = $this->getMock('Psr\Log\LoggerInterface');
+ $this->csrfToken = $this->getMockBuilder('Drupal\Core\Access\CsrfTokenGenerator')
+ ->disableOriginalConstructor()
+ ->getMock();
+ $this->formErrorHandler = $this->getMock('Drupal\Core\Form\FormErrorHandlerInterface');
+ }
+
+ /**
* Tests the 'validation_complete' $form_state flag.
*
* @covers ::validateForm
@@ -27,7 +60,7 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testValidationComplete() {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(NULL)
->getMock();
@@ -45,7 +78,7 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testPreventDuplicateValidation() {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('doValidateForm'))
->getMock();
$form_validator->expects($this->never())
@@ -65,18 +98,19 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testMustValidate() {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('doValidateForm'))
->getMock();
$form_validator->expects($this->once())
->method('doValidateForm');
+ $this->formErrorHandler->expects($this->once())
+ ->method('handleFormErrors');
$form = array();
$form_state = (new FormState())
->setValidationComplete()
->setValidationEnforced();
$form_validator->validateForm('test_form_id', $form, $form_state);
- $this->assertArrayHasKey('#errors', $form);
}
/**
@@ -86,16 +120,12 @@ class FormValidatorTest extends UnitTestCase {
$request_stack = new RequestStack();
$request = new Request(array(), array(), array(), array(), array(), array('REQUEST_URI' => '/test/example?foo=bar'));
$request_stack->push($request);
- $csrf_token = $this->getMockBuilder('Drupal\Core\Access\CsrfTokenGenerator')
- ->disableOriginalConstructor()
- ->getMock();
- $csrf_token->expects($this->once())
+ $this->csrfToken->expects($this->once())
->method('validate')
->will($this->returnValue(FALSE));
- $logger = $this->getMock('Psr\Log\LoggerInterface');
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->setConstructorArgs(array($request_stack, $this->getStringTranslationStub(), $csrf_token, $logger))
+ ->setConstructorArgs([$request_stack, $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('doValidateForm'))
->getMock();
$form_validator->expects($this->never())
@@ -118,16 +148,12 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testValidateValidFormToken() {
$request_stack = new RequestStack();
- $csrf_token = $this->getMockBuilder('Drupal\Core\Access\CsrfTokenGenerator')
- ->disableOriginalConstructor()
- ->getMock();
- $csrf_token->expects($this->once())
+ $this->csrfToken->expects($this->once())
->method('validate')
->will($this->returnValue(TRUE));
- $logger = $this->getMock('Psr\Log\LoggerInterface');
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->setConstructorArgs(array($request_stack, $this->getStringTranslationStub(), $csrf_token, $logger))
+ ->setConstructorArgs([$request_stack, $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('doValidateForm'))
->getMock();
$form_validator->expects($this->once())
@@ -145,38 +171,13 @@ class FormValidatorTest extends UnitTestCase {
}
/**
- * @covers ::setElementErrorsFromFormState
- */
- public function testSetElementErrorsFromFormState() {
- $form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
- ->setMethods(NULL)
- ->getMock();
-
- $form = array(
- '#parents' => array(),
- );
- $form['test'] = array(
- '#type' => 'textfield',
- '#title' => 'Test',
- '#parents' => array('test'),
- );
- $form_state = $this->getMockBuilder('Drupal\Core\Form\FormState')
- ->setMethods(array('drupalSetMessage'))
- ->getMock();
- $form_state->setErrorByName('test', 'invalid');
- $form_validator->validateForm('test_form_id', $form, $form_state);
- $this->assertSame('invalid', $form['test']['#errors']);
- }
-
- /**
* @covers ::handleErrorsWithLimitedValidation
*
* @dataProvider providerTestHandleErrorsWithLimitedValidation
*/
public function testHandleErrorsWithLimitedValidation($sections, $triggering_element, $values, $expected) {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(NULL)
->getMock();
@@ -272,7 +273,7 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testExecuteValidateHandlers() {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(NULL)
->getMock();
$mock = $this->getMock('stdClass', array('validate_handler', 'hash_validate'));
@@ -302,13 +303,8 @@ class FormValidatorTest extends UnitTestCase {
* @dataProvider providerTestRequiredErrorMessage
*/
public function testRequiredErrorMessage($element, $expected_message) {
- $csrf_token = $this->getMockBuilder('Drupal\Core\Access\CsrfTokenGenerator')
- ->disableOriginalConstructor()
- ->getMock();
- $logger = $this->getMock('Psr\Log\LoggerInterface');
-
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->setConstructorArgs(array(new RequestStack(), $this->getStringTranslationStub(), $csrf_token, $logger))
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('executeValidateHandlers'))
->getMock();
$form_validator->expects($this->once())
@@ -356,7 +352,7 @@ class FormValidatorTest extends UnitTestCase {
*/
public function testElementValidate() {
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->disableOriginalConstructor()
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('executeValidateHandlers'))
->getMock();
$form_validator->expects($this->once())
@@ -383,18 +379,13 @@ class FormValidatorTest extends UnitTestCase {
* @dataProvider providerTestPerformRequiredValidation
*/
public function testPerformRequiredValidation($element, $expected_message, $call_watchdog) {
- $csrf_token = $this->getMockBuilder('Drupal\Core\Access\CsrfTokenGenerator')
- ->disableOriginalConstructor()
- ->getMock();
- $logger = $this->getMock('Psr\Log\LoggerInterface');
-
$form_validator = $this->getMockBuilder('Drupal\Core\Form\FormValidator')
- ->setConstructorArgs(array(new RequestStack(), $this->getStringTranslationStub(), $csrf_token, $logger))
+ ->setConstructorArgs([new RequestStack(), $this->getStringTranslationStub(), $this->csrfToken, $this->logger, $this->formErrorHandler])
->setMethods(array('setError'))
->getMock();
if ($call_watchdog) {
- $logger->expects($this->once())
+ $this->logger->expects($this->once())
->method('error')
->with($this->isType('string'), $this->isType('array'));
}
diff --git a/core/tests/Drupal/Tests/UnitTestCase.php b/core/tests/Drupal/Tests/UnitTestCase.php
index 9e70a73..67f7bcd 100644
--- a/core/tests/Drupal/Tests/UnitTestCase.php
+++ b/core/tests/Drupal/Tests/UnitTestCase.php
@@ -9,6 +9,7 @@ namespace Drupal\Tests;
use Drupal\Component\FileCache\FileCacheFactory;
use Drupal\Component\Utility\Random;
+use Drupal\Component\Utility\SafeMarkup;
use Drupal\Core\Cache\CacheTagsInvalidatorInterface;
use Drupal\Core\DependencyInjection\ContainerBuilder;
@@ -208,6 +209,11 @@ abstract class UnitTestCase extends \PHPUnit_Framework_TestCase {
$translation->expects($this->any())
->method('translate')
->will($this->returnCallback('Drupal\Component\Utility\SafeMarkup::format'));
+ $translation->expects($this->any())
+ ->method('formatPlural')
+ ->willReturnCallback(function ($count, $singular, $plural, array $args = [], array $options = []) {
+ return $count === 1 ? SafeMarkup::format($singular, $args) : SafeMarkup::format($plural, $args + ['@count' => $count]);
+ });
return $translation;
}
diff --git a/core/themes/bartik/css/components/form.css b/core/themes/bartik/css/components/form.css
index a5cfcf6..4574da0 100644
--- a/core/themes/bartik/css/components/form.css
+++ b/core/themes/bartik/css/components/form.css
@@ -269,3 +269,13 @@ input.form-submit:focus {
margin-left: 0.6em;
margin-right: 0;
}
+
+/* Form error styles. */
+.form-item textarea.error + .cke {
+ border: 2px solid red;
+}
+
+/* Form error message styles. */
+.form-error-message {
+ color: #ea2800;
+}
diff --git a/core/themes/classy/templates/form/datetime-wrapper.html.twig b/core/themes/classy/templates/form/datetime-wrapper.html.twig
index 399dea2..56da173 100644
--- a/core/themes/classy/templates/form/datetime-wrapper.html.twig
+++ b/core/themes/classy/templates/form/datetime-wrapper.html.twig
@@ -23,6 +23,11 @@
<h4{{ title_attributes.addClass(title_classes) }}>{{ title }}</h4>
{% endif %}
{{ content }}
+{% if errors %}
+ <div class="form-error-message">
+ <strong>{{ errors }}</strong>
+ </div>
+{% endif %}
{% if description %}
<div class="description">{{ description }}</div>
{% endif %}
diff --git a/core/themes/classy/templates/form/fieldset.html.twig b/core/themes/classy/templates/form/fieldset.html.twig
index f7460cf..9286074 100644
--- a/core/themes/classy/templates/form/fieldset.html.twig
+++ b/core/themes/classy/templates/form/fieldset.html.twig
@@ -5,6 +5,7 @@
*
* Available variables:
* - attributes: HTML attributes for the fieldset element.
+ * - errors: (optional) Any errors for this fieldset element, may not be set.
* - required: Boolean indicating whether the fieldeset element is required.
* - legend: The legend element containing the following properties:
* - title: Title of the fieldset, intended for use as the text of the legend.
@@ -31,6 +32,11 @@
<span{{ legend_span.attributes.addClass(legend_span_classes) }}>{{ legend.title }}</span>
</legend>
<div class="fieldset-wrapper">
+ {% if errors %}
+ <div class="form-error-message">
+ <strong>{{ errors }}</strong>
+ </div>
+ {% endif %}
{% if prefix %}
<span class="field-prefix">{{ prefix }}</span>
{% endif %}
diff --git a/core/themes/classy/templates/form/form-element.html.twig b/core/themes/classy/templates/form/form-element.html.twig
index 5dc4000..f0d4d56 100644
--- a/core/themes/classy/templates/form/form-element.html.twig
+++ b/core/themes/classy/templates/form/form-element.html.twig
@@ -5,6 +5,7 @@
*
* Available variables:
* - attributes: HTML attributes for the containing element.
+ * - errors: (optional) Any errors for this form element, may not be set.
* - prefix: (optional) The form element prefix, may not be set.
* - suffix: (optional) The form element suffix, may not be set.
* - required: The required marker, or empty if the associated form element is
@@ -51,6 +52,7 @@
'form-item-' ~ name|clean_class,
title_display not in ['after', 'before'] ? 'form-no-label',
disabled == 'disabled' ? 'form-disabled',
+ errors ? 'form-error',
]
%}
{%
@@ -78,6 +80,11 @@
{% if label_display == 'after' %}
{{ label }}
{% endif %}
+ {% if errors %}
+ <div class="form-error-message">
+ <strong>{{ errors }}</strong>
+ </div>
+ {% endif %}
{% if description_display in ['after', 'invisible'] and description.content %}
<div{{ description.attributes.addClass(description_classes) }}>
{{ description.content }}
diff --git a/core/themes/seven/css/components/form.css b/core/themes/seven/css/components/form.css
index 5dd19be..07232de 100644
--- a/core/themes/seven/css/components/form.css
+++ b/core/themes/seven/css/components/form.css
@@ -55,7 +55,6 @@ label[for] {
.form-disabled label {
color: #737373;
}
-
.form-disabled input.form-text,
.form-disabled input.form-tel,
.form-disabled input.form-email,
@@ -70,16 +69,19 @@ label[for] {
background-color: hsla(0, 0%, 0%, .08);
box-shadow: none;
}
-
.form-item input.error,
.form-item textarea.error,
.form-item select.error {
- border-width: 2px;
+ border-width: 1px;
border-color: #e62600;
background-color: hsla(15, 75%, 97%, 1);
box-shadow: inset 0 5px 5px -5px #b8b8b8;
color: #a51b00;
}
+.form-item textarea.error + .cke {
+ border-width: 1px;
+ border-color: #e62600;
+}
.form-item input.error:focus,
.form-item textarea.error:focus,
.form-item select.error:focus {
@@ -94,6 +96,19 @@ label[for] {
width: 7px;
height: 7px;
}
+.form-error-message {
+ margin-top: 0.15em;
+ color: #ea2800;
+}
+.fieldset-wrapper > .form-error-message {
+ margin-top: 0;
+}
+.text-format-wrapper .form-error-message {
+ border: solid #ccc;
+ border-width: 0 1px;
+ margin: 0;
+ padding: 0.25em 0.666em 0;
+}
/* Filter */
@@ -105,6 +120,7 @@ div.description,
font-size: 0.95em;
}
.form-item .description.error {
+ margin-top: 0;
color: #a51b00;
}
@@ -183,15 +199,6 @@ textarea.form-textarea {
width: auto;
}
-.form-item .password-suggestions {
- float: left; /* LTR */
- clear: left; /* LTR */
- width: 100%;
-}
-[dir="rtl"] .form-item .password-suggestions {
- float: right;
- clear: right;
-}
.form-item-pass .description {
clear: both;
}