Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Login Commands - add admin flag to VdcUser
......................................................................


core: Login Commands - add admin flag to VdcUser

The login base command will now update the isAdmin property of the
VdcUser, so it can be shared on the seeion instead of calling
MultiLevelAdministrationHandler each time it's needed (e.g., in user
queries).

LoginAdminUserCommand is left in order to fail the canDoAction if the
user is not an admin, but the actual MLA checking is not done there
anymore.

Change-Id: I73d23480d9d47a199dabeaadc3979e4559d463fa
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginAdminUserCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginBaseCommand.java
2 files changed, 10 insertions(+), 3 deletions(-)

Approvals:
  Allon Mureinik: Verified
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I73d23480d9d47a199dabeaadc3979e4559d463fa
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to