Replace OnAccessCheck with OnPreMode to remove a number of redundant checks

[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 f6e38c2136bc6e515d5501caeeb90be4a284c2a8..8e397c0e13589946eedd17bb72b70f571d7a2da3 100644 (file)
--- a/src/modules/m_sethost.cpp
+++ b/src/modules/m_sethost.cpp
@@ -22,9 +22,8 @@ class CommandSethost : public Command
  private:
        char* hostmap;
  public:
-       CommandSethost (InspIRCd* Instance, char* hmap) : Command(Instance,"SETHOST","o",1), hostmap(hmap)
+       CommandSethost (InspIRCd* Instance, Module* Creator, char* hmap) : Command(Instance,Creator,"SETHOST","o",1), hostmap(hmap)
        {
-               this->source = "m_sethost.so";
                syntax = "<new-hostname>";
                TRANSLATE2(TR_TEXT, TR_END);
        }
@@ -54,7 +53,7 @@ class CommandSethost : public Command
                if (user->ChangeDisplayedHost(parameters[0].c_str()))
                {
                        ServerInstance->SNO->WriteGlobalSno('a', std::string(user->nick)+" used SETHOST to change their displayed host to "+user->dhost);
-                       return CMD_LOCALONLY;
+                       return CMD_SUCCESS;
                }
 
                return CMD_FAILURE;
@@ -64,15 +63,14 @@ class CommandSethost : public Command
 
 class ModuleSetHost : public Module
 {
-       CommandSethost* mycommand;
+       CommandSethost cmd;
        char hostmap[256];
  public:
        ModuleSetHost(InspIRCd* Me)
-               : Module(Me)
+               : Module(Me), cmd(Me, this, hostmap)
        {
                OnRehash(NULL);
-               mycommand = new CommandSethost(ServerInstance, hostmap);
-               ServerInstance->AddCommand(mycommand);
+               ServerInstance->AddCommand(&cmd);
                Implementation eventlist[] = { I_OnRehash };
                ServerInstance->Modules->Attach(eventlist, this, 1);
        }