Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Showing the fault error in the multipleActions error dialog ......................................................................
webadmin: Showing the fault error in the multipleActions error dialog In most of the commands the canDoActionMessages contain the '$action' and the '$type' replacements, no matter if the canDo passed or not. Since FrontendEventsHandlerImpl.runMultipleActionsFailed checked the canDoActionMessages size in order to decide wherer to show the canDo errors or the fault, the fault was rarely (if ever) displayed. Change-Id: I91819e7d67bbd76ec7c183608e0d0abbd70908ee Signed-off-by: Alona Kaplan <alkap...@redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/FrontendEventsHandlerImpl.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexander Wels: Looks good to me, approved Alona Kaplan: Verified Daniel Erez: Looks good to me, approved Lior Vernia: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/37724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91819e7d67bbd76ec7c183608e0d0abbd70908ee Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com> Gerrit-Reviewer: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
