Made SANICK not collide the user (theres no need to in the new 1.1 now we have return...

[user/henk/code/inspircd.git] / src / modules / m_sethost.cpp

diff --git a/src/modules/m_sethost.cpp b/src/modules/m_sethost.cpp
index 65b03c3d053708c7f9099d8789b4d53d1e9c1d90..296c0788aecb5dfd665f212141b6fce14cfc48a9 100644 (file)
--- a/src/modules/m_sethost.cpp
+++ b/src/modules/m_sethost.cpp
@@ -21,29 +21,29 @@ using namespace std;
 #include "users.h"
 #include "channels.h"
 #include "modules.h"
-#include "helperfuncs.h"
+
 #include "inspircd.h"
 
 /* $ModDesc: Provides support for the SETHOST command */
 
 
-extern InspIRCd* ServerInstance;
+
 
 class cmd_sethost : public command_t
 {
  public:
-       cmd_sethost() : command_t("SETHOST",'o',1)
+       cmd_sethost (InspIRCd* Instance) : command_t(Instance,"SETHOST",'o',1)
        {
                this->source = "m_sethost.so";
                syntax = "<new-hostname>";
        }
 
-       void Handle (const char** parameters, int pcnt, userrec *user)
+       CmdResult Handle (const char** parameters, int pcnt, userrec *user)
        {
                if (strlen(parameters[0]) > 64)
                {
                        user->WriteServ("NOTICE %s :*** SETHOST: Host too long",user->nick);
-                       return;
+                       return CMD_FAILURE;
                }
                for (unsigned int x = 0; x < strlen(parameters[0]); x++)
                {
@@ -52,12 +52,17 @@ class cmd_sethost : public command_t
                                if (((parameters[0][x] < '0') || (parameters[0][x]> '9')) && (parameters[0][x] != '-'))
                                {
                                        user->WriteServ("NOTICE "+std::string(user->nick)+" :*** Invalid characters in hostname");
-                                       return;
+                                       return CMD_FAILURE;
                                }
                        }
                }
                if (user->ChangeDisplayedHost(parameters[0]))
+               {
                        ServerInstance->WriteOpers(std::string(user->nick)+" used SETHOST to change their displayed host to "+std::string(parameters[0]));
+                       return CMD_SUCCESS;
+               }
+
+               return CMD_FAILURE;
        }
 };
 
@@ -70,7 +75,7 @@ class ModuleSetHost : public Module
                : Module::Module(Me)
        {
                
-               mycommand = new cmd_sethost();
+               mycommand = new cmd_sethost(ServerInstance);
                ServerInstance->AddCommand(mycommand);
        }