Change in ovirt-engine[master]: core, webadmin: Move VmValidationUtils

amureini at redhat.com amureini at redhat.com
Tue May 28 16:07:23 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core, webadmin: Move VmValidationUtils
......................................................................


core, webadmin: Move VmValidationUtils

This class has no business being in common - it's conceptually part of
the bll validation framework, and is indeed only used in bll.

Note:
As part of the change, its entry was removed from Common.gwt.xml (as
bll classes cannot be imported to GWT), with no adverse effect.
This only strengthens the point that this class was not supposed to be
in common.

Change-Id: I506dc65f8740cdc18d57007f46a75d7f89f169b1
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/VmValidationUtils.java
M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml
3 files changed, 2 insertions(+), 5 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I506dc65f8740cdc18d57007f46a75d7f89f169b1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list