Daniel Erez has posted comments on this change.

Change subject: webadmin: Add default description to multiple action result
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/FrontendEventsHandlerImpl.java
Line 65:             if (isRaiseErrorModalPanel(action, v.getFault()))
Line 66:             {
Line 67:                 for (String canDo : v.getCanDoActionMessages()) {
Line 68:                     String description =
Line 69:                             (v.getDescription() != null && 
!"".equals(v.getDescription().trim())) || returnValues.size() == 1 ? 
v.getDescription() : "Action " + actionNum; //$NON-NLS-1$ //$NON-NLS-2$
"Action" should be localized.
Line 70:                     Message msg = new Message(description, canDo);
Line 71:                     errors.add(msg);
Line 72:                 }
Line 73:             }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I25495eedf3ec61e11eb3c2bb075833fbcc8f9537
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to