Display error message when adding an existing configuration.

Change-Id: I44ce937ab9bab65c3680c11304e1ba5a6c366e1e
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com>

src/plugins/vcprojectmanager/widgets/configurationseditwidget.cpp

@@ -28,6 +28,7 @@
 **
 ****************************************************************************/
 #include "configurationseditwidget.h"
+
 #include "configurationswidget.h"
 #include "../interfaces/iattributecontainer.h"
 #include "../interfaces/iconfiguration.h"
@@ -48,6 +49,7 @@
 
 #include <utils/qtcassert.h>
 
+#include <QMessageBox>
 #include <QVBoxLayout>
 
 namespace VcProjectManager {
@@ -115,6 +117,8 @@ void ConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString cop
     IPlatforms *platforms = m_vsProject->platforms();
     QString copyFromConfigName = copyFrom.split(QLatin1Char('|')).at(0);
 
+    QStringList errorMessages;
+
     for (int i = 0; i < platforms->platformCount(); ++i) {
         IPlatform *platform = platforms->platform(i);
 
@@ -122,6 +126,12 @@ void ConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString cop
             QString newFullConfigName = newConfigName + QLatin1Char('|') + platform->displayName();
             QString copyFromFullConfigName;
 
+            // if configuration with the same name already exists show error message
+            if (m_vsProject->configurations()->configurationContainer()->configuration(newFullConfigName)) {
+                QString message = tr("Configuration %1 already exists.");
+                message = message.arg(newFullConfigName);
+                errorMessages.append(message);
+            } else {
                 if (!copyFromConfigName.isEmpty())
                     copyFromFullConfigName = copyFromConfigName + QLatin1Char('|') + platform->displayName();
 
@@ -131,6 +141,22 @@ void ConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString cop
         }
     }
 
+    if (errorMessages.size()) {
+        QString message = errorMessages.join(QLatin1Char('\n'));
+        QMessageBox msg(this);
+        msg.setStandardButtons(QMessageBox::Ok);
+        msg.setDefaultButton(QMessageBox::Ok);
+
+        if (errorMessages.size() == 1)
+            msg.setWindowTitle(tr("An Error Occurred While Adding a Configuration"));
+        else
+            msg.setWindowTitle(tr("Cannot Add Configurations"));
+        msg.setText(message);
+
+        msg.exec();
+    }
+}
+
 void ConfigurationsEditWidget::onNewConfigAdded(IConfiguration *config)
 {
     QTC_ASSERT(config, return);
@@ -264,10 +290,13 @@ void ConfigurationsEditWidget::addConfigToFiles(const QString &newConfigName, co
         it.next();
         ConfigurationContainer *container = it.value();
 
-        if (container) {
+        if (!container)
+            continue;
         IConfiguration *config = container->configuration(copyFrom);
 
-            if (config) {
+        if (!config)
+            continue;
+
         IConfiguration *newConfig = config->clone();
         if (newConfig) {
             newConfig->setFullName(newConfigName);
@@ -275,8 +304,6 @@ void ConfigurationsEditWidget::addConfigToFiles(const QString &newConfigName, co
         }
     }
 }
-    }
-}
 
 
 } // Internal

src/plugins/vcprojectmanager/widgets/fileconfigurationseditwidget.cpp

@@ -30,6 +30,7 @@
 #include "configurationbasewidget.h"
 #include "configurationswidget.h"
 #include "fileconfigurationseditwidget.h"
+
 #include "../interfaces/iattributecontainer.h"
 #include "../interfaces/iattributedescriptiondataitem.h"
 #include "../interfaces/iconfiguration.h"
@@ -54,6 +55,7 @@
 
 #include <utils/qtcassert.h>
 
+#include <QMessageBox>
 #include <QVBoxLayout>
 
 namespace VcProjectManager {
@@ -124,6 +126,8 @@ void FileConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString
     IPlatforms *platforms = m_vsProject->platforms();
     QString copyFromConfigName = copyFrom.split(QLatin1Char('|')).at(0);
 
+    QStringList errorMessages;
+
     for (int i = 0; i < platforms->platformCount(); ++i) {
         IPlatform *platform = platforms->platform(i);
 
@@ -131,6 +135,12 @@ void FileConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString
             QString newFullConfigName = newConfigName + QLatin1Char('|') + platform->displayName();
             QString copyFromFullConfigName;
 
+            if (m_vsProject->configurations()->configurationContainer()->configuration(newFullConfigName)) {
+                QString message = tr("Configuration %1 already exists.");
+                message = message.arg(newFullConfigName);
+                errorMessages.append(message);
+            }
+            else {
                 if (!copyFromConfigName.isEmpty())
                     copyFromFullConfigName = copyFromConfigName + QLatin1Char('|') + platform->displayName();
 
@@ -140,6 +150,22 @@ void FileConfigurationsEditWidget::onAddNewConfig(QString newConfigName, QString
         }
     }
 
+    if (errorMessages.size()) {
+        QString message = errorMessages.join(QLatin1Char('\n'));
+        QMessageBox msg(this);
+        msg.setStandardButtons(QMessageBox::Ok);
+        msg.setDefaultButton(QMessageBox::Ok);
+
+        if (errorMessages.size() == 1)
+            msg.setWindowTitle(tr("Cannot Add Configuration"));
+        else
+            msg.setWindowTitle(tr("Cannot Add Configurations"));
+        msg.setText(message);
+
+        msg.exec();
+    }
+}
+
 void FileConfigurationsEditWidget::onNewConfigAdded(IConfiguration *config)
 {
     QTC_ASSERT(config, return);