Merge pull request #2926 from magento-chaika/MAGETWO-92036

[chaika] MAGETWO-92036: The error icon does not appear on sections with required attributes that are empty when click on 'save' button

Author: Lysenko Olexandr

app/code/Magento/Ui/view/base/web/js/form/components/fieldset.js

@@ -22,7 +22,7 @@ define([
             opened: false,
             level: 0,
             visible: true,
-            initializeFieldsetDataByDefault: false,    /* Data in some fieldsets should be initialized before open */
+            initializeFieldsetDataByDefault: false, /* Data in some fieldsets should be initialized before open */
             disabled: false,
             listens: {
                 'opened': 'onVisibilityChange'
@@ -77,9 +77,9 @@ define([
             elem.initContainer(this);
 
             elem.on({
-                'update':   this.onChildrenUpdate,
-                'loading':  this.onContentLoading,
-                'error':  this.onChildrenError
+                'update': this.onChildrenUpdate,
+                'loading': this.onContentLoading,
+                'error': this.onChildrenError
             });
 
             if (this.disabled) {
@@ -155,11 +155,42 @@ define([
          * @param {String} message - error message.
          */
         onChildrenError: function (message) {
-            var hasErrors = this.elems.some('error');
+            var hasErrors = false;
+
+            if (!message) {
+                hasErrors = this._isChildrenHasErrors(hasErrors, this);
+            }
 
             this.error(hasErrors || message);
         },
 
+        /**
+         * Returns errors of children if exist
+         *
+         * @param {Boolean} hasErrors
+         * @param {*} container
+         * @return {Boolean}
+         * @private
+         */
+        _isChildrenHasErrors: function (hasErrors, container) {
+            var self = this;
+
+            if (hasErrors === false && container.hasOwnProperty('elems')) {
+                hasErrors = container.elems.some('error');
+
+                if (hasErrors === false && container.hasOwnProperty('_elems')) {
+                    container._elems.forEach(function (child) {
+
+                        if (hasErrors === false) {
+                            hasErrors = self._isChildrenHasErrors(hasErrors, child);
+                        }
+                    });
+                }
+            }
+
+            return hasErrors;
+        },
+
         /**
          * Callback that sets loading property to true.
          */