Remove channel argument from OnSendWhoLine, this information is already available...

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

diff --git a/src/commands/cmd_topic.cpp b/src/commands/cmd_topic.cpp
index 46c26806f3c83eda0ee0976644504633f47d4d95..2b1e7acfbb5875b6fe67373d39f7e37dd7b1479a 100644 (file)
--- a/src/commands/cmd_topic.cpp
+++ b/src/commands/cmd_topic.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -12,104 +12,68 @@
  */
 
 #include "inspircd.h"
-#include "commands/cmd_topic.h"
 
-
-extern "C" DllExport Command* init_command(InspIRCd* Instance)
+/** Handle /TOPIC. These command handlers can be reloaded by the core,
+ * and handle basic RFC1459 commands. Commands within modules work
+ * the same way, however, they can be fully unloaded, where these
+ * may not.
+ */
+class CommandTopic : public Command
 {
-       return new CommandTopic(Instance);
-}
+ public:
+       /** Constructor for topic.
+        */
+       CommandTopic ( Module* parent) : Command(parent,"TOPIC",1, 2) { syntax = "<channel> [<topic>]"; Penalty = 2; }
+       /** Handle command.
+        * @param parameters The parameters to the comamnd
+        * @param pcnt The number of parameters passed to teh command
+        * @param user The user issuing the command
+        * @return A value from CmdResult to indicate command success or failure.
+        */
+       CmdResult Handle(const std::vector<std::string>& parameters, User *user);
+};
 
-CmdResult CommandTopic::Handle (const char* const* parameters, int pcnt, User *user)
+CmdResult CommandTopic::Handle (const std::vector<std::string>& parameters, User *user)
 {
-       Channel* Ptr;
+       Channel* c;
+
+       c = ServerInstance->FindChan(parameters[0]);
+       if (!c)
+       {
+               user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick.c_str(), parameters[0].c_str());
+               return CMD_FAILURE;
+       }
 
-       if (pcnt == 1)
+       if (parameters.size() == 1)
        {
-               Ptr = ServerInstance->FindChan(parameters[0]);
-               if (Ptr)
+               if (c)
                {
-                       if ((Ptr->IsModeSet('s')) && (!Ptr->HasUser(user)))
+                       if ((c->IsModeSet('s')) && (!c->HasUser(user)))
                        {
-                               user->WriteServ("401 %s %s :No such nick/channel",user->nick, Ptr->name);
+                               user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick.c_str(), c->name.c_str());
                                return CMD_FAILURE;
                        }
-                       if (Ptr->topicset)
+
+                       if (c->topic.length())
                        {
-                               user->WriteServ("332 %s %s :%s", user->nick, Ptr->name, Ptr->topic);
-                               user->WriteServ("333 %s %s %s %d", user->nick, Ptr->name, Ptr->setby, Ptr->topicset);
+                               user->WriteNumeric(332, "%s %s :%s", user->nick.c_str(), c->name.c_str(), c->topic.c_str());
+                               user->WriteNumeric(333, "%s %s %s %lu", user->nick.c_str(), c->name.c_str(), c->setby.c_str(), (unsigned long)c->topicset);
                        }
                        else
                        {
-                               user->WriteServ("331 %s %s :No topic is set.", user->nick, Ptr->name);
+                               user->WriteNumeric(RPL_NOTOPICSET, "%s %s :No topic is set.", user->nick.c_str(), c->name.c_str());
                        }
                }
-               else
-               {
-                       user->WriteServ("401 %s %s :No such nick/channel",user->nick, parameters[0]);
-                       return CMD_FAILURE;
-               }
                return CMD_SUCCESS;
        }
-       else if (pcnt>1)
+       else if (parameters.size()>1)
        {
-               Ptr = ServerInstance->FindChan(parameters[0]);
-               if (Ptr)
-               {
-                       if (IS_LOCAL(user))
-                       {
-                               if (!Ptr->HasUser(user))
-                               {
-                                       user->WriteServ("442 %s %s :You're not on that channel!",user->nick, Ptr->name);
-                                       return CMD_FAILURE;
-                               }
-                               if ((Ptr->IsModeSet('t')) && (Ptr->GetStatus(user) < STATUS_HOP))
-                               {
-                                       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];
-
-                       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 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);
-                       }
-
-                       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))
-                               /* 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;
-               }
+               std::string t = parameters[1]; // needed, in case a module wants to change it
+               c->SetTopic(user, t);
        }
+
        return CMD_SUCCESS;
 }
 
+
+COMMAND_INIT(CommandTopic)