Change in ovirt-engine[master]: core: cleanup Handler classes
ofrenkel at redhat.com
ofrenkel at redhat.com
Thu Jan 31 14:02:33 UTC 2013
Omer Frenkel has submitted this change and it was merged.
Change subject: core: cleanup Handler classes
......................................................................
core: cleanup Handler classes
Change the visibility of ObjectIdentityChecker instance in VmHandler,
VdsHander and VmTemplateHandler to private and replace direct access to
those fields from outside of those classes, with delegation methods calls
Change-Id: I9dbea95ff90f0fc638c2f109578e932bc964e8f9
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmTemplateHandler.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/VmHandlerTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/VmTemplateHandlerTest.java
7 files changed, 23 insertions(+), 11 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11536
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9dbea95ff90f0fc638c2f109578e932bc964e8f9
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
More information about the Engine-commits
mailing list