Changeset 72177 in vbox

Timestamp:

May 9, 2018 3:57:09 PM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

122600

Message:

FE/Qt: bugref: 6769 Adding a path/name selector widget to our new vm wizards.

Location:

trunk/src/VBox/Frontends/VirtualBox/src

Files:

• settings/machine/UIMachineSettingsGeneral.cpp (modified) (1 diff)
• widgets/UINameAndSystemEditor.cpp (modified) (8 diffs)
• widgets/UINameAndSystemEditor.h (modified) (5 diffs)
• widgets/UIVMNamePathSelector.cpp (modified) (7 diffs)
• widgets/UIVMNamePathSelector.h (modified) (5 diffs)
• wizards/newvm/UIWizardNewVM.cpp (modified) (2 diffs)
• wizards/newvm/UIWizardNewVMPageBasic1.cpp (modified) (5 diffs)
• wizards/newvm/UIWizardNewVMPageBasic1.h (modified) (4 diffs)
• wizards/newvm/UIWizardNewVMPageExpert.cpp (modified) (2 diffs)
• wizards/newvm/UIWizardNewVMPageExpert.h (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/settings/machine/UIMachineSettingsGeneral.cpp

@@ -554 +554 @@
         {
             /* Configure widget: */
-            m_pNameAndSystemEditor->nameEditor()->setValidator(new QRegExpValidator(QRegExp(".+"), this));
+            m_pNameAndSystemEditor->setNameFieldValidator(".+");
         }
     }

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UINameAndSystemEditor.cpp

@@ -31 +31 @@
 # include "UIFilePathSelector.h"
 # include "UINameAndSystemEditor.h"
+# include "UIVMNamePathSelector.h"
 
 /* COM includes: */
@@ -50 +51 @@
     , m_fSupportsHWVirtEx(false)
     , m_fSupportsLongMode(false)
-    , m_pLabelName(0)
     , m_pLabelFamily(0)
     , m_pLabelType(0)
     , m_pIconType(0)
-    , m_pEditorName(0)
-    , m_pEditorLocation(0)
+    , m_pNamePathLabel(0)
+    , m_pNamePathSelector(0)
     , m_pComboFamily(0)
     , m_pComboType(0)
@@ -65 +65 @@
 QString UINameAndSystemEditor::name() const
 {
-    if (!m_fChooseLocation)
-        return m_pEditorName->text();
-    else
-        return m_pEditorLocation->path();
+    if (!m_pNamePathSelector)
+        return QString();
+    return m_pNamePathSelector->name();
+}
+
+QString UINameAndSystemEditor::path() const
+{
+    if (!m_pNamePathSelector)
+        return vboxGlobal().virtualBox().GetSystemProperties().GetDefaultMachineFolder();
+    return m_pNamePathSelector->path();
 }
 
 void UINameAndSystemEditor::setName(const QString &strName)
 {
-    if (!m_fChooseLocation)
-        m_pEditorName->setText(strName);
-    else
-        m_pEditorLocation->setPath(strName);
+    if (!m_pNamePathSelector)
+        return;
+    m_pNamePathSelector->setName(strName);
 }
 
