Remove usages of the deprecated valid_email_address().

+++ b/core/modules/action/src/Plugin/Action/EmailAction.php
@@ -58,13 +59,21 @@ class EmailAction extends ConfigurableActionBase implements ContainerFactoryPlug
+  /**
+   * The language manager.

unrelated change.

I know it is difficult to resist, but only change lines that are needed to stay in scope.

+++ b/core/modules/action/src/Plugin/Action/EmailAction.php
@@ -83,8 +91,10 @@ class EmailAction extends ConfigurableActionBase implements ContainerFactoryPlug
+   * @param \Egulias\EmailValidator\EmailValidator
+   *   The email validator.

since this a new line, we can make it meet standards (but dont fix the lines above that are not changed by this patch).

https://www.drupal.org/node/1354#param

add the var name.

pre-emptive rtbc, as it *should* come back green.

Read the whole thing and it stays in scope.

1. +++ b/core/lib/Drupal/Core/Render/Element/Email.php
   @@ -64,7 +64,7 @@ public static function validateEmail(&$element, FormStateInterface $form_state,
   -    if ($value !== '' && !valid_email_address($value)) {
   +    if ($value !== '' && !\Drupal::service('email.validator')->isValid($value)) {
   

   I think we can inject this too by implementing ContainerInjectionInterface.

2. +++ b/core/modules/contact/src/ContactFormEditForm.php
   @@ -92,7 +92,7 @@ public function validate(array $form, FormStateInterface $form_state) {
   +      if (!\Drupal::service('email.validator')->isValid($recipient)) {
   
   +++ b/core/modules/update/src/UpdateSettingsForm.php
   @@ -86,7 +86,7 @@ public function validateForm(array &$form, FormStateInterface $form_state) {
   +          if (\Drupal::service('email.validator')->isValid($email)) {
   

   We should be injecting the service here rather that using \Drupal.

@cilefen sorry I missed that it was a static method - we have to leave a \Drupal :(

+++ b/core/modules/update/src/UpdateSettingsForm.php
@@ -86,7 +86,7 @@ public function validateForm(array &$form, FormStateInterface $form_state) {
-          if (valid_email_address($email)) {
+          if (\Drupal::service('email.validator')->isValid($email)) {

This definitely can be injected :)

After applying the patch, the grep command for finding the usage of valid_email_address() gives only 1 result ( see issue summary):

grep -r valid_email_address core/ 
core/includes/common.inc:function valid_email_address($mail) {

Committed bdb618d and pushed to 8.0.x. Thanks!

Thanks for adding the beta evaluation to the issue summary.