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 <ahadas@redhat.com> --- 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 <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server