MAGETWO-37818: Customer invalid email message is not displayed

Author: okarpenko

app/code/Magento/Customer/Controller/Adminhtml/Index/Validate.php

@@ -57,9 +57,11 @@ protected function _validateCustomer($response)
         }
 
         if (!$errors->isValid()) {
+            $messages = [];
             foreach ($errors->getMessages() as $error) {
-                $this->messageManager->addError($error);
+                $messages[] = $error;
             }
+            $response->setMessages($messages);
             $response->setError(1);
         }
 
@@ -90,9 +92,11 @@ protected function _validateCustomerAddress($response)
 
             $errors = $addressForm->validateData($formData);
             if ($errors !== true) {
+                $messages = [];
                 foreach ($errors as $error) {
-                    $this->messageManager->addError($error);
+                    $messages[] = $error;
                 }
+                $response->setMessages($messages);
                 $response->setError(1);
             }
         }
@@ -114,9 +118,8 @@ public function execute()
         }
         $resultJson = $this->resultJsonFactory->create();
         if ($response->getError()) {
-            $layout = $this->layoutFactory->create();
-            $layout->initMessages();
-            $response->setHtmlMessage($layout->getMessagesBlock()->getGroupedHtml());
+            $response->setError(true);
+            $response->setMessages($response->getMessages());
         }
 
         $resultJson->setData($response);

app/code/Magento/Ui/view/base/web/js/form/client.js

@@ -29,6 +29,15 @@ define([
             success: function (resp) {
                 if (!resp.error) {
                     save.resolve();
+                } else {
+                    $('body').notification('clear');
+                    $.each(resp.messages, function(key, message) {
+                        $('body').notification('add', {
+                            error: resp.error,
+                            message: message,
+                            messageSelector: '#anchor-content'
+                        });
+                    });
                 }
             },
             complete: function () {

lib/web/mage/backend/notification.js

@@ -13,7 +13,8 @@ define([
     $.widget('mage.notification', {
         options: {
             templates: {
-                global: '<div class="messages"><div class="message <% if (data.error) { %>error<% } %>"><div><%- data.message %></div></div></div>'
+                global: '<div class="messages"><div class="message <% if (data.error) { %>error<% } %>"><div><%- data.message %></div></div></div>',
+                error: '<div class="messages"><div class="message message-error error"><div data-ui-id="messages-message-error" ><%- data.message %></div></div></div>'
             }
         },
 
@@ -47,20 +48,20 @@ define([
          * @param {Object} data - Data with a message to be displayed.
          */
         add: function (data) {
-            var message = mageTemplate(this.options.templates.global, {
+            var template = data.error ? this.options.templates.error : this.options.templates.global;
+            var message = mageTemplate(template, {
                 data: data
             });
-
-            $('#messages').append(message);
-
+            var messageSelector = data.messageSelector || '.messages';
+            $(messageSelector).prepend(message);
             return this;
         },
 
         /**
          * Removes error messages.
          */
         clear: function () {
-            $('#messages').html('');
+            $('.messages').html('');
         }
     });
 

lib/web/mage/backend/validation.js

@@ -145,34 +145,11 @@
          * @param {Object} data - Data that came from backend.
          */
         _showErrors: function(data) {
-            var attributes = data.attributes || {},
-                element;
-
-            if (data.attribute) {
-                attributes[data.attribute] = data.message;
-            }
-
-            $('body').notification('clear');
-
-            _.each(attributes, function(message, code) {
-                element = this._getByCode(code);
-
-                if(!element.length){
-                    $('body').notification('add', {
-                        error: true,
-                        message: message
-                    });
-
-                    return; 
-                }
-                
-                element
-                    .addClass('validate-ajax-error')
-                    .data('msg-validate-ajax-error', message);
-
-                this.validate.element(element);
-
-            }, this);
+            $('body').notification('clear')
+                .notification('add', {
+                    error: data.error,
+                    message: data.message
+                });
         },
 
         /**