Windows support. Tested and working to compile on freebsd and linux. Next step is...

[user/henk/code/inspircd.git] / src / cmd_topic.cpp

diff --git a/src/cmd_topic.cpp b/src/cmd_topic.cpp
index b9782143403feeb87a6b36e744d5bc4d73899d3a..06036fe010cf0d298356f90c7d8809e65007555e 100644 (file)
--- a/src/cmd_topic.cpp
+++ b/src/cmd_topic.cpp
@@ -2,12 +2,9 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *                <Craig@chatspike.net>
+ *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ * See: http://www.inspircd.org/wiki/index.php/Credits
  *
- * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
  *
@@ -18,15 +15,15 @@
 #include "users.h"
 #include "modules.h"
 #include "commands/cmd_topic.h"
+#include "inspircd.h"
 
 
-
-extern "C" command_t* init_command(InspIRCd* Instance)
+extern "C" DllExport command_t* init_command(InspIRCd* Instance)
 {
        return new cmd_topic(Instance);
 }
 
-void cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
+CmdResult cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
 {
        chanrec* Ptr;
 
@@ -38,7 +35,7 @@ void cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
                        if ((Ptr->modes[CM_SECRET]) && (!Ptr->HasUser(user)))
                        {
                                user->WriteServ("401 %s %s :No such nick/channel",user->nick, Ptr->name);
-                               return;
+                               return CMD_FAILURE;
                        }
                        if (Ptr->topicset)
                        {
@@ -53,8 +50,9 @@ void cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
                else
                {
                        user->WriteServ("401 %s %s :No such nick/channel",user->nick, parameters[0]);
+                       return CMD_FAILURE;
                }
-               return;
+               return CMD_SUCCESS;
        }
        else if (pcnt>1)
        {
@@ -66,38 +64,55 @@ void cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
                                if (!Ptr->HasUser(user))
                                {
                                        user->WriteServ("442 %s %s :You're not on that channel!",user->nick, Ptr->name);
-                                       return;
+                                       return CMD_FAILURE;
                                }
                                if ((Ptr->modes[CM_TOPICLOCK]) && (Ptr->GetStatus(user) < STATUS_HOP))
                                {
-                                       user->WriteServ("482 %s %s :You must be at least a half-operator to change modes on this channel", user->nick, Ptr->name);
-                                       return;
+                                       user->WriteServ("482 %s %s :You must be at least a half-operator to change the topic on this channel", user->nick, Ptr->name);
+                                       return CMD_FAILURE;
                                }
                        }
+
                        char topic[MAXTOPIC];
-                       strlcpy(topic,parameters[1],MAXTOPIC-1);
 
                        if (IS_LOCAL(user))
                        {
+                               /* XXX: we need two string copies for a local topic, because we cant
+                                * let a module see the topic as longer than it actually is
+                                */
                                int MOD_RESULT = 0;
+
+                               strlcpy(topic,parameters[1],MAXTOPIC-1);
                                FOREACH_RESULT(I_OnLocalTopicChange,OnLocalTopicChange(user,Ptr,topic));
                                if (MOD_RESULT)
-                                       return;
+                                       return CMD_FAILURE;
+
+                               strlcpy(Ptr->topic,topic,MAXTOPIC-1);
+                       }
+                       else
+                       {
+                               /* Sneaky shortcut, one string copy for a remote topic */
+                               strlcpy(Ptr->topic, parameters[1], MAXTOPIC-1);
                        }
 
-                       strlcpy(Ptr->topic,topic,MAXTOPIC-1);
-                       strlcpy(Ptr->setby,user->nick,NICKMAX-1);
+                       if (ServerInstance->Config->FullHostInTopic)
+                               strlcpy(Ptr->setby,user->GetFullHost(),127);
+                       else
+                               strlcpy(Ptr->setby,user->nick,127);
+
                        Ptr->topicset = ServerInstance->Time();
                        Ptr->WriteChannel(user, "TOPIC %s :%s", Ptr->name, Ptr->topic);
+
                        if (IS_LOCAL(user))
-                       {
-                               FOREACH_MOD(I_OnPostLocalTopicChange,OnPostLocalTopicChange(user,Ptr,topic));
-                       }
+                               /* We know 'topic' will contain valid data here */
+                               FOREACH_MOD(I_OnPostLocalTopicChange,OnPostLocalTopicChange(user, Ptr, topic));
                }
                else
                {
                        user->WriteServ("401 %s %s :No such nick/channel",user->nick, parameters[0]);
+                       return CMD_FAILURE;
                }
        }
+       return CMD_SUCCESS;
 }