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