Omer Frenkel has uploaded a new change for review. Change subject: core: fix jobProperties for delete vm and template from export (#818111) ......................................................................
core: fix jobProperties for delete vm and template from export (#818111) https://bugzilla.redhat.com/show_bug.cgi?id=818111 when removing vm/template from export domain, the job properties tried to use entities from db, which are not always available, and not correct to use. this fix implement override for getVm() and getVmTemplate() to get from export domain, and use it. most of the code was just moved from canDoAction also set the domain name which was not set, and fixed in ExecutionMessages.properties to the right var name Change-Id: I936501874e9a29b18bdddf1b65e7150e88cfe475 Signed-off-by: Omer Frenkel <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmFromImportExportCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmTemplateFromImportExportCommand.java M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 3 files changed, 103 insertions(+), 39 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/14/4714/1 -- To view, visit http://gerrit.ovirt.org/4714 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I936501874e9a29b18bdddf1b65e7150e88cfe475 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
