Omer Frenkel has submitted this change and it was merged. Change subject: core: remove MigrateVmCommand#canMigrateVm ......................................................................
core: remove MigrateVmCommand#canMigrateVm All that MigrateVmCommand#canDoAction was doing was to delegate to canMigrateVm method and canMigrateVm method was not called from anywhere else. Thus, this patch removes the canMigrateVm method and put its body in the canDoAction method instead. InternalMigrateVmCommand is also changed accordingly. Change-Id: If88012b8bdbc7d23ad9350acf207431727e1d51e Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InternalMigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java 2 files changed, 11 insertions(+), 13 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18468 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If88012b8bdbc7d23ad9350acf207431727e1d51e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