@@ -112 +117 @@
 void UINameAndSystemEditor::retranslateUi()
 {
-    m_pLabelName->setText(tr("N&ame:"));
     m_pLabelFamily->setText(tr("&Type:"));
     m_pLabelType->setText(tr("&Version:"));
-    if (!m_fChooseLocation)
-        m_pEditorName->setWhatsThis(tr("Holds the name of the virtual machine."));
-    else
-        m_pEditorLocation->setWhatsThis(tr("Holds the location of the virtual machine."));
+    m_pNamePathLabel->setText(tr("Path/Name:"));
+
     m_pComboFamily->setWhatsThis(tr("Selects the operating system family that "
                                     "you plan to install into this virtual machine."));
@@ -227 +229 @@
         pMainLayout->setContentsMargins(0, 0, 0, 0);
 
-        /* Create VM name label: */
-        m_pLabelName = new QLabel;
-        if (m_pLabelName)
-        {
-            /* Configure VM name label: */
-            m_pLabelName->setAlignment(Qt::AlignRight);
-            m_pLabelName->setSizePolicy(QSizePolicy::Minimum, QSizePolicy::Fixed);
-            /* Add VM name label into main-layout: */
-            pMainLayout->addWidget(m_pLabelName, 0, 0);
-        }
-
-        if (!m_fChooseLocation)
-        {
-            /* Create VM name editor: */
-            m_pEditorName = new QLineEdit;
-            if (m_pEditorName)
-            {
-                /* Configure VM name editor: */
-                m_pEditorName->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
-                m_pLabelName->setBuddy(m_pEditorName);
-                /* Add VM name editor into main-layout: */
-                pMainLayout->addWidget(m_pEditorName, 0, 1, 1, 2);
-            }
-        }
-        else
-        {
-            /* Create VM location editor: */
-            m_pEditorLocation = new UIFilePathSelector;
-            if (m_pEditorLocation)
-            {
-                /* Configure advanced VM name editor: */
-                m_pEditorLocation->setResetEnabled(false);
-                m_pEditorLocation->setMode(UIFilePathSelector::Mode_File_Save);
-                m_pEditorLocation->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
-                m_pEditorLocation->setHomeDir(vboxGlobal().virtualBox().GetSystemProperties().GetDefaultMachineFolder());
-                m_pLabelName->setBuddy(m_pEditorLocation);
-                /* Add advanced VM name editor into main-layout: */
-                pMainLayout->addWidget(m_pEditorLocation, 0, 1, 1, 2);
-            }
+        m_pNamePathLabel = new QLabel;
+        if (m_pNamePathLabel)
+        {
+            m_pNamePathLabel->setAlignment(Qt::AlignRight);
+            m_pNamePathLabel->setSizePolicy(QSizePolicy::Minimum, QSizePolicy::Fixed);
+            pMainLayout->addWidget(m_pNamePathLabel, 0, 0, 1, 1);
+        }
+        m_pNamePathSelector = new UIVMNamePathSelector;
+        if (m_pNamePathSelector)
+        {
+            m_pNamePathSelector->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
+            pMainLayout->addWidget(m_pNamePathSelector, 0, 1, 1, 2);
         }
 
@@ -325 +300 @@
                 pLayoutIcon->addWidget(m_pIconType);
             }
-
             /* Add stretch to sub-layout: */
             pLayoutIcon->addStretch();
@@ -358 +332 @@
 {
     /* Prepare connections: */
-    if (!m_fChooseLocation)
-        connect(m_pEditorName, &QLineEdit::textChanged,
-                this, &UINameAndSystemEditor::sigNameChanged);
-    else
-        connect(m_pEditorLocation, &UIFilePathSelector::pathChanged,
-                this, &UINameAndSystemEditor::sigNameChanged);
+    connect(m_pNamePathSelector, &UIVMNamePathSelector::sigNameChanged,
+            this, &UINameAndSystemEditor::sigNameChanged);
     connect(m_pComboFamily, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged),
             this, &UINameAndSystemEditor::sltFamilyChanged);
@@ -369 +339 @@
             this, &UINameAndSystemEditor::sltTypeChanged);
 }
+
+void UINameAndSystemEditor::setNameFieldValidator(const QString &strValidatorString)
+{
+    if (!m_pNamePathSelector)
+        return;
+    m_pNamePathSelector->setNameFieldValidator(strValidatorString);
+}

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UINameAndSystemEditor.h

@@ -36 +36 @@
 class QString;
 class UIFilePathSelector;
+class UIVMNamePathSelector;
 
 /** QWidget subclass providing complex editor for basic VM parameters. */
@@ -58 +59 @@
     UINameAndSystemEditor(QWidget *pParent, bool fChooseLocation = false);
 
-    /** Returns the VM name editor. */
-    QLineEdit *nameEditor() const { return m_pEditorName; }
-    /** Returns the VM location editor. */
-    UIFilePathSelector *locationEditor() const { return m_pEditorLocation; }
-
     /** Returns the VM name. */
     QString name() const;
