Document a ton of classes (e.g. give them a short description for http://svn.inspircd...

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

diff --git a/src/modules/m_setident.cpp b/src/modules/m_setident.cpp
index 3af511419233a47a8396823426bc7a2c594685f7..065c1d96dd65daca51ab82c56b14301cd12e8a52 100644 (file)
--- a/src/modules/m_setident.cpp
+++ b/src/modules/m_setident.cpp
@@ -1,32 +1,41 @@
 #include "users.h"
 #include "modules.h"
-#include "helperfuncs.h"
+#include "inspircd.h"
 
 /* $ModDesc: Provides support for the SETIDENT command */
 
-Server *Srv;
-
+/** Handle /SETIDENT
+ */
 class cmd_setident : public command_t
 {
  public:
-       cmd_setident() : command_t("SETIDENT", 'o', 1)
+ cmd_setident (InspIRCd* Instance) : command_t(Instance,"SETIDENT", 'o', 1)
        {
                this->source = "m_setident.so";
+               syntax = "<new-ident>";
        }
 
-       void Handle(char **parameters, int pcnt, userrec *user)
+       CmdResult Handle(const char** parameters, int pcnt, userrec *user)
        {
-               for(unsigned int x = 0; x < strlen(parameters[0]); x++)
+               size_t len = 0;
+               for(const char* x = parameters[0]; *x; x++, len++)
                {
-                       if(((parameters[0][x] >= 'A') && (parameters[0][x] <= '}')) || strchr(".-0123456789", parameters[0][x]))
+                       if(((*x >= 'A') && (*x <= '}')) || strchr(".-0123456789", *x))
                                continue;
-                       
-                       WriteServ(user->fd, "NOTICE %s :*** Invalid characters in ident", user->nick);
-                       return;
+
+                       user->WriteServ("NOTICE %s :*** Invalid characters in ident", user->nick);
+                       return CMD_FAILURE;
                }
-               
-               WriteOpers("%s used SETIDENT to change their ident from '%s' to '%s'", user->nick, user->ident, parameters[0]);
-               strlcpy(user->ident, parameters[0], IDENTMAX+2);
+               if (len > IDENTMAX)
+               {
+                       user->WriteServ("NOTICE %s :*** Ident is too long", user->nick);
+                       return CMD_FAILURE;
+               }
+
+               user->ChangeIdent(parameters[0]);
+               ServerInstance->WriteOpers("%s used SETIDENT to change their ident to '%s'", user->nick, user->ident);
+
+               return CMD_SUCCESS;
        }
 };
 
@@ -36,11 +45,11 @@ class ModuleSetIdent : public Module
        cmd_setident*   mycommand;
        
  public:
-       ModuleSetIdent(Server* Me) : Module::Module(Me)
+       ModuleSetIdent(InspIRCd* Me) : Module::Module(Me)
        {
-               Srv = Me;
-               mycommand = new cmd_setident();
-               Srv->AddCommand(mycommand);
+               
+               mycommand = new cmd_setident(ServerInstance);
+               ServerInstance->AddCommand(mycommand);
        }
        
        virtual ~ModuleSetIdent()
@@ -49,7 +58,7 @@ class ModuleSetIdent : public Module
        
        virtual Version GetVersion()
        {
-               return Version(1,0,0,0,VF_VENDOR);
+               return Version(1,0,0,0,VF_VENDOR,API_VERSION);
        }
        
 };
@@ -67,7 +76,7 @@ class ModuleSetIdentFactory : public ModuleFactory
        {
        }
        
-       virtual Module * CreateModule(Server* Me)
+       virtual Module * CreateModule(InspIRCd* Me)
        {
                return new ModuleSetIdent(Me);
        }