some more message types supported to make this work more smoothly with freenode.net

4 files changed, 49 insertions, 2 deletions

diff --git a/noncore/net/opieirc/ircmessageparser.cpp b/noncore/net/opieirc/ircmessageparser.cpp
index 5c70753..d1b70a5 100644
--- a/noncore/net/opieirc/ircmessageparser.cpp
+++ b/noncore/net/opieirc/ircmessageparser.cpp
@@ -16,4 +16,5 @@ IRCLiteralMessageParserStruct IRCMessageParser::literalParserProcTable[] = {
     { "MODE",    FUNC(parseLiteralMode) },
     { "KICK",    FUNC(parseLiteralKick) },
+    { "TOPIC",   FUNC(parseLiteralTopic) },
     { 0 , 0 }
 };
@@ -37,4 +38,6 @@ IRCNumericalMessageParserStruct IRCMessageParser::numericalParserProcTable[] = {
     { 254, FUNC(nullFunc)},                     // RPL_LUSERCHANNELS
     { 255, FUNC(parseNumericalStats) },         // RPL_LUSERNAME
+    { 332, FUNC(parseNumericalTopic) },         // RPL_TOPIC
+    { 333, FUNC(parseNumericalTopicWhoTime) },  // RPL_TOPICWHOTIME
     { 353, FUNC(parseNumericalNames) },         // RPL_NAMREPLY
     { 366, FUNC(parseNumericalEndOfNames) },    // RPL_ENDOFNAMES
@@ -192,5 +195,5 @@ void IRCMessageParser::parseLiteralPrivMsg(IRCMessage *message) {
             }
         } else {
-            emit outputReady(IRCOutput(OUTPUT_ERROR, tr("Channel message with unknown channel")));
+            emit outputReady(IRCOutput(OUTPUT_ERROR, tr("Channel message with unknown channel ") + message->param(0)));
         }
     } else {
@@ -243,4 +246,16 @@ void IRCMessageParser::parseLiteralQuit(IRCMessage *message) {
 }
 
+void IRCMessageParser::parseLiteralTopic(IRCMessage *message) {
+    IRCPerson mask(message->prefix());
+    IRCChannel *channel = m_session->getChannel(message->param(0));
+    if (channel) {
+        IRCOutput output(OUTPUT_TOPIC, mask.nick() + tr(" changed topic to ") + "\"" + message->param(1) + "\"");
+        output.addParam(channel);
+        emit outputReady(output);
+    } else {
+        emit outputReady(IRCOutput(OUTPUT_ERROR, tr("Unknown channel topic - desynchronized?")));
+    }
+}
+
 void IRCMessageParser::parseLiteralError(IRCMessage *message) {
     emit outputReady(IRCOutput(OUTPUT_ERROR, message->allParameters()));
@@ -485,2 +500,18 @@ void IRCMessageParser::parseNumericalNoSuchNick(IRCMessage *) {
     emit outputReady(IRCOutput(OUTPUT_ERROR, tr("No such nickname")));
 }
+
+void IRCMessageParser::parseNumericalTopic(IRCMessage *message) {
+    IRCChannel *channel = m_session->getChannel(message->param(1));
+    if (channel) {
+        IRCOutput output(OUTPUT_TOPIC, tr("Topic for channel " + channel->channelname() + " is \"" + message->param(2) + "\""));
+        output.addParam(channel);
+        emit outputReady(output);
+    } else {
+        IRCOutput output(OUTPUT_TOPIC, tr("Topic for channel " + message->param(1) + " is \"" + message->param(2) + "\""));
+        output.addParam(0);
+        emit outputReady(output);
+    }
+}
+
+void IRCMessageParser::parseNumericalTopicWhoTime(IRCMessage *message) {
+}
diff --git a/noncore/net/opieirc/ircmessageparser.h b/noncore/net/opieirc/ircmessageparser.h
index f774047..c4dd96c 100644
--- a/noncore/net/opieirc/ircmessageparser.h
+++ b/noncore/net/opieirc/ircmessageparser.h
@@ -73,4 +73,5 @@ private:
     void parseLiteralMode(IRCMessage *message);
     void parseLiteralKick(IRCMessage *message);
+    void parseLiteralTopic(IRCMessage *message);
     void parseNumerical001(IRCMessage *message);
     void parseNumerical002(IRCMessage *message);
@@ -83,4 +84,6 @@ private:
     void parseNumericalNicknameInUse(IRCMessage *message);
     void parseNumericalNoSuchNick(IRCMessage *message);
+    void parseNumericalTopic(IRCMessage *message);
+    void parseNumericalTopicWhoTime(IRCMessage *message);
     void parseCTCPPing(IRCMessage *message);
     void parseCTCPVersion(IRCMessage *message);
diff --git a/noncore/net/opieirc/ircoutput.h b/noncore/net/opieirc/ircoutput.h
index e8cc524..9c0b8bb 100644
--- a/noncore/net/opieirc/ircoutput.h
+++ b/noncore/net/opieirc/ircoutput.h
@@ -45,5 +45,6 @@ enum IRCOutputType {
     OUTPUT_CHANACTION     = 14,  /* parameters : channel (IRCChannel) person (IRCChannelPerson) */
     OUTPUT_QUERYACTION    = 15,  /* parameters : person (IRCPerson) */
-    OUTPUT_CHANPERSONMODE = 16   /* parameters : channel (IRCCHannel) person (IRCChannelPerson) */
+    OUTPUT_CHANPERSONMODE = 16,  /* parameters : channel (IRCCHannel) person (IRCChannelPerson) */
+    OUTPUT_TOPIC          = 17   /* parameters : channel (IRCChannel) */
 };
 
diff --git a/noncore/net/opieirc/ircservertab.cpp b/noncore/net/opieirc/ircservertab.cpp
index 4be60ef..aea58a3 100644
--- a/noncore/net/opieirc/ircservertab.cpp
+++ b/noncore/net/opieirc/ircservertab.cpp
@@ -220,4 +220,16 @@ void IRCServerTab::display(IRCOutput output) {
             }
             break;
+        case OUTPUT_TOPIC: {
+                IRCChannel *channel = (IRCChannel *) output.getParam(0);
+                if (channel) {
+                    IRCChannelTab *channelTab = getTabForChannel(channel);
+                    if (channelTab) {
+                        channelTab->appendText("<font color=\"" + m_notificationColor + "\">"+output.htmlMessage()+"</font><br>");
+                        return;
+                    }
+                }
+                appendText("<font color=\"" + m_notificationColor + "\">"+output.htmlMessage()+"</font><br>");
+            }
+            break;
         case OUTPUT_QUIT: {
                 QString nick = ((IRCPerson *)output.getParam(0))->nick();