VirtualBox

Changeset 60645 in vbox


Ignore:
Timestamp:
Apr 22, 2016 12:00:42 PM (9 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
106812
Message:

FE/Qt: ​​​​​​​​​bugref:6451: Sort the formats using the ID order but put all the preferred formats first.

Location:
trunk/src/VBox/Frontends/VirtualBox/src/wizards
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VirtualBox/src/wizards/clonevd/UIWizardCloneVDPageBasic2.cpp

    r60637 r60645  
    104104            m_pFormatButtonGroup = new QButtonGroup(this);
    105105            {
    106                 CSystemProperties systemProperties = vboxGlobal().virtualBox().GetSystemProperties();
    107                 const QVector<CMediumFormat> &medFormats = systemProperties.GetMediumFormats();
    108                 for (int i = 0; i < medFormats.size(); ++i)
     106                /* Enumerate medium formats in special order: */
     107                CSystemProperties properties = vboxGlobal().virtualBox().GetSystemProperties();
     108                const QVector<CMediumFormat> &formats = properties.GetMediumFormats();
     109                QMap<QString, CMediumFormat> vdi, preferred;
     110                foreach (const CMediumFormat &format, formats)
    109111                {
    110                     const CMediumFormat &medFormat = medFormats[i];
    111                     if (medFormat.GetName() == "VDI")
    112                         addFormatButton(this, pFormatLayout, medFormat);
     112                    /* VDI goes first: */
     113                    if (format.GetName() == "VDI")
     114                        vdi[format.GetId()] = format;
     115                    else
     116                    {
     117                        const QVector<KMediumFormatCapabilities> &capabilities = format.GetCapabilities();
     118                        /* Then preferred: */
     119                        if (capabilities.contains(KMediumFormatCapabilities_Preferred))
     120                            preferred[format.GetId()] = format;
     121                    }
    113122                }
    114                 for (int i = 0; i < medFormats.size(); ++i)
    115                 {
    116                     const CMediumFormat &medFormat = medFormats[i];
    117                     const QVector<KMediumFormatCapabilities> &capabilities = medFormat.GetCapabilities();
    118                     if (medFormat.GetName() != "VDI" && capabilities.contains(KMediumFormatCapabilities_Preferred))
    119                         addFormatButton(this, pFormatLayout, medFormat);
    120                 }
     123
     124                /* Create buttons for VDI and preferred: */
     125                foreach (const QString &strId, vdi.keys())
     126                    addFormatButton(this, pFormatLayout, vdi.value(strId));
     127                foreach (const QString &strId, preferred.keys())
     128                    addFormatButton(this, pFormatLayout, preferred.value(strId));
     129
    121130                if (!m_pFormatButtonGroup->buttons().isEmpty())
    122131                {

  • trunk/src/VBox/Frontends/VirtualBox/src/wizards/clonevd/UIWizardCloneVDPageExpert.cpp

    r60637 r60645  
    9494                m_pFormatButtonGroup = new QButtonGroup(m_pFormatCnt);
    9595                {
    96                     CSystemProperties systemProperties = vboxGlobal().virtualBox().GetSystemProperties();
    97                     const QVector<CMediumFormat> &medFormats = systemProperties.GetMediumFormats();
    98                     for (int i = 0; i < medFormats.size(); ++i)
     96                    /* Enumerate medium formats in special order: */
     97                    CSystemProperties properties = vboxGlobal().virtualBox().GetSystemProperties();
     98                    const QVector<CMediumFormat> &formats = properties.GetMediumFormats();
     99                    QMap<QString, CMediumFormat> vdi, preferred, others;
     100                    foreach (const CMediumFormat &format, formats)
    99101                    {
    100                         const CMediumFormat &medFormat = medFormats[i];
    101                         if (medFormat.GetName() == "VDI")
    102                             addFormatButton(m_pFormatCnt, pFormatCntLayout, medFormat, true);
     102                        /* VDI goes first: */
     103                        if (format.GetName() == "VDI")
     104                            vdi[format.GetId()] = format;
     105                        else
     106                        {
     107                            const QVector<KMediumFormatCapabilities> &capabilities = format.GetCapabilities();
     108                            /* Then goes preferred: */
     109                            if (capabilities.contains(KMediumFormatCapabilities_Preferred))
     110                                preferred[format.GetId()] = format;
     111                            /* Then others: */
     112                            else
     113                                others[format.GetId()] = format;
     114                        }
    103115                    }
    104                     for (int i = 0; i < medFormats.size(); ++i)
    105                     {
    106                         const CMediumFormat &medFormat = medFormats[i];
    107                         const QVector<KMediumFormatCapabilities> &capabilities = medFormat.GetCapabilities();
    108                         if (medFormat.GetName() != "VDI")
    109                             addFormatButton(m_pFormatCnt, pFormatCntLayout, medFormat, capabilities.contains(KMediumFormatCapabilities_Preferred));
    110                     }
     116
     117                    /* Create buttons for VDI, preferred and others: */
     118                    foreach (const QString &strId, vdi.keys())
     119                        addFormatButton(this, pFormatCntLayout, vdi.value(strId), true);
     120                    foreach (const QString &strId, preferred.keys())
     121                        addFormatButton(this, pFormatCntLayout, preferred.value(strId), true);
     122                    foreach (const QString &strId, others.keys())
     123                        addFormatButton(this, pFormatCntLayout, others.value(strId));
     124
    111125                    if (!m_pFormatButtonGroup->buttons().isEmpty())
    112126                    {

  • trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvd/UIWizardNewVDPageBasic1.cpp

    r60637 r60645  
    104104            m_pFormatButtonGroup = new QButtonGroup(this);
    105105            {
    106                 CSystemProperties systemProperties = vboxGlobal().virtualBox().GetSystemProperties();
    107                 const QVector<CMediumFormat> &medFormats = systemProperties.GetMediumFormats();
    108                 for (int i = 0; i < medFormats.size(); ++i)
     106                /* Enumerate medium formats in special order: */
     107                CSystemProperties properties = vboxGlobal().virtualBox().GetSystemProperties();
     108                const QVector<CMediumFormat> &formats = properties.GetMediumFormats();
     109                QMap<QString, CMediumFormat> vdi, preferred;
     110                foreach (const CMediumFormat &format, formats)
    109111                {
    110                     const CMediumFormat &medFormat = medFormats[i];
    111                     if (medFormat.GetName() == "VDI")
    112                         addFormatButton(this, pFormatLayout, medFormat);
     112                    /* VDI goes first: */
     113                    if (format.GetName() == "VDI")
     114                        vdi[format.GetId()] = format;
     115                    else
     116                    {
     117                        const QVector<KMediumFormatCapabilities> &capabilities = format.GetCapabilities();
     118                        /* Then preferred: */
     119                        if (capabilities.contains(KMediumFormatCapabilities_Preferred))
     120                            preferred[format.GetId()] = format;
     121                    }
    113122                }
    114                 for (int i = 0; i < medFormats.size(); ++i)
    115                 {
    116                     const CMediumFormat &medFormat = medFormats[i];
    117                     const QVector<KMediumFormatCapabilities> &capabilities = medFormat.GetCapabilities();
    118                     if (medFormat.GetName() != "VDI" && capabilities.contains(KMediumFormatCapabilities_Preferred))
    119                         addFormatButton(this, pFormatLayout, medFormat);
    120                 }
     123
     124                /* Create buttons for VDI and preferred: */
     125                foreach (const QString &strId, vdi.keys())
     126                    addFormatButton(this, pFormatLayout, vdi.value(strId));
     127                foreach (const QString &strId, preferred.keys())
     128                    addFormatButton(this, pFormatLayout, preferred.value(strId));
     129
    121130                if (!m_pFormatButtonGroup->buttons().isEmpty())
    122131                {

  • trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvd/UIWizardNewVDPageExpert.cpp

    r60636 r60645  
    122122                m_pFormatButtonGroup = new QButtonGroup(m_pFormatCnt);
    123123                {
    124                     CSystemProperties systemProperties = vboxGlobal().virtualBox().GetSystemProperties();
    125                     const QVector<CMediumFormat> &medFormats = systemProperties.GetMediumFormats();
    126                     for (int i = 0; i < medFormats.size(); ++i)
     124                    /* Enumerate medium formats in special order: */
     125                    CSystemProperties properties = vboxGlobal().virtualBox().GetSystemProperties();
     126                    const QVector<CMediumFormat> &formats = properties.GetMediumFormats();
     127                    QMap<QString, CMediumFormat> vdi, preferred, others;
     128                    foreach (const CMediumFormat &format, formats)
    127129                    {
    128                         const CMediumFormat &medFormat = medFormats[i];
    129                         if (medFormat.GetName() == "VDI")
    130                             addFormatButton(m_pFormatCnt, pFormatCntLayout, medFormat, true);
     130                        /* VDI goes first: */
     131                        if (format.GetName() == "VDI")
     132                            vdi[format.GetId()] = format;
     133                        else
     134                        {
     135                            const QVector<KMediumFormatCapabilities> &capabilities = format.GetCapabilities();
     136                            /* Then goes preferred: */
     137                            if (capabilities.contains(KMediumFormatCapabilities_Preferred))
     138                                preferred[format.GetId()] = format;
     139                            /* Then others: */
     140                            else
     141                                others[format.GetId()] = format;
     142                        }
    131143                    }
    132                     for (int i = 0; i < medFormats.size(); ++i)
    133                     {
    134                         const CMediumFormat &medFormat = medFormats[i];
    135                         const QVector<KMediumFormatCapabilities> &capabilities = medFormat.GetCapabilities();
    136                         if (medFormat.GetName() != "VDI")
    137                             addFormatButton(m_pFormatCnt, pFormatCntLayout, medFormat, capabilities.contains(KMediumFormatCapabilities_Preferred));
    138                     }
     144
     145                    /* Create buttons for VDI, preferred and others: */
     146                    foreach (const QString &strId, vdi.keys())
     147                        addFormatButton(this, pFormatCntLayout, vdi.value(strId), true);
     148                    foreach (const QString &strId, preferred.keys())
     149                        addFormatButton(this, pFormatCntLayout, preferred.value(strId), true);
     150                    foreach (const QString &strId, others.keys())
     151                        addFormatButton(this, pFormatCntLayout, others.value(strId));
     152
    139153                    if (!m_pFormatButtonGroup->buttons().isEmpty())
    140154                    {
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette