Changeset 4324 in vbox for trunk/src/VBox/Frontends/VirtualBox/ui

Timestamp:

Aug 23, 2007 7:01:31 PM (18 years ago)

Author:

vboxsync

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

23856

Message:

Main: Improved serial port API (mostly spelling);
FE/Qt: Improved serial port UI (spelling and duplicate handling).

Location:

trunk/src/VBox/Frontends/VirtualBox/ui

Files:

• VBoxVMNetworkSettings.ui.h (modified) (1 diff)
• VBoxVMSerialPortSettings.ui (added)
• VBoxVMSerialPortSettings.ui.h (added)
• VBoxVMSettingsDlg.ui (modified) (7 diffs)
• VBoxVMSettingsDlg.ui.h (modified) (19 diffs)

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxVMNetworkSettings.ui.h

@@ -26 +26 @@
 *****************************************************************************/
 
-/**
- *  VBoxVMNetworkSettings class to use as network interface setup page.
- */
 void VBoxVMNetworkSettings::init()
 {

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxVMSettingsDlg.ui

@@ -270 +270 @@
         <item>
             <property name="text">
-                <string> Serial Port </string>
+                <string> Serial Ports </string>
             </property>
             <property name="text">
@@ -276 +276 @@
             </property>
             <property name="text">
-                <string>#serial</string>
+                <string>#serialPorts</string>
             </property>
             <property name="pixmap">
-                <pixmap>refresh_16px.png</pixmap>
+                <pixmap></pixmap>
             </property>
             <property name="pixmap">
@@ -2712 +2712 @@
                     <number>9</number>
                 </attribute>
+                <vbox>
+                    <property name="name">
+                        <cstring>unnamed</cstring>
+                    </property>
+                    <property name="margin">
+                        <number>0</number>
+                    </property>
+                    <widget class="QTabWidget">
+                        <property name="name">
+                            <cstring>tbwSerialPorts</cstring>
+                        </property>
+                        <property name="sizePolicy">
+                            <sizepolicy>
+                                <hsizetype>1</hsizetype>
+                                <vsizetype>1</vsizetype>
+                                <horstretch>0</horstretch>
+                                <verstretch>0</verstretch>
+                            </sizepolicy>
+                        </property>
+                    </widget>
+                </vbox>
             </widget>
         </widget>
@@ -3055 +3076 @@
     <include location="local" impldecl="in declaration">VBoxGlobal.h</include>
     <include location="local" impldecl="in implementation">VBoxVMNetworkSettings.h</include>
+    <include location="local" impldecl="in implementation">VBoxVMSerialPortSettings.h</include>
     <include location="local" impldecl="in implementation">VBoxUSBFilterSettings.h</include>
     <include location="local" impldecl="in implementation">VBoxSharedFoldersSettings.h</include>
@@ -3066 +3088 @@
     <forward>class VBoxUSBMenu</forward>
     <forward>class VBoxSharedFoldersSettings</forward>
-    <forward>class VBoxVMSerialPortSettings</forward>
     <forward>class QIRichLabel</forward>
     <forward>class BootItemsList</forward>
@@ -3092 +3113 @@
     <variable access="private">bool mUSBFilterListModified;</variable>
     <variable access="private">VBoxSharedFoldersSettings *mSharedFolders;</variable>
-    <variable access="private">VBoxVMSerialPortSettings *mSerialPorts;</variable>
     <variable access="private">QString warningString;</variable>
     <variable access="private">VBoxMediaComboBox *cbHDA;</variable>
@@ -3116 +3136 @@
     <slot>showImageManagerISOFloppy()</slot>
     <slot>showVDImageManager( QUuid *id, VBoxMediaComboBox *le, QLabel *tx = NULL )</slot>
-    <slot>addNetworkAdapter( const CNetworkAdapter &amp; adapter )</slot>
+    <slot>addNetworkAdapter( const CNetworkAdapter &amp; )</slot>
+    <slot>addSerialPort( const CSerialPort &amp; )</slot>
     <slot>slRAM_valueChanged( int val )</slot>
     <slot>leRAM_textChanged( const QString &amp; text )</slot>

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxVMSettingsDlg.ui.h

@@ -470 +470 @@
 
 
-class VBoxVMSerialPortSettings : public QWidget
-{
-    Q_OBJECT
-
-public:
-
-    VBoxVMSerialPortSettings (QWidget *aParent)
-        : QWidget (aParent, "VBoxVMSerialPortSettings")
-        , mTabWidget (0)
-    {
-        /* Basic initialization */
-        mTabWidget = new QTabWidget (this, "mTabWidget");
-        mTabWidget->setMargin (11);
-        mTabWidget->setSizePolicy (QSizePolicy::Expanding, QSizePolicy::Preferred);
-        QVBoxLayout *mainLayout = new QVBoxLayout (this);
-        mainLayout->addWidget (mTabWidget);
-        mainLayout->addItem (new QSpacerItem (0, 0, QSizePolicy::Preferred,
-                                                    QSizePolicy::Expanding));
-
-        /* Create a list of known COM interfaces configurations */
-        mConfigList << PortConfig (0x3f8, 4, 1);
-        mConfigList << PortConfig (0x2f8, 3, 2);
-        mConfigList << PortConfig (0x3e8, 4, 3);
-        mConfigList << PortConfig (0x2e8, 3, 4);
-    }
-
-    void getFromMachine (const CMachine &aMachine)
-    {
-        mMachine = aMachine;
-
-        ulong count = vboxGlobal().virtualBox().GetSystemProperties().GetSerialPortCount();
-        for (ulong slot = 0; slot < count; ++ slot)
-        {
-            CSerialPort port = mMachine.GetSerialPort (slot);
-            addPage (port, slot);
-        }
-
-        /* Revalidate all pages */
-        portConfigUpdate();
-    }
-
-    void putBackToMachine()
-    {
-        for (int slot = 0; slot < mTabWidget->count(); ++ slot)
-        {
-            CSerialPort port = mMachine.GetSerialPort (slot);
-
-            port.SetEnabled (static_cast<QGroupBox*> (mTabWidget->page (slot))->isChecked());
-
-            PortConfig config = parsePortConfig (mPortList.at (slot)->currentText());
-            port.SetIOBase (config.io);
-            port.SetIRQ (config.irq);
-
-            port.SetHostMode (vboxGlobal().toSerialHostMode (mModeList.at (slot)->currentText()));
-
-            QCheckBox *chbIsServer = mIsServerList.at (slot);
-            port.SetServer (chbIsServer->isEnabled() ?
-                            chbIsServer->isChecked() : false);
-
-            QLineEdit *lePath = mPathList.at (slot);
-            port.SetPath (lePath->isEnabled() ?
-                          lePath->text() : QString::null);
-        }
-    }
-
-private slots:
-
-    void portConfigUpdate()
-    {
-        for (int index = 0; index < mTabWidget->count(); ++ index)
-        {
-            QComboBox *cbPortConfig = mPortList.at (index);
-            QString oldText = cbPortConfig->currentText();
-            QStringList newList = portConfigList (index);
-            cbPortConfig->clear();
-            cbPortConfig->insertStringList (newList);
-            cbPortConfig->setCurrentText (oldText);
-        }
-    }
-
-    void hostModeUpdate (int aIndex = -1)
-    {
-        if (aIndex == -1)
-            aIndex = mTabWidget->currentPageIndex();
-        QCheckBox *chbIsServer = mIsServerList.at (aIndex);
-        QLineEdit *lePath = mPathList.at (aIndex);
-        CEnums::SerialHostMode mode = vboxGlobal().toSerialHostMode (
-            mModeList.at (aIndex)->currentText());
-        chbIsServer->setEnabled (mode == CEnums::HostPipe);
-        lePath->setEnabled (mode != CEnums::Disconnected);
-    }
-
-private:
-
-    struct PortConfig
-    {
-        PortConfig ()
-            : io (0), irq (0), number (0) {}
-        PortConfig (ulong aIO, ulong aIRQ, ulong aNumber)
-            : io (aIO), irq (aIRQ), number (aNumber) {}
-
-        bool isNull() { return !io && !irq; }
-
-        bool operator== (const PortConfig &aCfg) const
-        {
-            return io  == aCfg.io && irq == aCfg.irq;
-        }
-
-        static QString format()
-        {
-            return QString ("I/O Port = 0x%1, IRQ = %2");
-        }
-        QString toString()
-        {
-            QString info = format().arg (QString::number (io, 16)).arg (irq);
-            if (number)
-                info += QString (" (COM%1)").arg (number);
-            return info;
-        }
-
-        ulong io;
-        ulong irq;
-        ulong number;
-    };
-    typedef QValueList<PortConfig> PortConfigs;
-
-    void addPage (const CSerialPort &aPort, ulong aSlot)
-    {
-        /* Create Serial Port page */
-        QGroupBox *gBox = new QGroupBox (tr ("&Enable Serial Port"));
-        gBox->setColumnLayout (0, Qt::Vertical);
-        gBox->layout()->setMargin (11);
-        gBox->layout()->setSpacing (6);
-        gBox->setCheckable (true);
-
-        QLabel *lbPort = new QLabel (tr ("Serial Port &Number"), gBox);
-        QComboBox *cbPort = new QComboBox (gBox);
-        lbPort->setBuddy (cbPort);
-        connect (cbPort, SIGNAL (activated (int)),
-                 this , SLOT (portConfigUpdate()));
-
-        QLabel *lbType = new QLabel (tr ("Serial Port Host &Mode"), gBox);
-        QComboBox *cbMode = new QComboBox (gBox);
-        lbType->setBuddy (cbMode);
-        cbMode->insertItem (vboxGlobal().toString (CEnums::Disconnected));
-        cbMode->insertItem (vboxGlobal().toString (CEnums::HostPipe));
-        cbMode->insertItem (vboxGlobal().toString (CEnums::HostDevice));
-        connect (cbMode, SIGNAL (activated (int)),
-                 this , SLOT (hostModeUpdate()));
-
-        QCheckBox *chbIsServer = new QCheckBox (tr ("Create Pipe &Automatically"), gBox);
-
-        QLabel *lbPath = new QLabel (tr ("Serial Port &Path"), gBox);
-        QLineEdit *lePath = new QLineEdit (gBox);
-        lbPath->setBuddy (lePath);
-
-        QGridLayout *pageLayout = new QGridLayout (gBox->layout());
-        pageLayout->setAlignment (Qt::AlignTop);
-        pageLayout->addWidget (lbPort, 0, 0);
-        pageLayout->addMultiCellWidget (cbPort, 0, 0, 1, 2);
-        pageLayout->addWidget (lbType, 1, 0);
-        pageLayout->addWidget (cbMode, 1, 1);
-        pageLayout->addWidget (chbIsServer, 1, 2);
-        pageLayout->addWidget (lbPath, 2, 0);
-        pageLayout->addMultiCellWidget (lePath, 2, 2, 1, 2);
-
-        /* Load machine information */
-        gBox->setChecked (aPort.GetEnabled());
-
-        PortConfig config (aPort.GetIOBase(), aPort.GetIRQ(), 0);
-        if (!mConfigList.contains (config))
-            mConfigList << config;
-        cbPort->insertStringList (portConfigList (mTabWidget->count()));
-        for (int id = 0; id < cbPort->count(); ++ id)
-            if (parsePortConfig (cbPort->text (id)) == config)
-                cbPort->setCurrentItem (id);
-
-        cbMode->setCurrentText (vboxGlobal().toString (aPort.GetHostMode()));
-
-        chbIsServer->setChecked (aPort.GetServer());
-
-        lePath->setText (aPort.GetPath());
-
-        /* Append newly createrd widget */
-        mTabWidget->addTab (gBox, tr ("Port &%1").arg (aSlot));
-        mPortList.append (cbPort);
-        mModeList.append (cbMode);
-        mIsServerList.append (chbIsServer);
-        mPathList.append (lePath);
-
-        /* Revalidate page */
-        hostModeUpdate (aSlot);
-    }
-
-    QStringList portConfigList (int aIndex)
-    {
-        QStringList list;
-        PortConfigs config = portConfigs (aIndex);
-        for (ulong id = 0; id < config.count(); ++ id)
-            list << config [id].toString();
-        return list;
-    }
-
-    PortConfigs portConfigs (int aIndex)
-    {
-        PortConfigs config (mConfigList);
-        if (aIndex == mTabWidget->count())
-            return config;
-
-        for (int index = 0; index < mTabWidget->count(); ++ index)
-        {
-            if (index != aIndex)
-            {
-                PortConfig used = parsePortConfig (mPortList.at (index)->currentText());
-                Assert (!used.isNull());
-                config.remove (used);
-            }
-        }
-        return config;
-    }
-
-    PortConfig parsePortConfig (const QString &aString)
-    {
-        QRegExp regExp (PortConfig::format().arg ("([0-9a-f]+)").arg ("(\\d+)"));
-        if (regExp.search (aString) != -1)
-            return PortConfig (regExp.cap (1).toULong(0, 16),
-                               regExp.cap (2).toULong(), 0);
-        Assert (0);
-        return PortConfig();
-    }
-
-    QTabWidget *mTabWidget;
-    PortConfigs mConfigList;
-
-    QPtrList<QComboBox> mPortList;
-    QPtrList<QComboBox> mModeList;
-    QPtrList<QCheckBox> mIsServerList;
-    QPtrList<QLineEdit> mPathList;
-
-    CMachine mMachine;
-};
-
-
 void VBoxVMSettingsDlg::init()
 {
@@ -957 +714 @@
     QToolTip::add (pbHostRemove, tr ("Remove"));
 
+    /* Serial Port Page */
+
     /* USB Page */
 
@@ -1021 +780 @@
     mSharedFolders->setDialogType (VBoxSharedFoldersSettings::MachineType);
     pageFoldersLayout->addWidget (mSharedFolders);
-
-    /* Serial Port Page */
-
-    QVBoxLayout* pageSerialLayout = new QVBoxLayout (pageSerial, 0, 0);
-    mSerialPorts = new VBoxVMSerialPortSettings (pageSerial);
-    pageSerialLayout->addWidget (mSerialPorts);
 
     /*
@@ -1549 +1302 @@
     Q_UNUSED (wval);
 
+    /* reset the warning text; interested parties will set it during
+     * validation */
+    setWarning (QString::null);
+
     /* detect the overall validity */
     bool newValid = true;
@@ -1567 +1324 @@
         valid = newValid;
         buttonOk->setEnabled (valid);
-        if (valid)
-            setWarning(0);
-        warningLabel->setHidden(valid);
-        warningPixmap->setHidden(valid);
-    }
-}
-
-
-void VBoxVMSettingsDlg::revalidate( QIWidgetValidator *wval )
+        warningLabel->setHidden (valid);
+        warningPixmap->setHidden (valid);
+    }
+}
+
+
+void VBoxVMSettingsDlg::revalidate (QIWidgetValidator *wval)
 {
     /* do individual validations for pages */
@@ -1581 +1336 @@
     bool valid = wval->isOtherValid();
 
+    QString warningText;
+    QString pageTitle = ::path (listView->currentItem());
+
     if (pg == pageHDD)
     {
@@ -1593 +1351 @@
             {
                 valid = false;
-                setWarning (tr ("Primary Master hard disk is not selected."));
+                warningText = tr ("Primary Master hard disk is not selected");
             }
             else uuids << uuidHDA;
@@ -1603 +1361 @@
             {
                 valid = false;
-                setWarning (tr ("Primary Slave hard disk is not selected."));
+                warningText = tr ("Primary Slave hard disk is not selected");
             }
             else
@@ -1616 +1374 @@
                     uuids << uuidHDB;
                 else
-                    setWarning (tr ("Primary Slave hard disk is already attached "
-                                    "to a different slot."));
+                    warningText = tr ("Primary Slave hard disk is already attached "
+                                      "to a different slot");
             }
         }
