Changeset 78994 in vbox

Timestamp:

Jun 5, 2019 1:09:44 PM (5 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9484: Export Appliance wizard: Move cloud export procedure onto own appliance object.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/wizards/exportappliance/UIWizardExportApp.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/wizards/exportappliance/UIWizardExportApp.cpp

@@ -56 +56 @@
 bool UIWizardExportApp::exportAppliance()
 {
-    /* Get export appliance widget & fetch all settings from the appliance editor: */
-    UIApplianceExportEditorWidget *pExportApplianceWidget = field("applianceWidget").value<ExportAppliancePointer>();
-    AssertPtrReturn(pExportApplianceWidget, false);
-    pExportApplianceWidget->prepareExport();
-
-    /* Acquire the appliance: */
-    CAppliance *pComAppliance = pExportApplianceWidget->appliance();
-    AssertPtrReturn(pComAppliance, false);
-
-    /* For Filesystem formats only: */
-    if (!field("isFormatCloudOne").toBool())
-    {
+    /* Check whether there was cloud target selected: */
+    const bool fIsFormatCloudOne = field("isFormatCloudOne").toBool();
+    if (fIsFormatCloudOne)
+    {
+        /* Get appliance: */
+        CAppliance comAppliance = field("appliance").value<CAppliance>();
+
+        /* Export the VMs, on success we are finished: */
+        return exportVMs(comAppliance);
+    }
+    else
+    {
+        /* Get export appliance widget & fetch all settings from the appliance editor: */
+        UIApplianceExportEditorWidget *pExportApplianceWidget = field("applianceWidget").value<ExportAppliancePointer>();
+        AssertPtrReturn(pExportApplianceWidget, false);
+        pExportApplianceWidget->prepareExport();
+
+        /* Acquire the appliance: */
+        CAppliance *pComAppliance = pExportApplianceWidget->appliance();
+        AssertPtrReturn(pComAppliance, false);
+
         /* We need to know every filename which will be created, so that we can ask the user for confirmation of overwriting.
          * For that we iterating over all virtual systems & fetch all descriptions of the type HardDiskImage. Also add the
@@ -140 +149 @@
                 return msgCenter().cannotCheckFiles(comExplorer, this);
         }
-    }
-
-    /* Export the VMs, on success we are finished: */
-    return exportVMs(*pComAppliance);
+
+        /* Export the VMs, on success we are finished: */
+        return exportVMs(*pComAppliance);
+    }
 }