+    /** Returns path string selected by the user. */
+    QString path() const;
+
     /** Defines the VM @a strName. */
     void setName(const QString &strName);
@@ -72 +71 @@
     /** Defines the VM OS @a enmType. */
     void setType(const CGuestOSType &enmType);
+
+    void setNameFieldValidator(const QString &strValidatorString);
 
 protected:
@@ -113 +114 @@
     bool                    m_fSupportsLongMode;
 
-    /** Holds the VM name label instance. */
-    QLabel                 *m_pLabelName;
     /** Holds the VM OS family label instance. */
     QLabel                 *m_pLabelFamily;
@@ -121 +120 @@
     /** Holds the VM OS type icon instance. */
     QLabel                 *m_pIconType;
-    /** Holds the VM name editor instance. */
-    QLineEdit              *m_pEditorName;
-    /** Holds the VM location editor instance. */
-    UIFilePathSelector     *m_pEditorLocation;
+
+    QLabel                 *m_pNamePathLabel;
+    UIVMNamePathSelector   *m_pNamePathSelector;
     /** Holds the VM OS family combo instance. */
     QComboBox              *m_pComboFamily;

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UIVMNamePathSelector.cpp

@@ -23 +23 @@
 # include <QDir>
 # include <QHBoxLayout>
+# include <QRegExpValidator>
 
 /* GUI includes: */
+# include "QIFileDialog.h"
 # include "QILineEdit.h"
 # include "QILabel.h"
+# include "QIToolButton.h"
+# include "UIIconPool.h"
 # include "UIVMNamePathSelector.h"
 # include "VBoxGlobal.h"
@@ -35 +39 @@
 #endif /* !VBOX_WITH_PRECOMPILED_HEADERS */
 
-
-
 UIVMNamePathSelector::UIVMNamePathSelector(QWidget *pParent /* = 0 */)
     : QIWithRetranslateUI<QWidget>(pParent)
@@ -43 +45 @@
     , m_pName(0)
     , m_pSeparator(0)
+    ,m_pFileDialogButton(0)
 {
     prepareWidgets();
+    retranslateUi();
 }
 
@@ -52 +56 @@
     CVirtualBox vbox = vboxGlobal().virtualBox();
     /* Get default machine folder: */
-    /*const QString strDefaultMachineFolder = */return vbox.GetSystemProperties().GetDefaultMachineFolder();
+    return vbox.GetSystemProperties().GetDefaultMachineFolder();
 
 }
@@ -65 +69 @@
     m_pMainLayout->setSpacing(0);
     setLayout(m_pMainLayout);
+
+    m_pFileDialogButton = new QIToolButton(this);
+    if (m_pFileDialogButton)
+    {
+        m_pMainLayout->addWidget(m_pFileDialogButton);
+        m_pFileDialogButton->setIcon(UIIconPool::iconSet(QString(":/sf_add_16px.png")));
+        connect(m_pFileDialogButton, &QIToolButton::clicked, this, &UIVMNamePathSelector::sltOpenPathSelector);
+    }
 
     m_pPath = new QILineEdit;
@@ -84 +96 @@
     {
         m_pMainLayout->addWidget(m_pName);
+        connect(m_pName, &QILineEdit::textChanged,
+                this, &UIVMNamePathSelector::sigNameChanged);
+
     }
 
@@ -121 +136 @@
 void UIVMNamePathSelector::retranslateUi()
 {
+    setToolTip(tr("The Virtual Machine files will be saved under ..."));
 }
+
+void UIVMNamePathSelector::sltOpenPathSelector()
+{
+
+    QString strSelectedPath = QIFileDialog::getExistingDirectory(m_pPath->text(), this,
+                                                                 QString("Select a parent folder for new Virtual Machine"));
+    if (!strSelectedPath.isEmpty())
+    {
+        m_pPath->setText(strSelectedPath);
+    }
+}
+
+void UIVMNamePathSelector::setNameFieldValidator(const QString &strValidatorString)
+{
+    if (!m_pName)
+        return;
+    m_pName->setValidator(new QRegExpValidator(QRegExp(strValidatorString), this));
+}

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UIVMNamePathSelector.h