@@ -1626 +1384 @@
             {
                 valid = false;
-                setWarning (tr ("Secondary Slave hard disk is not selected."));
+                warningText = tr ("Secondary Slave hard disk is not selected");
             }
             else
@@ -1639 +1397 @@
                     uuids << uuidHDB;
                 else
-                    setWarning (tr ("Secondary Slave hard disk is already attached "
-                                    "to a different slot."));
+                    warningText = tr ("Secondary Slave hard disk is already attached "
+                                      "to a different slot");
             }
         }
@@ -1667 +1425 @@
 
         if (!valid)
-            setWarning (tr ("CD/DVD image file is not selected."));
+            warningText = tr ("CD/DVD image file is not selected");
     }
     else if (pg == pageFloppy)
@@ -1684 +1442 @@
 
         if (!valid)
-            setWarning (tr ("Floppy image file is not selected."));
+            warningText = tr ("Floppy image file is not selected");
     }
     else if (pg == pageNetwork)
     {
+        QWidget *tab = NULL;
+        for (int index = 0; index < tbwNetwork->count(); ++ index)
+        {
+            QWidget *tab = tbwNetwork->page (index);
+            VBoxVMNetworkSettings *page =
+                static_cast <VBoxVMNetworkSettings *> (tab);
+            valid = page->isPageValid (mInterfaceList);
+            if (!valid) break;
+        }
+        if (!valid)
+        {
+            Assert (tab);
+            warningText = tr ("Incorrect host network interface is selected");
+            pageTitle += ": " + tbwNetwork->tabLabel (tab);
+        }
+    }
+    else if (pg == pageSerial)
+    {
+        QValueList <QString> ports;
+        QValueList <QString> paths;
+
         int index = 0;
-        for (; index < tbwNetwork->count(); ++index)
-        {
-            QWidget *tab = tbwNetwork->page (index);
-            VBoxVMNetworkSettings *set = static_cast<VBoxVMNetworkSettings*> (tab);
-            valid = set->isPageValid (mInterfaceList);
-            if (!valid) break;
-        }
-        if (!valid)
-            setWarning (tr ("Incorrect host network interface is selected "
-                            "for Adapter %1.").arg (index));
+        for (; index < tbwSerialPorts->count(); ++ index)
+        {
+            QWidget *tab = tbwSerialPorts->page (index);
+            VBoxVMSerialPortSettings *page =
+                static_cast <VBoxVMSerialPortSettings *> (tab);
+
+            /* skip disabled ports */
+            if (!page->mSerialPortBox->isChecked())
+                continue;
+            /* check the predefined port number unicity */
+            if (!page->isUserDefined())
+            {
+                QString port = page->mPortNumCombo->currentText();
+                valid = !ports.contains (port);
+                if (!valid)
+                {
+                    warningText = tr ("Duplicate port number is selected ");
+                    pageTitle += ": " + tbwSerialPorts->tabLabel (tab);
+                    break;
+                }
+                ports << port;
+            }
+            /* check the port path unicity */
+            CEnums::PortMode mode =
+                vboxGlobal().toPortMode (page->mHostModeCombo->currentText());
+            if (mode != CEnums::DisconnectedPort)
+            {
+                QString path = page->mPortPathLine->text();
+                valid = !paths.contains (path);
+                if (!valid)
+                {
+                    warningText = tr ("Duplicate port path is entered ");
+                    pageTitle += ": " + tbwSerialPorts->tabLabel (tab);
+                    break;
+                }
+                paths << path;
+            }
+        }
     }
     else if (pg == pageVRDP)
@@ -1707 +1514 @@
                     (leVRDPPort->text().isEmpty() || leVRDPTimeout->text().isEmpty()));
             if (!valid && leVRDPPort->text().isEmpty())
-                setWarning (tr ("VRDP Port is not set."));
+                warningText = tr ("VRDP Port is not set ");
             if (!valid && leVRDPTimeout->text().isEmpty())
-                setWarning (tr ("VRDP Timeout is not set."));
+                warningText = tr ("VRDP Timeout is not set ");
         }
         else
             valid = true;
     }
+
+    if (!valid)
+        setWarning (tr ("%1 on the <b>%2</b> page.")
+                    .arg (warningText, pageTitle));
 
     wval->setOtherValid (valid);
@@ -1980 +1791 @@
     }
 
+    /* serial ports */
+    {
+        ulong count = vbox.GetSystemProperties().GetSerialPortCount();
+        for (ulong slot = 0; slot < count; ++ slot)
+        {
+            CSerialPort port = machine.GetSerialPort (slot);
+            addSerialPort (port);
+        }
+    }
+
     /* USB */
     {
@@ -2049 +1870 @@
     {
         mSharedFolders->getFromMachine (machine);
-    }
-
-    /* serial ports */
-    {
-        mSerialPorts->getFromMachine (machine);
     }
 
@@ -2254 +2070 @@
     }
 
