Unhide buffer when joining or querying and it was temporarily hidden. fixes #866

[quassel.git] / src / client / clientuserinputhandler.cpp

diff --git a/src/client/clientuserinputhandler.cpp b/src/client/clientuserinputhandler.cpp
index 8abc79d..aa614f2 100644 (file)
--- a/src/client/clientuserinputhandler.cpp
+++ b/src/client/clientuserinputhandler.cpp
@@ -29,6 +29,9 @@
 #include "network.h"
 #include "types.h"
 #include "bufferinfo.h"
+#include "clientbufferviewconfig.h"
+#include "clientbufferviewmanager.h"
+#include "messagemodel.h"
 
 #include <QDateTime>
 
@@ -81,12 +84,20 @@ void ClientUserInputHandler::handleExec(const BufferInfo &bufferInfo, const QStr
 }
 
 void ClientUserInputHandler::handleJoin(const BufferInfo &bufferInfo, const QString &text) {
+  if(text.isEmpty()) {
+    Client::messageModel()->insertErrorMessage(bufferInfo, tr("/JOIN expects a channel"));
+    return;
+  }
   switchBuffer(bufferInfo.networkId(), text.section(' ', 0, 0));
   // send to core
   defaultHandler("JOIN", bufferInfo, text);
 }
 
 void ClientUserInputHandler::handleQuery(const BufferInfo &bufferInfo, const QString &text) {
+  if(text.isEmpty()) {
+    Client::messageModel()->insertErrorMessage(bufferInfo, tr("/QUERY expects at least a nick"));
+    return;
+  }
   switchBuffer(bufferInfo.networkId(), text.section(' ', 0, 0));
   // send to core
   defaultHandler("QUERY", bufferInfo, text);
@@ -99,5 +110,18 @@ void ClientUserInputHandler::switchBuffer(const NetworkId &networkId, const QStr
   }
   else {
     Client::bufferModel()->switchToBuffer(newBufId);
+    // unhide the buffer
+    ClientBufferViewManager *clientBufferViewManager = Client::bufferViewManager();
+    QList<ClientBufferViewConfig*> bufferViewConfigList = clientBufferViewManager->clientBufferViewConfigs();
+    foreach (ClientBufferViewConfig *bufferViewConfig, bufferViewConfigList) {
+      if (bufferViewConfig->temporarilyRemovedBuffers().contains(newBufId)) {
+        bufferViewConfig->addBuffer(newBufId, bufferViewConfig->bufferList().length());
+        //if (bufferViewConfig->sortAlphabetically()) {
+          // TODO we need to trigger a sort here, but can't reach the model required
+          // to get a bufferviewfilter, as the bufferviewmanager only managers configs
+          //BufferViewFilter *filter = qobject_cast<BufferViewFilter *>(model());
+        //}
+      }
+    }
   }
 }