MAGETWO-37818: Customer invalid email message is not displayed

Author: okarpenko

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

@@ -29,15 +29,19 @@ 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
-                        });
-                    });
+                    return true;
                 }
+
+                $('body').notification('clear');
+                $.each(resp.messages, function(key, message) {
+                    $('body').notification('add', {
+                        error: resp.error,
+                        message: message,
+                        insertMethod: function(message) {
+                            $('.page-main-actions').after(message);
+                        }
+                    });
+                });
             },
             complete: function () {
                 $('body').trigger('processStop');

app/code/Magento/Ui/view/base/web/templates/form/fieldset.html

@@ -5,23 +5,22 @@
  */
 -->
 
-<div 
-    class="admin__fieldset-wrapper"
-    data-bind="css: {'admin__collapsible-block-wrapper': collapsible, 'opened': opened}">
-    <div 
-        class="admin__fieldset-wrapper-title"
-        tabindex="3"
-        data-bind="click: toggleOpened, keyboard: { 13: toggleOpened }">
+<div
+        class="admin__fieldset-wrapper"
+        data-bind="css: {'admin__collapsible-block-wrapper': collapsible, 'opened': opened}">
+    <div
+            class="admin__fieldset-wrapper-title"
+            tabindex="3"
+            data-bind="click: toggleOpened, keyboard: { 13: toggleOpened }">
         <strong class="title">
             <span data-bind="text: label"></span>
         </strong>
     </div>
     <div class="admin__fieldset-wrapper-content" data-bind="visible: opened">
-        <div class="messages"></div>
         <fieldset class="admin__fieldset">
-        <!-- ko foreach: { data: elems, as: 'element' } -->
+            <!-- ko foreach: { data: elems, as: 'element' } -->
             <!-- ko template: element.getTemplate() --><!-- /ko -->
-        <!-- /ko -->
+            <!-- /ko -->
         </fieldset>
     </div>
 </div>

lib/web/mage/backend/notification.js

@@ -13,8 +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>',
-                error: '<div class="messages"><div class="message message-error error"><div data-ui-id="messages-message-error" ><%- data.message %></div></div></div>'
+                global: '<div id="messages"><div class="message <% if (data.error) { %>error<% } %>"><div><%- data.message %></div></div></div>',
+                error: '<div id="messages"><div class="messages"><div class="message message-error error"><div data-ui-id="messages-message-error"><%- data.message %></div></div></div></div>'
             }
         },
 
@@ -52,16 +52,20 @@ define([
             var message = mageTemplate(template, {
                 data: data
             });
-            var messageContainer = data.messageContainer || '.messages:first';
-            $(messageContainer).prepend(message);
+            if (typeof data.insertMethod === 'function') {
+                data.insertMethod(message);
+            } else {
+                var messageContainer = data.messageContainer || '#messages';
+                $(messageContainer).prepend(message);
+            }
             return this;
         },
 
         /**
          * Removes error messages.
          */
         clear: function () {
-            $('.messages').html('');
+            $('#messages').html('');
         }
     });
 

lib/web/mage/backend/validation.js

@@ -148,7 +148,10 @@
             $('body').notification('clear')
                 .notification('add', {
                     error: data.error,
-                    message: data.message
+                    message: data.message,
+                    insertMethod: function(message) {
+                        $('.messages:first').html(message);
+                    }
                 });
         },