#1083 Correct handling of create/drop of databases and collections.

src/robomongo/core/domain/MongoDatabase.cpp

@@ -157,4 +157,19 @@ namespace Robomongo
     {
         _collections.push_back(collection);
     }
+
+    void MongoDatabase::handle(CreateCollectionResponse *event) {
+        genericResponseHandler(event, "Failed to create collection.");
+    }
+
+    void MongoDatabase::handle(DropCollectionResponse *event) {
+        genericResponseHandler(event, "Failed to drop collection.");
+    }
+
+    void MongoDatabase::genericResponseHandler(Event *event, const std::string &userFriendlyMessage) {
+        if (!event->isError())
+            return;
+
+        _bus->publish(new OperationFailedEvent(this, event->error().errorMessage(), userFriendlyMessage));
+    }
 }

src/robomongo/core/domain/MongoDatabase.h

@@ -83,10 +83,13 @@ namespace Robomongo
         void handle(LoadUsersResponse *event);
         void handle(LoadFunctionsResponse *event);
         void handle(CreateUserResponse *event);
+        void handle(CreateCollectionResponse *event);
+        void handle(DropCollectionResponse *event);
 
     private:
         void clearCollections();
         void addCollection(MongoCollection *collection);
+        void genericResponseHandler(Event *event, const std::string &userFriendlyMessage);
 
     private:
         MongoServer *_server;

src/robomongo/core/domain/MongoServer.cpp

@@ -194,4 +194,19 @@ namespace Robomongo {
             AppRegistry::instance().settingsManager()->mongoTimeoutSec(),
             AppRegistry::instance().settingsManager()->shellTimeoutSec());
     }
+
+    void MongoServer::handle(CreateDatabaseResponse *event) {
+        genericResponseHandler(event, "Failed to create database.");
+    }
+
+    void MongoServer::handle(DropDatabaseResponse *event) {
+        genericResponseHandler(event, "Failed to drop database.");
+    }
+
+    void MongoServer::genericResponseHandler(Event *event, const std::string &userFriendlyMessage) {
+        if (!event->isError())
+            return;
+
+        _bus->publish(new OperationFailedEvent(this, event->error().errorMessage(), userFriendlyMessage));
+    }
 }

src/robomongo/core/domain/MongoServer.h

@@ -8,11 +8,15 @@ namespace Robomongo
 {
     class MongoWorker;
     class MongoDatabase;
+    class EventBus;
+    class App;
+
+    // Messages
     class EstablishConnectionResponse;
     class LoadDatabaseNamesResponse;
     class InsertDocumentResponse;
-    class EventBus;
-    class App;
+    class CreateDatabaseResponse;
+    class DropDatabaseResponse;
 
     /**
      * @brief MongoServer represents active connection to MongoDB server.
@@ -71,10 +75,13 @@ namespace Robomongo
         void handle(LoadDatabaseNamesResponse *event);
         void handle(InsertDocumentResponse *event);
         void handle(RemoveDocumentResponse *event);
+        void handle(CreateDatabaseResponse *event);
+        void handle(DropDatabaseResponse *event);
 
     private:
         void clearDatabases();
         void addDatabase(MongoDatabase *database);
+        void genericResponseHandler(Event *event, const std::string &userFriendlyMessage);
 
         MongoWorker *_client;
         ConnectionSettings *_settings;

src/robomongo/core/events/MongoEvents.cpp

@@ -66,4 +66,5 @@ namespace Robomongo
     R_REGISTER_EVENT(ListenSshConnectionResponse)
     R_REGISTER_EVENT(LogEvent)
     R_REGISTER_EVENT(StopScriptRequest)
+    R_REGISTER_EVENT(OperationFailedEvent)
 }

src/robomongo/core/events/MongoEvents.h

@@ -1012,6 +1012,20 @@ namespace Robomongo
             Event(sender) { }
     };
 
+    class OperationFailedEvent : public Event
+    {
+        R_EVENT
+
+    public:
+        OperationFailedEvent(QObject *sender, const std::string &technicalErrorMessage, const std::string &userFriendlyErrorMessage) :
+            technicalErrorMessage(technicalErrorMessage),
+            userFriendlyErrorMessage(userFriendlyErrorMessage),
+            Event(sender) { }
+
+        std::string technicalErrorMessage;
+        std::string userFriendlyErrorMessage;
+    };
+
     class QueryWidgetUpdatedEvent : public Event
     {
         R_EVENT

src/robomongo/core/mongodb/MongoWorker.cpp

@@ -484,7 +484,7 @@ namespace Robomongo
     {
         try {
             boost::scoped_ptr<MongoClient> client(getClient());
-            client->createCollection(event->getNs().toString(), event->getSize(), event->getCapped(), 
+            client->createCollection(event->getNs().toString(), event->getSize(), event->getCapped(),
                 event->getMaxDocNum(), event->getExtraOptions());
             client->done();
 

src/robomongo/gui/MainWindow.cpp

@@ -504,6 +504,7 @@ namespace Robomongo
         AppRegistry::instance().bus()->subscribe(this, ScriptExecutedEvent::Type);
         AppRegistry::instance().bus()->subscribe(this, ScriptExecutingEvent::Type);
         AppRegistry::instance().bus()->subscribe(this, QueryWidgetUpdatedEvent::Type);
+        AppRegistry::instance().bus()->subscribe(this, OperationFailedEvent::Type);
     }
 
     void MainWindow::createStylesMenu()
@@ -921,6 +922,15 @@ namespace Robomongo
         _executeAction->setDisabled(false);
     }
 
+    void MainWindow::handle(OperationFailedEvent *event)
+    {
+        std::stringstream ss;
+        ss << event->userFriendlyErrorMessage << std::endl << std::endl
+            << "Error:" << std::endl << event->technicalErrorMessage;
+
+        QMessageBox::information(NULL, "Operation failed", QtUtils::toQString(ss.str()));
+    }
+
     void MainWindow::handle(QueryWidgetUpdatedEvent *event)
     {
         _orientationAction->setEnabled(event->numOfResults() > 1);

src/robomongo/gui/MainWindow.h

@@ -13,6 +13,8 @@ namespace Robomongo
     class ConnectionFailedEvent;
     class ScriptExecutingEvent;
     class ScriptExecutedEvent;
+    class OperationFailedEvent;
+
     class QueryWidgetUpdatedEvent;
     class WorkAreaTabWidget;
     class ConnectionMenu;
@@ -66,6 +68,7 @@ namespace Robomongo
         void handle(ScriptExecutingEvent *event);
         void handle(ScriptExecutedEvent *event);
         void handle(QueryWidgetUpdatedEvent *event);
+        void handle(OperationFailedEvent *event);
     private Q_SLOTS:
         void updateMenus();
         void setUtcTimeZone();

src/robomongo/gui/dialogs/CreateCollectionDialog.cpp

@@ -43,7 +43,7 @@ namespace Robomongo
     CreateCollectionDialog::CreateCollectionDialog(const QString &serverName, double dbVersion, const std::string& storageEngine, 
         const QString &database, const QString &collection, QWidget *parent) :
         QDialog(parent), _dbVersion(dbVersion), _storageEngine(storageEngine), 
-        _activeFrame(_storageEngineFrame), _activeObj(&_storageEngineObj)
+        _activeFrame(NULL), _activeObj(&_storageEngineObj)
     {
         setWindowTitle(tr("Create Collection"));
         setMinimumWidth(300);