@@ -29 +29 @@
 class QILabel;
 class QILineEdit;
-
+class QIToolButton;
 
 class SHARED_LIBRARY_STUFF UIVMNamePathSelector : public QIWithRetranslateUI<QWidget>
@@ -36 +36 @@
 
 signals:
+
+    void sigNameChanged(const QString &strName);
 
 public:
@@ -48 +50 @@
     QString name() const;
     void    setName(const QString &name);
+    void    setNameFieldValidator(const QString &strValidatorString);
 
 protected:
@@ -55 +58 @@
 private slots:
 
+    void sltOpenPathSelector();
 
 private:
@@ -61 +65 @@
     QString      defaultMachineFolder() const;
 
-    QHBoxLayout *m_pMainLayout;
-    QILineEdit  *m_pPath;
-    QILineEdit  *m_pName;
-    QILabel     *m_pSeparator;
+    QHBoxLayout  *m_pMainLayout;
+    QILineEdit   *m_pPath;
+    QILineEdit   *m_pName;
+    QILabel      *m_pSeparator;
+    QIToolButton *m_pFileDialogButton;
+
 };
 

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVM.cpp

@@ -106 +106 @@
         if (!m_strGroup.isEmpty())
             groups << m_strGroup;
-        m_machine = vbox.CreateMachine(QString() /* no file-path for now */,
+        m_machine = vbox.CreateMachine(field("machineFilePath").toString(),
                                        field("name").toString(),
                                        groups, strTypeId, QString());
@@ -397 +397 @@
     return strControllerName;
 }
-

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVMPageBasic1.cpp

@@ -217 +217 @@
 bool UIWizardNewVMPage1::createMachineFolder()
 {
+    if (!m_pNameAndSystemEditor)
+        return false;
     /* Cleanup previosly created folder if any: */
     if (machineFolderCreated() && !cleanupMachineFolder())
@@ -227 +229 @@
     CVirtualBox vbox = vboxGlobal().virtualBox();
     /* Get default machine folder: */
-    const QString strDefaultMachineFolder = vbox.GetSystemProperties().GetDefaultMachineFolder();
+    //const QString strMachineFolder = vbox.GetSystemProperties().GetDefaultMachineFolder();
+
     /* Compose machine filename: */
-    const QString strMachineFilePath = vbox.ComposeMachineFilename(m_pNameAndSystemEditor->name(),
-                                                                   m_strGroup,
-                                                                   QString(),
-                                                                   strDefaultMachineFolder);
+    m_strMachineFilePath = vbox.ComposeMachineFilename(m_pNameAndSystemEditor->name(),
+                                                       m_strGroup,
+                                                       QString(),
+                                                       m_pNameAndSystemEditor->path());
     /* Compose machine folder/basename: */
-    const QFileInfo fileInfo(strMachineFilePath);
-    const QString strMachineFolder = fileInfo.absolutePath();
-    const QString strMachineBaseName = fileInfo.completeBaseName();
+    const QFileInfo fileInfo(m_strMachineFilePath);
+    m_strMachineFolder = fileInfo.absolutePath();
+    m_strMachineBaseName = fileInfo.completeBaseName();
 
     /* Make sure that folder doesn't exists: */
-    if (QDir(strMachineFolder).exists())
+    if (QDir(m_strMachineFolder).exists())
     {
-        msgCenter().cannotRewriteMachineFolder(strMachineFolder, thisImp());
+        msgCenter().cannotRewriteMachineFolder(m_strMachineFolder, thisImp());
         return false;
     }
 
     /* Try to create new folder (and it's predecessors): */
-    bool fMachineFolderCreated = QDir().mkpath(strMachineFolder);
+    bool fMachineFolderCreated = QDir().mkpath(m_strMachineFolder);
     if (!fMachineFolderCreated)
     {
-        msgCenter().cannotCreateMachineFolder(strMachineFolder, thisImp());
+        msgCenter().cannotCreateMachineFolder(m_strMachineFolder, thisImp());
         return false;
     }
-
-    /* Initialize fields: */
-    m_strMachineFolder = strMachineFolder;
-    m_strMachineBaseName = strMachineBaseName;
     return true;
 }
