Change in ovirt-engine[master]: webadmin: RunMultipleAction - error handling added

gchaplik at redhat.com gchaplik at redhat.com
Thu May 31 14:21:22 UTC 2012


Gilad Chaplik has submitted this change and it was merged.

Change subject: webadmin: RunMultipleAction - error handling added
......................................................................


webadmin: RunMultipleAction - error handling added

Error handling code of RunMultipleAction is modified to
display more than one error message.

Both CanDoAction messages and Execution failed(Fault)
messages are combined and shown in the error popup dialog.

Change-Id: I8d9ab1c30b7352af029cc283ca98b6370388bb0e
Signed-off-by: Kanagaraj M <kmayilsa at redhat.com>
---
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/IFrontendEventsHandler.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/ErrorMessageFormatter.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/FrontendEventsHandlerImpl.java
M frontend/webadmin/modules/userportal/src/main/java/org/ovirt/engine/ui/userportal/client/uicommonext/FrontendEventsHandlerImpl.java
5 files changed, 91 insertions(+), 63 deletions(-)

Approvals:
  Gilad Chaplik: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/4878
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d9ab1c30b7352af029cc283ca98b6370388bb0e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>



More information about the Engine-commits mailing list