Arik Hadas has posted comments on this change.

Change subject: webadmin: fix import-vm from export domain sub-tab
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/40096/1/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/VmBackupModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/VmBackupModel.java:

Line 417:                     @Override
Line 418:                     public void executed(
Line 419:                             FrontendMultipleActionAsyncResult result) 
{
Line 420: 
Line 421:                         VmBackupModel vmBackupModel = 
VmBackupModel.this;
> you could just drop the explicit extraction of vmBackupModel and use the me
Done
Line 422:                         vmBackupModel.getWindow().stopProgress();
Line 423:                         vmBackupModel.cancel();
Line 424:                         ArrayList<VdcReturnValueBase> retVals =
Line 425:                                 (ArrayList<VdcReturnValueBase>) result


-- 
To view, visit https://gerrit.ovirt.org/40096
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I08656cf51d8a5c20639a67e90d0d742c3dcdd8e7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Martin Betak <mbe...@redhat.com>
Gerrit-Reviewer: Martin Beták <mato...@gmail.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to