Michael Kublin has submitted this change and it was merged.

Change subject: core: SessionDataContainer: added setUser method
......................................................................


core: SessionDataContainer: added setUser method

This patch introduces the setUser(session,user) and setUser(user)
methods, and refactors other pieces of code that privously used SetData
in order to set a user on the session to use these new methods.

This is done in order to stop exposing SessionDataContainer's internal
implementation to other components.

Change-Id: I7cf58ee93b45360cc823e541c0d3e200e17543ed
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginBaseCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/session/SessionDataContainer.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AbstractGetEntitiesWithPermittedActionParametersQueryTest.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/BackendTest.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommandBaseTest.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetUserBySessionIdQueryTest.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/session/SessionDataContainerTest.java
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/itests/AbstractBackendTest.java
8 files changed, 49 insertions(+), 16 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved
  Allon Mureinik: Verified; Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7cf58ee93b45360cc823e541c0d3e200e17543ed
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to