@@ -293 +292 @@
     registerField("name*", m_pNameAndSystemEditor, "name", SIGNAL(sigNameChanged(const QString &)));
     registerField("type", m_pNameAndSystemEditor, "type", SIGNAL(sigOsTypeChanged()));
+    registerField("machineFilePath", this, "machineFilePath");
     registerField("machineFolder", this, "machineFolder");
     registerField("machineBaseName", this, "machineBaseName");
@@ -315 +315 @@
 
     /* Translate widgets: */
-    m_pLabel->setText(UIWizardNewVM::tr("Please choose a descriptive name for the new virtual machine "
+    m_pLabel->setText(UIWizardNewVM::tr("Please choose a descriptive name and destination folder for the new virtual machine "
                                         "and select the type of operating system you intend to install on it. "
                                         "The name you choose will be used throughout VirtualBox "
@@ -340 +340 @@
     return createMachineFolder();
 }
-

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVMPageBasic1.h

@@ -43 +43 @@
     bool cleanupMachineFolder();
 
+    QString machineFilePath() const { return m_strMachineFilePath; }
+    void setMachineFilePath(const QString &strMachineFilePath) { m_strMachineFilePath = strMachineFilePath; }
+
     /** Returns the machine folder value. */
     QString machineFolder() const { return m_strMachineFolder; }
@@ -53 +56 @@
     void setMachineBaseName(const QString &strMachineBaseName) { m_strMachineBaseName = strMachineBaseName; }
 
-    /** Holds the machine folder value. */
+    /** Full path (including the file name) of the machine's configuration file. */
+    QString m_strMachineFilePath;
+    /** Path of the folder hosting the machine's configuration file. Generated from m_strMachineFilePath. */
     QString m_strMachineFolder;
-    /** Holds the machine base-name value. */
+    /** Base name of the machine is generated from the m_strMachineFilePath. */
     QString m_strMachineBaseName;
 
-    /* Widgets: */
+    /** Provides a path selector and a line edit field for path and name entry. */
     UINameAndSystemEditor *m_pNameAndSystemEditor;
 
-    /* Variables: */
+    /** Variables: */
     QString m_strGroup;
     bool m_fSupportsHWVirtEx;
@@ -71 +76 @@
 {
     Q_OBJECT;
+    Q_PROPERTY(QString machineFilePath READ machineFilePath WRITE setMachineFilePath);
     Q_PROPERTY(QString machineFolder READ machineFolder WRITE setMachineFolder);
     Q_PROPERTY(QString machineBaseName READ machineBaseName WRITE setMachineBaseName);
@@ -107 +113 @@
 
 #endif // __UIWizardNewVMPageBasic1_h__
-

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVMPageExpert.cpp

@@ -160 +160 @@
     registerField("name*", m_pNameAndSystemEditor, "name", SIGNAL(sigNameChanged(const QString &)));
     registerField("type", m_pNameAndSystemEditor, "type", SIGNAL(sigOsTypeChanged()));
+    registerField("machineFilePath", this, "machineFilePath");
     registerField("machineFolder", this, "machineFolder");
     registerField("machineBaseName", this, "machineBaseName");
@@ -304 +305 @@
     return fResult;
 }
-

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVMPageExpert.h

@@ -36 +36 @@
     Q_PROPERTY(QString machineFolder READ machineFolder WRITE setMachineFolder);
     Q_PROPERTY(QString machineBaseName READ machineBaseName WRITE setMachineBaseName);
+    Q_PROPERTY(QString machineFilePath READ machineFilePath WRITE setMachineFilePath);
     Q_PROPERTY(CMedium virtualDisk READ virtualDisk WRITE setVirtualDisk);
     Q_PROPERTY(QString virtualDiskId READ virtualDiskId WRITE setVirtualDiskId);
@@ -84 +85 @@
 
 #endif // __UIWizardNewVMPageExpert_h__
-