+    /* serial ports */
+    {
+        for (int index = 0; index < tbwSerialPorts->count(); index++)
+        {
+            VBoxVMSerialPortSettings *page =
+                (VBoxVMSerialPortSettings *) tbwSerialPorts->page (index);
+            Assert (page);
+            page->putBackToPort();
+        }
+    }
+
     /* usb */
     {
@@ -2316 +2143 @@
     }
 
-    /* serial ports */
-    {
-        mSerialPorts->putBackToMachine();
-    }
-
     return COMResult();
 }
@@ -2400 +2222 @@
 
 #endif
+}
+
+void VBoxVMSettingsDlg::addSerialPort (const CSerialPort &aPort)
+{
+    VBoxVMSerialPortSettings *page = new VBoxVMSerialPortSettings();
+    page->getFromPort (aPort);
+    tbwSerialPorts->addTab (page, QString (tr ("Port %1", "serial ports"))
+                                               .arg (aPort.GetSlot()));
+
+    /* fix the tab order so that main dialog's buttons are always the last */
+    setTabOrder (page->mPortPathLine, buttonHelp);
+    setTabOrder (buttonHelp, buttonOk);
+    setTabOrder (buttonOk, buttonCancel);
+
+    /* setup validation */
+    QIWidgetValidator *wval = new QIWidgetValidator (pageSerial, this);
+    connect (page->mSerialPortBox, SIGNAL (toggled (bool)),
+             wval, SLOT (revalidate()));
+    connect (page->mIRQLine, SIGNAL (textChanged (const QString &)),
+             wval, SLOT (revalidate()));
+    connect (page->mIOPortLine, SIGNAL (textChanged (const QString &)),
+             wval, SLOT (revalidate()));
+    connect (page->mHostModeCombo, SIGNAL (activated (const QString &)),
+             wval, SLOT (revalidate()));
+    connect (wval, SIGNAL (validityChanged (const QIWidgetValidator *)),
+             this, SLOT (enableOk (const QIWidgetValidator *)));
+    connect (wval, SIGNAL (isValidRequested (QIWidgetValidator *)),
+             this, SLOT (revalidate (QIWidgetValidator *)));
+
+    wval->revalidate();
 }