From 4a13ccf26db5844f9d9a6993514a0fc34ce1da7a Mon Sep 17 00:00:00 2001 From: Ravi Chandra Date: Sat, 15 Dec 2018 16:42:41 +0530 Subject: [PATCH] Fixed The ui-component field validation error not opening accordion tab that owns the field (field does not get focused) --- .../Magento/Ui/view/base/web/js/form/components/fieldset.js | 4 ++++ app/code/Magento/Ui/view/base/web/js/form/element/abstract.js | 1 + 2 files changed, 5 insertions(+) diff --git a/app/code/Magento/Ui/view/base/web/js/form/components/fieldset.js b/app/code/Magento/Ui/view/base/web/js/form/components/fieldset.js index 6d33386fa1f1c..5f2fda830f5ba 100644 --- a/app/code/Magento/Ui/view/base/web/js/form/components/fieldset.js +++ b/app/code/Magento/Ui/view/base/web/js/form/components/fieldset.js @@ -162,6 +162,10 @@ define([ } this.error(hasErrors || message); + + if (hasErrors || message) { + this.open(); + } }, /** diff --git a/app/code/Magento/Ui/view/base/web/js/form/element/abstract.js b/app/code/Magento/Ui/view/base/web/js/form/element/abstract.js index 013910bbd2e96..5617110590e50 100755 --- a/app/code/Magento/Ui/view/base/web/js/form/element/abstract.js +++ b/app/code/Magento/Ui/view/base/web/js/form/element/abstract.js @@ -408,6 +408,7 @@ define([ isValid = this.disabled() || !this.visible() || result.passed; this.error(message); + this.error.valueHasMutated(); this.bubble('error', message); //TODO: Implement proper result propagation for form