Changeset 75218 in vbox

Timestamp:

Nov 1, 2018 6:36:01 PM (6 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9152: Export appliance wizard: Page 3: Check for Main API errors.

Location:

trunk/src/VBox/Frontends/VirtualBox/src

Files:

• globals/UIMessageCenter.cpp (modified) (2 diffs)
• globals/UIMessageCenter.h (modified) (1 diff)
• wizards/exportappliance/UIWizardExportAppPageBasic3.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/globals/UIMessageCenter.cpp

@@ -57 +57 @@
 
 /* COM includes: */
+# include "CCloudClient.h"
 # include "CCloudProfile.h"
 # include "CCloudProvider.h"
@@ -1683 +1684 @@
 }
 
+void UIMessageCenter::cannotCreateCloudClient(const CCloudProfile &comProfile, QWidget *pParent /* = 0 */) const
+{
+    error(pParent, MessageType_Error,
+          tr("Failed to create cloud client."),
+          UIErrorString::formatErrorInfo(comProfile));
+}
+
+void UIMessageCenter::cannotAcquireCloudClientParameter(const CCloudClient &comClient, QWidget *pParent /* = 0 */) const
+{
+    error(pParent, MessageType_Error,
+          tr("Failed to acquire cloud client parameter."),
+          UIErrorString::formatErrorInfo(comClient));
+}
+
 bool UIMessageCenter::confirmCloudProfileRemoval(const QString &strName, QWidget *pParent /* = 0 */) const
 {

trunk/src/VBox/Frontends/VirtualBox/src/globals/UIMessageCenter.h

@@ -371 +371 @@
     void cannotAcquireCloudProfileParameter(const CCloudProfile &comProfile, QWidget *pParent = 0) const;
     void cannotAssignCloudProfileParameter(const CCloudProfile &comProfile, QWidget *pParent = 0) const;
+    void cannotCreateCloudClient(const CCloudProfile &comProfile, QWidget *pParent = 0) const;
+    void cannotAcquireCloudClientParameter(const CCloudClient &comClient, QWidget *pParent = 0) const;
     bool confirmCloudProfileRemoval(const QString &strName, QWidget *pParent = 0) const;
     bool confirmCloudProfilesImport(QWidget *pParent = 0) const;

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

@@ -64 +64 @@
     /* Create Cloud Client: */
     CCloudClient comCloudClient = comCloudProfile.CreateCloudClient();
-    AssertMsgReturnVoid(comCloudProfile.isOk() && comCloudClient.isNotNull(),
-                        ("Can't create Cloud Client object!"));
-
-    /* Read Cloud Client parameters for Export VM operation: */
-    QString strJSON = comCloudClient.GetExportParameters();
-
-    /* Create JSON document on the basis of it, make sure it isn't empty: */
-    const QJsonDocument document = QJsonDocument::fromJson(strJSON.toUtf8());
-    AssertMsgReturnVoid(!document.isEmpty(), ("JSON document is empty!"));
-
-    /* Parse JSON document: */
-    m_listCloudClientParameters = parseJsonDocument(document);
+    /* Show error message if necessary: */
+    if (!comCloudProfile.isOk())
+        msgCenter().cannotCreateCloudClient(comCloudProfile);
+    else
+    {
+        /* Read Cloud Client parameters for Export VM operation: */
+        const QString strJSON = comCloudClient.GetExportParameters();
+        /* Show error message if necessary: */
+        if (!comCloudClient.isOk())
+            msgCenter().cannotAcquireCloudClientParameter(comCloudClient);
+        else
+        {
+            /* Create JSON document and parse it: */
+            const QJsonDocument document = QJsonDocument::fromJson(strJSON.toUtf8());
+            if (!document.isEmpty())
+                m_listCloudClientParameters = parseJsonDocument(document);
+        }
+    }
 }