Works with the m_testclient test program/suite!

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

diff --git a/src/modules/m_helpop.cpp b/src/modules/m_helpop.cpp
index 9a7124dda5d066d5dcfa1c3f4fe0ca28153a5cb2..c1457765d4e6c487eaf60bc064002e7fbe692ab6 100644 (file)
--- a/src/modules/m_helpop.cpp
+++ b/src/modules/m_helpop.cpp
@@ -25,12 +25,39 @@ using namespace std;
 static ConfigReader *helpop;
 static Server *Srv;
 
-void handle_helpop(char**, int, userrec*);
-bool do_helpop(char**, int, userrec*);
-void sendtohelpop(userrec*, int, char**);
+bool do_helpop(const char**, int, userrec*);
+void sendtohelpop(userrec*, int, const char**);
 
 /* $ModDesc: /helpop Command, Works like Unreal helpop */
 
+class Helpop : public ModeHandler
+{
+ public:
+       Helpop() : ModeHandler('h', 0, 0, false, MODETYPE_USER, true) { }
+
+       ModeAction OnModeChange(userrec* source, userrec* dest, chanrec* channel, std::string &parameter, bool adding)
+       {
+               if (adding)
+               {
+                       if (!dest->IsModeSet('h'))
+                       {
+                               dest->SetMode('h',true);
+                               return MODEACTION_ALLOW;
+                       }
+               }
+               else
+               {
+                       if (dest->IsModeSet('h'))
+                       {
+                               dest->SetMode('h',false);
+                               return MODEACTION_ALLOW;
+                       }
+               }
+
+               return MODEACTION_DENY;
+       }
+};
+
 class cmd_helpop : public command_t
 {
  public:
@@ -39,7 +66,7 @@ class cmd_helpop : public command_t
                 this->source = "m_helpop.so";
         }
 
-       void Handle (char **parameters, int pcnt, userrec *user)
+       void Handle (const char** parameters, int pcnt, userrec *user)
        {
                char a[MAXBUF];
                std::string output = " ";
@@ -53,12 +80,12 @@ class cmd_helpop : public command_t
                        return;
                }
 
-               if (parameters[0][0] == '!')
+               if (*parameters[0] == '!')
                {
                        // Force send to all +h users
                        sendtohelpop(user, pcnt, parameters);
                }
-               else if (parameters[0][0] == '?')
+               else if (*parameters[0] == '?')
                {
                        // Force to the helpop system with no forward if not found.
                        if (do_helpop(parameters, pcnt, user) == false)
@@ -100,25 +127,22 @@ class cmd_helpop : public command_t
 };
 
 
-bool do_helpop(char **parameters, int pcnt, userrec *src)
+bool do_helpop(const char** parameters, int pcnt, userrec *src)
 {
-       char *search;
+       char search[MAXBUF];
        std::string output = " "; // a fix bought to you by brain :p
        char a[MAXBUF];
        int nlines = 0;
 
        if (!pcnt)
        {
-               search = "start";
+               strcpy(search,"start");
        }
        else
        {
-               search = parameters[0];
-       }
-
-       if (search[0] == '?')
-       {
-               search++;
+               if (*parameters[0] == '?')
+                       parameters[0]++;
+               strlcpy(search,parameters[0],MAXBUF);
        }
 
        strlower(search);
@@ -138,9 +162,9 @@ bool do_helpop(char **parameters, int pcnt, userrec *src)
 
 
 
-void sendtohelpop(userrec *src, int pcnt, char **params)
+void sendtohelpop(userrec *src, int pcnt, const char **params)
 {
-       char* first = params[0];
+       const char* first = params[0];
        if (*first == '!')
        {
                first++;
@@ -169,6 +193,7 @@ class ModuleHelpop : public Module
                ConfigReader *conf;
                std::string  h_file;
                cmd_helpop* mycommand;
+               Helpop* ho;
 
        public:
                ModuleHelpop(Server* Me)
@@ -177,12 +202,8 @@ class ModuleHelpop : public Module
                        Srv  = Me;
 
                        ReadConfig();
-                       if (!Srv->AddExtendedMode('h',MT_CLIENT,true,0,0))
-                       {
-                               Srv->Log(DEFAULT,"Unable to claim the +h usermode.");
-                               return;
-                       }
-
+                       ho = new Helpop();
+                       Srv->AddMode(ho, 'h');
                        mycommand = new cmd_helpop();
                        Srv->AddCommand(mycommand);
                }
@@ -211,7 +232,7 @@ class ModuleHelpop : public Module
 
                void Implements(char* List)
                {
-                       List[I_OnRehash] = List[I_OnExtendedMode] = List[I_OnWhois] = 1;
+                       List[I_OnRehash] = List[I_OnWhois] = 1;
                }
 
                virtual void OnRehash(const std::string &parameter)
@@ -223,18 +244,9 @@ class ModuleHelpop : public Module
                        ReadConfig();
                }
 
-               virtual int OnExtendedMode(userrec* user, void* target, char modechar, int type, bool mode_on, string_list &params)
-               {
-                       if ((modechar == 'h') && (type == MT_CLIENT))
-                       {
-                               return 1;
-                       }
-                       return 0;
-               }
-
                virtual void OnWhois(userrec* src, userrec* dst)
                {
-                       if (dst->modes['h'-65])
+                       if (dst->IsModeSet('h'))
                        {
                                Srv->SendTo(NULL,src,"310 "+std::string(src->nick)+" "+std::string(dst->nick)+" :is available for help.");
                        }
@@ -244,6 +256,7 @@ class ModuleHelpop : public Module
                {
                        DELETE(conf);
                        DELETE(helpop);
+                       DELETE(ho);
                }
        
                virtual Version GetVersion()