Change in ovirt-engine[master]: core: fixing remove user messages
oourfali at redhat.com
oourfali at redhat.com
Thu Jun 28 10:40:53 UTC 2012
Oved Ourfali has submitted this change and it was merged.
Change subject: core: fixing remove user messages
......................................................................
core: fixing remove user messages
This patch does some fixes in "cannot removed logged in user" message:
1. Add a space after the "." in the middle of the message
2. Add user type, so that it would appear in the message
3. Override setActionMessageParameters to get the correct action and
type
Change-Id: I479251005125d5285cd27af8c2c0a66f5f2322bc
Signed-off-by: Oved Ourfali <oourfali at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveUserCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
6 files changed, 17 insertions(+), 4 deletions(-)
Approvals:
Omer Frenkel: Looks good to me, approved
Oved Ourfali: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/5766
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I479251005125d5285cd27af8c2c0a66f5f2322bc
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
More information about the Engine-commits
mailing list