]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/mode.cpp
Fix to check for colon (accidentally typed pos_of_dot)
[user/henk/code/inspircd.git] / src / mode.cpp
index 24531651a4ca08c63039d8d7944751890ee34b2d..7290248619fbf0a631b6d6b7ee56a8d5054c6583 100644 (file)
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  Inspire is copyright (C) 2002-2004 ChatSpike-Dev.
- *                       E-mail:
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
+ *                      E-mail:
  *                <brain@chatspike.net>
  *               <Craig@chatspike.net>
  *     
@@ -18,16 +18,12 @@ using namespace std;
 
 #include "inspircd_config.h"
 #include "inspircd.h"
-#include "inspircd_io.h"
+#include "configreader.h"
 #include <unistd.h>
 #include <sys/errno.h>
 #include <time.h>
 #include <string>
-#ifdef GCC3
-#include <ext/hash_map>
-#else
-#include <hash_map>
-#endif
+#include "hash_map.h"
 #include <map>
 #include <sstream>
 #include <vector>
@@ -46,6 +42,15 @@ using namespace std;
 #include "helperfuncs.h"
 #include "mode.h"
 
+/* +s (secret) */
+#include "modes/cmode_s.h"
+/* +p (private) */
+#include "modes/cmode_p.h"
+/* +b (bans) */
+#include "modes/cmode_b.h"
+/* +m (moderated) */
+#include "modes/cmode_m.h"
+
 extern int MODCOUNT;
 extern std::vector<Module*> modules;
 extern std::vector<ircd_module*> factory;
@@ -54,1639 +59,590 @@ extern ServerConfig* Config;
 
 extern time_t TIME;
 
-char* ModeParser::GiveOps(userrec *user,char *dest,chanrec *chan,int status)
+ModeHandler::ModeHandler(char modeletter, int parameters_on, int parameters_off, bool listmode, ModeType type, bool operonly)
+       : mode(modeletter), n_params_on(parameters_on), n_params_off(parameters_off), list(listmode), m_type(type), oper(operonly)
+{
+}
+
+ModeHandler::~ModeHandler()
+{
+}
+
+bool ModeHandler::IsListMode()
+{
+       return list;
+}
+
+ModeType ModeHandler::GetModeType()
+{
+       return m_type;
+}
+
+bool ModeHandler::NeedsOper()
+{
+       return oper;
+}
+
+int ModeHandler::GetNumParams(bool adding)
+{
+       return adding ? n_params_on : n_params_off;
+}
+
+char ModeHandler::GetModeChar()
+{
+       return mode;
+}
+
+ModeAction ModeHandler::OnModeChange(userrec* source, userrec* dest, chanrec* channel, std::string &parameter, bool adding)
+{
+       return MODEACTION_DENY;
+}
+
+void ModeHandler::DisplayList(userrec* user, chanrec* channel)
+{
+}
+
+bool ModeHandler::CheckTimeStamp(time_t theirs, time_t ours, const std::string &their_param, const std::string &our_param, chanrec* channel)
+{
+       return (ours < theirs);
+}
+
+ModeWatcher::ModeWatcher(char modeletter, ModeType type) : mode(modeletter), m_type(type)
+{
+}
+
+ModeWatcher::~ModeWatcher()
+{
+}
+
+char ModeWatcher::GetModeChar()
+{
+       return mode;
+}
+
+ModeType ModeWatcher::GetModeType()
+{
+       return m_type;
+}
+
+bool ModeWatcher::BeforeMode(userrec* source, userrec* dest, chanrec* channel, std::string &parameter, bool adding, ModeType type)
+{
+       return true;
+}
+
+void ModeWatcher::AfterMode(userrec* source, userrec* dest, chanrec* channel, const std::string &parameter, bool adding, ModeType type)
+{
+}
+
+userrec* ModeParser::SanityChecks(userrec *user,char *dest,chanrec *chan,int status)
 {
        userrec *d;
-       
        if ((!user) || (!dest) || (!chan) || (!*dest))
        {
-               log(DEFAULT,"*** BUG *** GiveOps was given an invalid parameter");
                return NULL;
        }
        d = Find(dest);
        if (!d)
        {
-               log(DEFAULT,"the target nickname given to GiveOps couldnt be found");
                WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
                return NULL;
        }
-       else
-       {
+       return d;
+}
 
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_OP));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+char* ModeParser::Grant(userrec *d,chanrec *chan,int MASK)
+{
+       if (!chan)
+               return NULL;
+
+       for (std::vector<ucrec*>::const_iterator i = d->chans.begin(); i != d->chans.end(); i++)
+       {
+               if (((ucrec*)(*i))->channel == chan)
                {
-                       if ((status < STATUS_OP) && (!is_uline(user->server)))
+                       if (((ucrec*)(*i))->uc_modes & MASK)
                        {
-                               log(DEBUG,"%s cant give ops to %s because they nave status %d and needs %d",user->nick,dest,status,STATUS_OP);
-                               WriteServ(user->fd,"482 %s %s :You're not a channel operator",user->nick, chan->name);
                                return NULL;
                        }
-               }
-
-
-               for (unsigned int i = 0; i < d->chans.size(); i++)
-               {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       ((ucrec*)(*i))->uc_modes = ((ucrec*)(*i))->uc_modes | MASK;
+                       switch (MASK)
                        {
-                       if (d->chans[i].uc_modes & UCMODE_OP)
-                               {
-                                       /* mode already set on user, dont allow multiple */
-                                       log(DEFAULT,"The target user given to GiveOps was already opped on the channel");
-                                       return NULL;
-                               }
-                               d->chans[i].uc_modes = d->chans[i].uc_modes | UCMODE_OP;
-                               log(DEBUG,"gave ops: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
+                               case UCMODE_OP:
+                                       ((ucrec*)(*i))->channel->AddOppedUser(d);
+                               break;
+                               case UCMODE_HOP:
+                                       ((ucrec*)(*i))->channel->AddHalfoppedUser(d);
+                               break;
+                               case UCMODE_VOICE:
+                                       ((ucrec*)(*i))->channel->AddVoicedUser(d);
+                               break;
                        }
+                       log(DEBUG,"grant: %s %s",((ucrec*)(*i))->channel->name,d->nick);
+                       return d->nick;
                }
-               log(DEFAULT,"The target channel given to GiveOps was not in the users mode list");
        }
        return NULL;
 }
 
-char* ModeParser::GiveHops(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::Revoke(userrec *d,chanrec *chan,int MASK)
 {
-       userrec *d;
-       
-       if ((!user) || (!dest) || (!chan) || (!*dest))
-       {
-               log(DEFAULT,"*** BUG *** GiveHops was given an invalid parameter");
+       if (!chan)
                return NULL;
-       }
 
-       d = Find(dest);
-       if (!d)
-       {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
-               return NULL;
-       }
-       else
+       for (std::vector<ucrec*>::const_iterator i = d->chans.begin(); i != d->chans.end(); i++)
        {
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_HALFOP));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+               if (((ucrec*)(*i))->channel == chan)
                {
-                       if ((status < STATUS_OP) && (!is_uline(user->server)))
+                       if ((((ucrec*)(*i))->uc_modes & MASK) == 0)
                        {
-                               WriteServ(user->fd,"482 %s %s :You're not a channel operator",user->nick, chan->name);
                                return NULL;
                        }
+                       ((ucrec*)(*i))->uc_modes ^= MASK;
+                       switch (MASK)
+                       {
+                               case UCMODE_OP:
+                                       ((ucrec*)(*i))->channel->DelOppedUser(d);
+                               break;
+                               case UCMODE_HOP:
+                                       ((ucrec*)(*i))->channel->DelHalfoppedUser(d);
+                               break;
+                               case UCMODE_VOICE:
+                                       ((ucrec*)(*i))->channel->DelVoicedUser(d);
+                               break;
+                       }
+                       log(DEBUG,"revoke: %s %s",((ucrec*)(*i))->channel->name,d->nick);
+                       return d->nick;
                }
+       }
+       return NULL;
+}
 
-               for (unsigned int i = 0; i < d->chans.size(); i++)
+char* ModeParser::GiveOps(userrec *user,char *dest,chanrec *chan,int status)
+{
+       userrec *d = this->SanityChecks(user,dest,chan,status);
+       
+       if (d)
+       {
+               if (IS_LOCAL(user))
                {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_OP));
+                       
+                       if (MOD_RESULT == ACR_DENY)
+                               return NULL;
+                       if (MOD_RESULT == ACR_DEFAULT)
                        {
-                               if (d->chans[i].uc_modes & UCMODE_HOP)
+                               if ((status < STATUS_OP) && (!is_uline(user->server)))
                                {
-                                       /* mode already set on user, dont allow multiple */
+                                       WriteServ(user->fd,"482 %s %s :You're not a channel operator",user->nick, chan->name);
                                        return NULL;
                                }
-                               d->chans[i].uc_modes = d->chans[i].uc_modes | UCMODE_HOP;
-                               log(DEBUG,"gave h-ops: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
                        }
                }
+
+               return this->Grant(d,chan,UCMODE_OP);
        }
        return NULL;
 }
 
-char* ModeParser::GiveVoice(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::GiveHops(userrec *user,char *dest,chanrec *chan,int status)
 {
-       userrec *d;
+       userrec *d = this->SanityChecks(user,dest,chan,status);
        
-       if ((!user) || (!dest) || (!chan) || (!*dest))
-       {
-               log(DEFAULT,"*** BUG *** GiveVoice was given an invalid parameter");
-               return NULL;
-       }
-
-       d = Find(dest);
-       if (!d)
-       {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
-               return NULL;
-       }
-       else
+       if (d)
        {
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_VOICE));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+               if (IS_LOCAL(user))
                {
-                       if ((status < STATUS_HOP) && (!is_uline(user->server)))
-                       {
-                               WriteServ(user->fd,"482 %s %s :You must be at least a half-operator to change modes on this channel",user->nick, chan->name);
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_HALFOP));
+               
+                       if (MOD_RESULT == ACR_DENY)
                                return NULL;
-                       }
-               }
-
-               for (unsigned int i = 0; i < d->chans.size(); i++)
-               {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       if (MOD_RESULT == ACR_DEFAULT)
                        {
-                               if (d->chans[i].uc_modes & UCMODE_VOICE)
+                               if ((status < STATUS_OP) && (!is_uline(user->server)))
                                {
-                                       /* mode already set on user, dont allow multiple */
+                                       WriteServ(user->fd,"482 %s %s :You're not a channel operator",user->nick, chan->name);
                                        return NULL;
                                }
-                               d->chans[i].uc_modes = d->chans[i].uc_modes | UCMODE_VOICE;
-                               log(DEBUG,"gave voice: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
                        }
                }
+
+               return this->Grant(d,chan,UCMODE_HOP);
        }
        return NULL;
 }
 
-char* ModeParser::TakeOps(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::GiveVoice(userrec *user,char *dest,chanrec *chan,int status)
 {
-       userrec *d;
+       userrec *d = this->SanityChecks(user,dest,chan,status);
        
-       if ((!user) || (!dest) || (!chan) || (!*dest))
+       if (d)
        {
-               log(DEFAULT,"*** BUG *** TakeOps was given an invalid parameter");
-               return NULL;
-       }
-
-       d = Find(dest);
-       if (!d)
-       {
-               log(DEBUG,"TakeOps couldnt resolve the target nickname: %s",dest);
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
-               return NULL;
-       }
-       else
-       {
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEOP));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+               if (IS_LOCAL(user))
                {
-                       if ((status < STATUS_OP) && (!is_uline(user->server)))
-                       {
-                               WriteServ(user->fd,"482 %s %s :You are not a channel operator",user->nick, chan->name);
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_VOICE));
+                       
+                       if (MOD_RESULT == ACR_DENY)
                                return NULL;
-                       }
-               }
-
-               for (unsigned int i = 0; i < d->chans.size(); i++)
-               {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       if (MOD_RESULT == ACR_DEFAULT)
                        {
-                               if ((d->chans[i].uc_modes & UCMODE_OP) == 0)
+                               if ((status < STATUS_HOP) && (!is_uline(user->server)))
                                {
-                                       /* mode already set on user, dont allow multiple */
+                                       WriteServ(user->fd,"482 %s %s :You must be at least a half-operator to change modes on this channel",user->nick, chan->name);
                                        return NULL;
                                }
-                               d->chans[i].uc_modes ^= UCMODE_OP;
-                               log(DEBUG,"took ops: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
                        }
                }
-               log(DEBUG,"TakeOps couldnt locate the target channel in the target users list");
+
+               return this->Grant(d,chan,UCMODE_VOICE);
        }
        return NULL;
 }
 
-char* ModeParser::TakeHops(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::TakeOps(userrec *user,char *dest,chanrec *chan,int status)
 {
-       userrec *d;
+       userrec *d = this->SanityChecks(user,dest,chan,status);
        
-       if ((!user) || (!dest) || (!chan) || (!*dest))
-       {
-               log(DEFAULT,"*** BUG *** TakeHops was given an invalid parameter");
-               return NULL;
-       }
-
-       d = Find(dest);
-       if (!d)
-       {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
-               return NULL;
-       }
-       else
+       if (d)
        {
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEHALFOP));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+               if (IS_LOCAL(user))
                {
-                       /* Tweak by Brain suggested by w00t, allow a halfop to dehalfop themselves */
-                       if ((user != d) && ((status < STATUS_OP) && (!is_uline(user->server))))
-                       {
-                               WriteServ(user->fd,"482 %s %s :You are not a channel operator",user->nick, chan->name);
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEOP));
+                       
+                       if (MOD_RESULT == ACR_DENY)
                                return NULL;
-                       }
-               }
-
-               for (unsigned int i = 0; i < d->chans.size(); i++)
-               {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       if (MOD_RESULT == ACR_DEFAULT)
                        {
-                               if ((d->chans[i].uc_modes & UCMODE_HOP) == 0)
+                               if ((status < STATUS_OP) && (!is_uline(user->server)) && (IS_LOCAL(user)))
                                {
-                                       /* mode already set on user, dont allow multiple */
+                                       WriteServ(user->fd,"482 %s %s :You are not a channel operator",user->nick, chan->name);
                                        return NULL;
                                }
-                               d->chans[i].uc_modes ^= UCMODE_HOP;
-                               log(DEBUG,"took h-ops: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
                        }
                }
+
+               return this->Revoke(d,chan,UCMODE_OP);
        }
        return NULL;
 }
 
-char* ModeParser::TakeVoice(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::TakeHops(userrec *user,char *dest,chanrec *chan,int status)
 {
-       userrec *d;
+       userrec *d = this->SanityChecks(user,dest,chan,status);
        
-       if ((!user) || (!dest) || (!chan) || (!*dest))
-       {
-               log(DEFAULT,"*** BUG *** TakeVoice was given an invalid parameter");
-               return NULL;
-       }
-
-       d = Find(dest);
-       if (!d)
+       if (d)
        {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, dest);
-               return NULL;
-       }
-       else
-       {
-               int MOD_RESULT = 0;
-               FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEVOICE));
-               
-               if (MOD_RESULT == ACR_DENY)
-                       return NULL;
-               if (MOD_RESULT == ACR_DEFAULT)
+               if (IS_LOCAL(user))
                {
-                       if ((status < STATUS_HOP) && (!is_uline(user->server)))
-                       {
-                               WriteServ(user->fd,"482 %s %s :You must be at least a half-operator to change modes on this channel",user->nick, chan->name);
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEHALFOP));
+                       
+                       if (MOD_RESULT == ACR_DENY)
                                return NULL;
-                       }
-               }
-
-               for (unsigned int i = 0; i < d->chans.size(); i++)
-               {
-                       if ((d->chans[i].channel != NULL) && (chan != NULL))
-                       if (!strcasecmp(d->chans[i].channel->name,chan->name))
+                       if (MOD_RESULT == ACR_DEFAULT)
                        {
-                               if ((d->chans[i].uc_modes & UCMODE_VOICE) == 0)
+                               /* Tweak by Brain suggested by w00t, allow a halfop to dehalfop themselves */
+                               if ((user != d) && ((status < STATUS_OP) && (!is_uline(user->server))))
                                {
-                                       /* mode already set on user, dont allow multiple */
+                                       WriteServ(user->fd,"482 %s %s :You are not a channel operator",user->nick, chan->name);
                                        return NULL;
                                }
-                               d->chans[i].uc_modes ^= UCMODE_VOICE;
-                               log(DEBUG,"took voice: %s %s",d->chans[i].channel->name,d->nick);
-                               return d->nick;
                        }
                }
+
+               return this->Revoke(d,chan,UCMODE_HOP);
        }
        return NULL;
 }
 
-char* ModeParser::AddBan(userrec *user,char *dest,chanrec *chan,int status)
+char* ModeParser::TakeVoice(userrec *user,char *dest,chanrec *chan,int status)
 {
-       BanItem b;
-       unsigned int l;
-       int toomanyexclamation = 0;
-       int toomanyat = 0;
-
-       if ((!user) || (!dest) || (!chan) || (!*dest))
-       {
-               log(DEFAULT,"*** BUG *** AddBan was given an invalid parameter");
-               return NULL;
-       }
-
-       l = strlen(dest);
-
-       for (unsigned int i = 0; i < l; i++)
-       {
-               if (dest[i] < 32)
-                       return NULL;
-               if (dest[i] > 126)
-                       return NULL;
-               if (dest[i] == '!')
-                       toomanyexclamation++;
-               if (dest[i] == '@')
-                       toomanyat++;
-       }
-
-       if (toomanyexclamation != 1 || toomanyat != 1)
-               /*
-                * this stops sillyness like n!u!u!u@h, though note that most
-                * ircds don't actually verify banmask validity. --w00t
-                */
-               return NULL;
-
-       long maxbans = GetMaxBans(chan->name);
-       if ((unsigned)chan->bans.size() > (unsigned)maxbans)
-       {
-               WriteServ(user->fd,"478 %s %s :Channel ban list for %s is full (maximum entries for this channel is %d)",user->nick, chan->name,chan->name,maxbans);
-               return NULL;
-       }
-
-       log(DEBUG,"AddBan: %s %s",chan->name,user->nick);
+       userrec *d = this->SanityChecks(user,dest,chan,status);
 
-       int MOD_RESULT = 0;
-       FOREACH_RESULT(I_OnAddBan,OnAddBan(user,chan,dest));
-       if (MOD_RESULT)
-               return NULL;
-
-       TidyBan(dest);
-       for (BanList::iterator i = chan->bans.begin(); i != chan->bans.end(); i++)
+       if (d)  
        {
-               if (!strcasecmp(i->data,dest))
+               if (IS_LOCAL(user))
                {
-                       // dont allow a user to set the same ban twice
-                       return NULL;
+                       int MOD_RESULT = 0;
+                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,d,chan,AC_DEVOICE));
+                       
+                       if (MOD_RESULT == ACR_DENY)
+                               return NULL;
+                       if (MOD_RESULT == ACR_DEFAULT)
+                       {
+                               if ((status < STATUS_HOP) && (!is_uline(user->server)))
+                               {
+                                       WriteServ(user->fd,"482 %s %s :You must be at least a half-operator to change modes on this channel",user->nick, chan->name);
+                                       return NULL;
+                               }
+                       }
                }
-       }
 
-       b.set_time = TIME;
-       strncpy(b.data,dest,MAXBUF);
-       strncpy(b.set_by,user->nick,NICKMAX);
-       chan->bans.push_back(b);
-       return dest;
+               return this->Revoke(d,chan,UCMODE_VOICE);
+       }
+       return NULL;
 }
 
-char* ModeParser::TakeBan(userrec *user,char *dest,chanrec *chan,int status)
+void ModeParser::Process(char **parameters, int pcnt, userrec *user, bool servermode)
 {
-       if ((!user) || (!dest) || (!chan) || (!*dest)) {
-               log(DEFAULT,"*** BUG *** TakeBan was given an invalid parameter");
-               return 0;
-       }
+       std::string target = parameters[0];
+       ModeType type = MODETYPE_USER;
+       unsigned char mask = 0;
+       chanrec* targetchannel = FindChan(parameters[0]);
+       userrec* targetuser  = Find(parameters[0]);
 
-       log(DEBUG,"del_ban: %s %s",chan->name,user->nick);
-       for (BanList::iterator i = chan->bans.begin(); i != chan->bans.end(); i++)
+       log(DEBUG,"ModeParser::Process start");
+
+       if (pcnt > 1)
        {
-               if (!strcasecmp(i->data,dest))
+               if (targetchannel)
                {
-                       int MOD_RESULT = 0;
-                       FOREACH_RESULT(I_OnDelBan,OnDelBan(user,chan,dest));
-                       if (MOD_RESULT)
-                               return NULL;
-                       chan->bans.erase(i);
-                       return dest;
+                       log(DEBUG,"Target type is CHANNEL");
+                       type = MODETYPE_CHANNEL;
+                       mask = MASK_CHANNEL;
                }
-       }
-       return NULL;
-}
-
-// tidies up redundant modes, e.g. +nt-nt+i becomes +-+i,
-// a section further down the chain tidies up the +-+- crap.
-std::string ModeParser::CompressModes(std::string modes,bool channelmodes)
-{
-       int counts[127];
-       bool active[127];
-       memset(counts,0,sizeof(counts));
-       memset(active,0,sizeof(active));
-       for (unsigned int i = 0; i < modes.length(); i++)
-       {
-               if ((modes[i] == '+') || (modes[i] == '-'))
-                       continue;
-               if (channelmodes)
+               else if (targetuser)
                {
-                       if ((strchr("itnmsp",modes[i])) || ((ModeDefined(modes[i],MT_CHANNEL)) && (ModeDefinedOn(modes[i],MT_CHANNEL)==0) && (ModeDefinedOff(modes[i],MT_CHANNEL)==0)))
-                       {
-                               log(DEBUG,"Tidy mode %c",modes[i]);
-                               counts[(unsigned int)modes[i]]++;
-                               active[(unsigned int)modes[i]] = true;
-                       }
+                       log(DEBUG,"Target type is USER");
+                       type = MODETYPE_USER;
+                       mask = MASK_USER;
                }
                else
                {
-                       log(DEBUG,"Tidy mode %c",modes[i]);
-                       counts[(unsigned int)modes[i]]++;
-                       active[(unsigned int)modes[i]] = true;
+                       /* No such nick/channel */
+                       log(DEBUG,"Target type is UNKNOWN, bailing");
+                       return;
                }
-       }
-       for (int j = 65; j < 127; j++)
-       {
-               if ((counts[j] > 1) && (active[j] == true))
+               std::string mode_sequence = parameters[1];
+               std::string parameter = "";
+               std::ostringstream parameter_list;
+               std::string output_sequence = "";
+               bool adding = true, state_change = false;
+               int handler_id = 0;
+               int parameter_counter = 2; /* Index of first parameter */
+
+               for (std::string::const_iterator modeletter = mode_sequence.begin(); modeletter != mode_sequence.end(); modeletter++)
                {
-                       static char v[2];
-                       v[0] = (unsigned char)j;
-                       v[1] = '\0';
-                       std::string mode_str = v;
-                       std::string::size_type pos = modes.find(mode_str);
-                       if (pos != std::string::npos)
+                       switch (*modeletter)
                        {
-                               log(DEBUG,"all occurances of mode %c to be deleted...",(unsigned char)j);
-                               while (modes.find(mode_str) != std::string::npos)
-                                       modes.erase(modes.find(mode_str),1);
-                               log(DEBUG,"New mode line: %s",modes.c_str());
-                       }
-               }
-       }
-       return modes;
-}
-
-void ModeParser::ProcessModes(char **parameters,userrec* user,chanrec *chan,int status, int pcnt, bool servermode, bool silent, bool local)
-{
-       if (!parameters) {
-               log(DEFAULT,"*** BUG *** process_modes was given an invalid parameter");
-               return;
-       }
 
-       char modelist[MAXBUF];
-       char outlist[MAXBUF];
-       char outstr[MAXBUF];
-       char outpars[32][MAXBUF];
-       int param = 2;
-       int pc = 0;
-       int ptr = 0;
-       int mdir = 1;
-       char* r = NULL;
-       bool k_set = false, l_set = false, previously_set_l = false, previously_unset_l = false, previously_set_k = false, previously_unset_k = false;
-
-       if (pcnt < 2)
-       {
-               return;
-       }
-
-       int MOD_RESULT = 0;
-       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,NULL,chan,AC_GENERAL_MODE));
-       
-       if (MOD_RESULT == ACR_DENY)
-               return;
-
-       log(DEBUG,"process_modes: start: parameters=%d",pcnt);
-
-       strlcpy(modelist,parameters[1],MAXBUF); /* mode list, e.g. +oo-o *
-                                                * parameters[2] onwards are parameters for
-                                                * modes that require them :) */
-       strlcpy(outlist,"+",MAXBUF);
-       mdir = 1;
-
-       log(DEBUG,"process_modes: modelist: %s",modelist);
-
-       std::string tidied = this->CompressModes(modelist,true);
-       strlcpy(modelist,tidied.c_str(),MAXBUF);
-
-       int len = strlen(modelist);
-       while (modelist[len-1] == ' ')
-               modelist[--len] = '\0';
-       for (ptr = 0; ptr < len; ptr++)
-       {
-               r = NULL;
-
-               {
-                       log(DEBUG,"process_modes: modechar: %c",modelist[ptr]);
-
-                       char modechar = modelist[ptr];
-                       switch (modelist[ptr])
-                       {
-                               case '-':
-                                       if (mdir != 0)
-                                       {
-                                               int t = strlen(outlist)-1;
-                                               if ((outlist[t] == '+') || (outlist[t] == '-'))
-                                               {
-                                                       outlist[t] = '-';
-                                               }
-                                               else
-                                               {
-                                                       strcat(outlist,"-");
-                                               }
-                                       }
-                                       mdir = 0;
-                                       
-                               break;                  
+                               log(DEBUG,"Iterate mode letter %c",*modeletter);
 
+                               /* NB:
+                                * For + and - mode characters, we don't just stick the character into the output sequence.
+                                * This is because the user may do something dumb, like: +-+ooo or +oo-+. To prevent this
+                                * appearing in the output sequence, we store a flag which says there was a state change,
+                                * which is set on any + or -, however, the + or - that we finish on is only appended to
+                                * the output stream in the event it is followed by a non "+ or -" character, such as o or v.
+                                */
                                case '+':
-                                       if (mdir != 1)
-                                       {
-                                               int t = strlen(outlist)-1;
-                                               if ((outlist[t] == '+') || (outlist[t] == '-'))
-                                               {
-                                                       outlist[t] = '+';
-                                               }
-                                               else
-                                               {
-                                                       strcat(outlist,"+");
-                                               }
-                                       }
-                                       mdir = 1;
-                               break;
-
-                               case 'o':
-                                       log(DEBUG,"Ops");
-                                       if ((param >= pcnt)) break;
-                                       log(DEBUG,"Enough parameters left");
-                                       if (mdir == 1)
-                                       {
-                                               MOD_RESULT = 0;
-                                               FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'o', parameters[param], true, 1));
-                                               if (!MOD_RESULT)
-                                               {
-                                                       log(DEBUG,"calling GiveOps");
-                                                       r = GiveOps(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       else
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'o', parameters[param], false, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       log(DEBUG,"calling TakeOps");
-                                                       r = TakeOps(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       if (r)
-                                       {
-                                               strlcat(outlist,"o",MAXBUF);
-                                               strlcpy(outpars[pc++],r,MAXBUF);
-                                       }
+                                       /* The following expression prevents: +o+o nick nick, compressing it to +oo nick nick,
+                                        * however, will allow the + if it is the first item in the sequence, regardless.
+                                        */
+                                       if ((!adding) || (!output_sequence.length()))
+                                               state_change = true;
+                                       adding = true;
+                                       continue;
                                break;
-                       
-                               case 'h':
-                                       if (((param >= pcnt)) || (!Config->AllowHalfop)) break;
-                                       if (mdir == 1)
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'h', parameters[param], true, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = GiveHops(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       else
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'h', parameters[param], false, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = TakeHops(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       if (r)
-                                       {
-                                               strlcat(outlist,"h",MAXBUF);
-                                               strlcpy(outpars[pc++],r,MAXBUF);
-                                       }
-                               break;
-                       
-                               
-                               case 'v':
-                                       if ((param >= pcnt)) break;
-                                       if (mdir == 1)
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'v', parameters[param], true, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = GiveVoice(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       else
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'v', parameters[param], false, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = TakeVoice(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       if (r)
-                                       {
-                                               strlcat(outlist,"v",MAXBUF);
-                                               strlcpy(outpars[pc++],r,MAXBUF);
-                                       }
-                               break;
-                               
-                               case 'b':
-                                       if ((param >= pcnt)) break;
-                                       if (mdir == 1)
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'b', parameters[param], true, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = AddBan(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       else
-                                       {
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'b', parameters[param], false, 1));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       r = TakeBan(user,parameters[param++],chan,status);
-                                               }
-                                               else param++;
-                                       }
-                                       if (r)
-                                       {
-                                               strlcat(outlist,"b",MAXBUF);
-                                               strlcpy(outpars[pc++],parameters[param-1],MAXBUF);
-                                       }
+                               case '-':
+                                       if ((adding) || (!output_sequence.length()))
+                                               state_change = true;
+                                       adding = false;
+                                       continue;
                                break;
+                               default:
 
+                                       /**
+                                        * Watch carefully for the sleight of hand trick.
+                                        * 65 is the ascii value of 'A'. We take this from
+                                        * the char we're looking at to get a number between
+                                        * 1 and 127. We then logic-or it to get the hashed
+                                        * position, dependent on wether its a channel or
+                                        * a user mode. This is a little stranger, but a lot
+                                        * faster, than using a map of pairs.
+                                        */
+                                       handler_id = (*modeletter - 65) | mask;
 
-                               case 'k':
-                                       if ((param >= pcnt))
-                                               break;
-
-                                       if (mdir == 1)
+                                       if (modehandlers[handler_id])
                                        {
-                                               if (k_set)
-                                                       break;
-
-                                               if (previously_unset_k)
-                                                       break;
-                                               previously_set_k = true;
-                                               
-                                               if (!strcmp(chan->key,""))
+                                               bool abort = false;
+
+                                               log(DEBUG,"Found a ModeHandler* for mode %c",*modeletter);
+
+                                               for (ModeWatchIter watchers = modewatchers[handler_id].begin(); watchers != modewatchers[handler_id].end(); watchers++)
                                                {
-                                                       MOD_RESULT = 0;
-                                                       FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'k', parameters[param], true, 1));
-                                                       if (!MOD_RESULT)
-                                                       {
-                                                               strcat(outlist,"k");
-                                                               char key[MAXBUF];
-                                                               strlcpy(key,parameters[param++],32);
-                                                               strlcpy(outpars[pc++],key,MAXBUF);
-                                                               strlcpy(chan->key,key,MAXBUF);
-                                                               k_set = true;
-                                                       }
-                                                       else param++;
+                                                       log(DEBUG,"Call a ModeWatcher*");
+                                                       if ((*watchers)->BeforeMode(user, targetuser, targetchannel, parameter, adding, type) == MODEACTION_DENY)
+                                                               abort = true;
                                                }
-                                       }
-                                       else
-                                       {
-                                               /* checks on -k are case sensitive and only accurate to the
-                                                  first 32 characters */
-                                               if (previously_set_k)
-                                                       break;
-                                               previously_unset_k = true;
-
-                                               char key[MAXBUF];
-                                               MOD_RESULT = 0;
-                                               FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'k', parameters[param], false, 1));
-                                               if (!MOD_RESULT)
+                                               if ((modehandlers[handler_id]->GetModeType() == type) && (!abort))
                                                {
-                                                       strlcpy(key,parameters[param++],32);
-                                                       /* only allow -k if correct key given */
-                                                       if (!strcmp(chan->key,key))
-                                                       {
-                                                               strlcat(outlist,"k",MAXBUF);
-                                                               strlcpy(chan->key,"",MAXBUF);
-                                                               strlcpy(outpars[pc++],key,MAXBUF);
-                                                       }
-                                               }
-                                               else param++;
-                                       }
-                               break;
-                               
-                               case 'l':
-                                       if (mdir == 0)
-                                       {
-                                               if (previously_set_l)
-                                                       break;
-                                               previously_unset_l = true;
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'l', "", false, 0));
-                                                if (!MOD_RESULT)
-                                                {
-                                                       if (chan->limit)
+                                                       log(DEBUG,"Modetype match, calling handler");
+
+                                                       if (modehandlers[handler_id]->GetNumParams(adding))
                                                        {
-                                                               strcat(outlist,"l");
-                                                               chan->limit = 0;
+                                                               log(DEBUG,"ModeHandler* for this mode says it has parameters. pcnt=%d parameter_counter=%d",pcnt,parameter_counter);
+
+                                                               if (parameter_counter < pcnt)
+                                                               {
+                                                                       parameter = parameters[parameter_counter++];
+                                                               }
+                                                               else
+                                                               {
+                                                                       /* No parameter, continue to the next mode */
+                                                                       continue;
+                                                               }
                                                        }
-                                               }
-                                       }
-                                       
-                                       if ((param >= pcnt)) break;
-                                       if (mdir == 1)
-                                       {
-                                               if (l_set)
-                                                       break;
-                                               if (previously_unset_l)
-                                                       break;
-                                               previously_set_l = true;
-                                               bool invalid = false;
-                                               for (unsigned int i = 0; i < strlen(parameters[param]); i++)
-                                               {
-                                                       if ((parameters[param][i] < '0') || (parameters[param][i] > '9'))
+                                                       ModeAction ma = modehandlers[handler_id]->OnModeChange(user, targetuser, targetchannel, parameter, adding);
+                                                       if (ma == MODEACTION_ALLOW)
                                                        {
-                                                               invalid = true;
-                                                       }
-                                               }
-                                               if (atoi(parameters[param]) < 1)
-                                               {
-                                                       invalid = true;
-                                               }
+                                                               log(DEBUG,"ModeAction was allow");
 
-                                               if (invalid)
-                                                       break;
+                                                               /* We're about to output a valid mode letter - was there previously a pending state-change? */
+                                                               if (state_change)
+                                                               {
+                                                                       log(DEBUG,"Appending state change");
+                                                                       output_sequence.append(adding ? "+" : "-");
+                                                               }
+                                                               
+                                                               /* Add the mode letter */
+                                                               output_sequence = output_sequence + *modeletter;
+                                                               log(DEBUG,"Added mode letter to output sequence, sequence now: '%s'",output_sequence.c_str());
 
-                                                MOD_RESULT = 0;
-                                                FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'l', parameters[param], true, 1));
-                                                if (!MOD_RESULT)
-                                                {
-       
-                                                       chan->limit = atoi(parameters[param]);
-                                                       
-                                                       // reported by mech: large values cause underflow
-                                                       if (chan->limit < 0)
-                                                               chan->limit = 0x7FFF;
-                                               }
-                                                       
-                                               if (chan->limit)
-                                               {
-                                                       strlcat(outlist,"l",MAXBUF);
-                                                       strlcpy(outpars[pc++],parameters[param++],MAXBUF);
-                                                       l_set = true;
-                                               }
-                                       }
-                               break;
-                               
-                               case 'i':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'i', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                               if (mdir)
-                                               {
-                                                       if (!(chan->binarymodes & CM_INVITEONLY)) strlcat(outlist,"i",MAXBUF);
-                                                       chan->binarymodes |= CM_INVITEONLY;
-                                               }
-                                               else
-                                               {
-                                                       if (chan->binarymodes & CM_INVITEONLY) strlcat(outlist,"i",MAXBUF);
-                                                       chan->binarymodes &= ~CM_INVITEONLY;
-                                               }
-                                       }
-                               break;
-                               
-                               case 't':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 't', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                               if (mdir)
-                                                {
-                                                       if (!(chan->binarymodes & CM_TOPICLOCK)) strlcat(outlist,"t",MAXBUF);
-                                                        chan->binarymodes |= CM_TOPICLOCK;
-                                                }
-                                                else
-                                                {
-                                                       if (chan->binarymodes & CM_NOEXTERNAL) strlcat(outlist,"t",MAXBUF);
-                                                        chan->binarymodes &= ~CM_TOPICLOCK;
-                                                }
-                                       }
-                               break;
-                               
-                               case 'n':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'n', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                                if (mdir)
-                                                {
-                                                       if (!(chan->binarymodes & CM_NOEXTERNAL)) strlcat(outlist,"n",MAXBUF);
-                                                        chan->binarymodes |= CM_NOEXTERNAL;
-                                                }
-                                                else
-                                                {
-                                                       if (chan->binarymodes & CM_NOEXTERNAL) strlcat(outlist,"n",MAXBUF);
-                                                        chan->binarymodes &= ~CM_NOEXTERNAL;
-                                                }
-                                       }
-                               break;
-                               
-                               case 'm':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'm', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                                if (mdir)
-                                                {
-                                                        if (!(chan->binarymodes & CM_MODERATED)) strlcat(outlist,"m",MAXBUF);
-                                                        chan->binarymodes |= CM_MODERATED;
-                                                }
-                                                else
-                                                {
-                                                        if (chan->binarymodes & CM_MODERATED) strlcat(outlist,"m",MAXBUF);
-                                                        chan->binarymodes &= ~CM_MODERATED;
-                                                }
-                                       }
-                               break;
-                               
-                               case 's':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 's', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                                if (mdir)
-                                                {
-                                                        if (!(chan->binarymodes & CM_SECRET)) strlcat(outlist,"s",MAXBUF);
-                                                        chan->binarymodes |= CM_SECRET;
-                                                        if (chan->binarymodes & CM_PRIVATE)
-                                                        {
-                                                                chan->binarymodes &= ~CM_PRIVATE;
-                                                                if (mdir)
-                                                                {
-                                                                        strlcat(outlist,"-p+",MAXBUF);
-                                                                }
-                                                        }
-                                                }
-                                                else
-                                                {
-                                                        if (chan->binarymodes & CM_SECRET) strlcat(outlist,"s",MAXBUF);
-                                                        chan->binarymodes &= ~CM_SECRET;
-                                                }
-                                       }
-                               break;
-                               
-                               case 'p':
-                                        MOD_RESULT = 0;
-                                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, 'p', "", mdir, 0));
-                                        if (!MOD_RESULT)
-                                        {
-                                                if (mdir)
-                                                {
-                                                        if (!(chan->binarymodes & CM_PRIVATE)) strlcat(outlist,"p",MAXBUF);
-                                                        chan->binarymodes |= CM_PRIVATE;
-                                                        if (chan->binarymodes & CM_SECRET)
-                                                        {
-                                                                chan->binarymodes &= ~CM_SECRET;
-                                                                if (mdir)
-                                                                {
-                                                                        strlcat(outlist,"-s+",MAXBUF);
-                                                                }
-                                                        }
-                                                }
-                                                else
-                                                {
-                                                        if (chan->binarymodes & CM_PRIVATE) strlcat(outlist,"p",MAXBUF);
-                                                        chan->binarymodes &= ~CM_PRIVATE;
-                                                }
-                                       }
-                               break;
-                               
-                               default:
-                                       log(DEBUG,"Preprocessing custom mode %c: modelist: %s",modechar,chan->custom_modes);
-                                       string_list p;
-                                       p.clear();
-                                       if (((!strchr(chan->custom_modes,modechar)) && (!mdir)) || ((strchr(chan->custom_modes,modechar)) && (mdir)))
-                                       {
-                                               if (!ModeIsListMode(modechar,MT_CHANNEL))
-                                               {
-                                                       log(DEBUG,"Mode %c isnt set on %s but trying to remove!",modechar,chan->name);
-                                                       break;
-                                               }
-                                       }
-                                       if (ModeDefined(modechar,MT_CHANNEL))
-                                       {
-                                               log(DEBUG,"A module has claimed this mode");
-                                               if (param<pcnt)
-                                               {
-                                                       if ((ModeDefinedOn(modechar,MT_CHANNEL)>0) && (mdir))
-                                                       {
-                                                               p.push_back(parameters[param]);
-                                                       }
-                                                       if ((ModeDefinedOff(modechar,MT_CHANNEL)>0) && (!mdir))
-                                                       {
-                                                               p.push_back(parameters[param]);
-                                                       }
-                                               }
-                                               bool handled = false;
-                                               if (param>=pcnt)
-                                               {
-                                                       // we're supposed to have a parameter, but none was given... so dont handle the mode.
-                                                       if (((ModeDefinedOn(modechar,MT_CHANNEL)>0) && (mdir)) || ((ModeDefinedOff(modechar,MT_CHANNEL)>0) && (!mdir))) 
-                                                       {
-                                                               log(DEBUG,"Not enough parameters for module-mode %c",modechar);
-                                                               handled = true;
-                                                               param++;
-                                                       }
-                                               }
-
-                                               // BIG ASS IDIOTIC CODER WARNING!
-                                               // Using OnRawMode on another modules mode's behavour 
-                                               // will confuse the crap out of admins! just because you CAN
-                                               // do it, doesnt mean you SHOULD!
-                                               MOD_RESULT = 0;
-                                               std::string para = "";
-                                               if (p.size())
-                                                       para = p[0];
-                                               FOREACH_RESULT(I_OnRawMode,OnRawMode(user, chan, modechar, para, mdir, pcnt));
-                                               if (!MOD_RESULT)
-                                               {
-                                                       for (int i = 0; i <= MODCOUNT; i++)
-                                                       {
-                                                               if (!handled)
+                                                               /* Is there a valid parameter for this mode? If so add it to the parameter list */
+                                                               if ((modehandlers[handler_id]->GetNumParams(adding)) && (parameter != ""))
                                                                {
-                                                                       int t = modules[i]->OnExtendedMode(user,chan,modechar,MT_CHANNEL,mdir,p);
-                                                                       if (t != 0)
-                                                                       {
-                                                                               log(DEBUG,"OnExtendedMode returned nonzero for a module");
-                                                                               char app[] = {modechar, 0};
-                                                                               if (ModeIsListMode(modechar,MT_CHANNEL))
-                                                                               {
-                                                                                       if (t == -1)
-                                                                                       {
-                                                                                               //pc++;
-                                                                                               param++;
-                                                                                       }
-                                                                                       else
-                                                                                       {
-                                                                                               if (ptr>0)
-                                                                                               {
-                                                                                                       strlcat(outlist, app,MAXBUF);
-                                                                                               }
-                                                                                               strlcpy(outpars[pc++],parameters[param++],MAXBUF);
-                                                                                       }
-                                                                               }
-                                                                               else
-                                                                               {
-                                                                                       if (ptr>0)
-                                                                                       {
-                                                                                               if ((modelist[ptr-1] == '+') || (modelist[ptr-1] == '-'))
-                                                                                               {
-                                                                                                       strlcat(outlist, app,MAXBUF);
-                                                                                               }
-                                                                                               else if (!strchr(outlist,modechar))
-                                                                                               {
-                                                                                                       strlcat(outlist, app,MAXBUF);
-                                                                                               }
-                                                                                       }
-                                                                                       chan->SetCustomMode(modechar,mdir);
-                                                                                       // include parameters in output if mode has them
-                                                                                       if ((ModeDefinedOn(modechar,MT_CHANNEL)>0) && (mdir))
-                                                                                       {
-                                                                                               chan->SetCustomModeParam(modelist[ptr],parameters[param],mdir);
-                                                                                               strlcpy(outpars[pc++],parameters[param++],MAXBUF);
-                                                                                       }
-                                                                               }
-                                                                               // break, because only one module can handle the mode.
-                                                                               handled = true;
-                                                                               }
-                                                                       }
-                                                       }
+                                                                       log(DEBUG,"Added parameter to parameter_list, list now: '%s'",parameter_list.str().c_str());
+                                                                       parameter_list << " " << parameter;
+                                                               }
+
+                                                               /* Call all the AfterMode events in the mode watchers for this mode */
+                                                               for (ModeWatchIter watchers = modewatchers[handler_id].begin(); watchers != modewatchers[handler_id].end(); watchers++)
+                                                               {
+                                                                       log(DEBUG,"Called a ModeWatcher* after event");
+                                                                       (*watchers)->AfterMode(user, targetuser, targetchannel, parameter, adding, type);
+                                                               }
+
+                                                               /* Reset the state change flag */
+                                                               state_change = false;
+                                                       }
                                                }
-                                       }
-                                       else
-                                       {
-                                               WriteServ(user->fd,"472 %s %c :is unknown mode char to me",user->nick,modechar);
                                        }
                                break;
-                               
                        }
                }
-       }
-
-       /* this ensures only the *valid* modes are sent out onto the network */
-       int xt = strlen(outlist)-1;
-       while ((outlist[xt] == '-') || (outlist[xt] == '+'))
-       {
-               outlist[xt] = '\0';
-               xt = strlen(outlist)-1;
-       }
-       if (outlist[0])
-       {
-               strlcpy(outstr,outlist,MAXBUF);
-               for (ptr = 0; ptr < pc; ptr++)
-               {
-                       strlcat(outstr," ",MAXBUF);
-                       strlcat(outstr,outpars[ptr],MAXBUF);
-               }
-               if (local)
-               {
-                       log(DEBUG,"Local mode change");
-                       WriteChannelLocal(chan, user, "MODE %s %s",chan->name,outstr);
-                       FOREACH_MOD(I_OnMode,OnMode(user, chan, TYPE_CHANNEL, outstr));
-               }
-               else
+               /* Was there at least one valid mode in the sequence? */
+               if (output_sequence != "")
                {
                        if (servermode)
                        {
-                               if (!silent)
+                               if (type == MODETYPE_CHANNEL)
                                {
-                                       WriteChannelWithServ(Config->ServerName,chan,"MODE %s %s",chan->name,outstr);
+                                       WriteChannelWithServ(Config->ServerName,targetchannel,"MODE %s %s%s",targetchannel->name,output_sequence.c_str(),parameter_list.str().c_str());
                                }
-                                       
                        }
                        else
                        {
-                               if (!silent)
+                               if (type == MODETYPE_CHANNEL)
                                {
-                                       WriteChannel(chan,user,"MODE %s %s",chan->name,outstr);
-                                       FOREACH_MOD(I_OnMode,OnMode(user, chan, TYPE_CHANNEL, outstr));
+                                       log(DEBUG,"Write output sequence and parameters to channel: %s %s%s",targetchannel->name,output_sequence.c_str(),parameter_list.str().c_str());
+                                       WriteChannel(targetchannel,user,"MODE %s %s%s",targetchannel->name,output_sequence.c_str(),parameter_list.str().c_str());
+                                       FOREACH_MOD(I_OnMode,OnMode(user, targetchannel, TYPE_CHANNEL, output_sequence + parameter_list.str()));
                                }
                        }
                }
        }
 }
 
-// based on sourcemodes, return true or false to determine if umode is a valid mode a user may set on themselves or others.
-
-bool ModeParser::AllowedUmode(char umode, char* sourcemodes,bool adding,bool serveroverride)
-{
-       log(DEBUG,"Allowed_umode: %c %s",umode,sourcemodes);
-       // Servers can +o and -o arbitrarily
-       if ((serveroverride == true) && (umode == 'o'))
-       {
-               return true;
-       }
-       // RFC1459 specified modes
-       if ((umode == 'w') || (umode == 's') || (umode == 'i'))
-       {
-               log(DEBUG,"umode %c allowed by RFC1459 scemantics",umode);
-               return true;
-       }
-       
-       // user may not +o themselves or others, but an oper may de-oper other opers or themselves
-       if ((strchr(sourcemodes,'o')) && (!adding))
-       {
-               log(DEBUG,"umode %c allowed by RFC1459 scemantics",umode);
-               return true;
-       }
-       else if (umode == 'o')
-       {
-               log(DEBUG,"umode %c allowed by RFC1459 scemantics",umode);
-               return false;
-       }
-       
-       // process any module-defined modes that need oper
-       if ((ModeDefinedOper(umode,MT_CLIENT)) && (strchr(sourcemodes,'o')))
-       {
-               log(DEBUG,"umode %c allowed by module handler (oper only mode)",umode);
-               return true;
-       }
-       else
-       if (ModeDefined(umode,MT_CLIENT))
-       {
-               // process any module-defined modes that don't need oper
-               log(DEBUG,"umode %c allowed by module handler (non-oper mode)",umode);
-               if ((ModeDefinedOper(umode,MT_CLIENT)) && (!strchr(sourcemodes,'o')))
-               {
-                       // no, this mode needs oper, and this user 'aint got what it takes!
-                       return false;
-               }
-               return true;
-       }
-
-       // anything else - return false.
-       log(DEBUG,"umode %c not known by any ruleset",umode);
-       return false;
-}
-
-bool ModeParser::ProcessModuleUmode(char umode, userrec* source, void* dest, bool adding)
-{
-       userrec* s2;
-       bool faked = false;
-       if (!source)
-       {
-               s2 = new userrec;
-               strlcpy(s2->nick,Config->ServerName,NICKMAX);
-               strlcpy(s2->modes,"o",52);
-               s2->fd = -1;
-               source = s2;
-               faked = true;
-       }
-       string_list p;
-       p.clear();
-       if (ModeDefined(umode,MT_CLIENT))
-       {
-               for (int i = 0; i <= MODCOUNT; i++)
-               {
-                       if (modules[i]->OnExtendedMode(source,(void*)dest,umode,MT_CLIENT,adding,p))
-                       {
-                               log(DEBUG,"Module %s claims umode %c",Config->module_names[i].c_str(),umode);
-                               return true;
-                       }
-               }
-               log(DEBUG,"No module claims umode %c",umode);
-               if (faked)
-               {
-                       delete s2;
-                       source = NULL;
-               }
-               return false;
-       }
-       else
-       {
-               if (faked)
-               {
-                       delete s2;
-                       source = NULL;
-               }
-               return false;
-       }
-}
 
 void cmd_mode::Handle (char **parameters, int pcnt, userrec *user)
 {
-       chanrec* Ptr;
-       userrec* dest;
-       int can_change;
-       int direction = 1;
-       char outpars[MAXBUF];
-
-       dest = Find(parameters[0]);
-
        if (!user)
-       {
                return;
-       }
-
-       if ((dest) && (pcnt == 1))
-       {
-               WriteServ(user->fd,"221 %s :+%s",dest->nick,dest->modes);
-               return;
-       }
 
-       if ((dest) && (pcnt > 1))
-       {
-               std::string tidied = ServerInstance->ModeGrok->CompressModes(parameters[1],false);
-               parameters[1] = (char*)tidied.c_str();
+       ServerInstance->ModeGrok->Process(parameters, pcnt, user, false);
 
-               char dmodes[MAXBUF];
-               strlcpy(dmodes,dest->modes,52);
-               log(DEBUG,"pulled up dest user modes: %s",dmodes);
-
-               can_change = 0;
-               if (user != dest)
-               {
-                       if ((strchr(user->modes,'o')) || (is_uline(user->server)))
-                       {
-                               can_change = 1;
-                       }
-               }
-               else
-               {
-                       can_change = 1;
-               }
-               if (!can_change)
-               {
-                       WriteServ(user->fd,"482 %s :Can't change mode for other users",user->nick);
-                       return;
-               }
-               
-               outpars[0] = '+';
-               outpars[1] = 0;
-               direction = 1;
-
-               if ((parameters[1][0] != '+') && (parameters[1][0] != '-'))
-                       return;
-
-               for (unsigned int i = 0; i < strlen(parameters[1]); i++)
-               {
-                       if (parameters[1][i] == ' ')
-                               continue;
-                       if (parameters[1][i] == '+')
-                       {
-                               if (direction != 1)
-                               {
-                                       int t = strlen(outpars)-1;
-                                       if ((outpars[t] == '+') || (outpars[t] == '-'))
-                                       {
-                                               outpars[t] = '+';
-                                       }
-                                       else
-                                       {
-                                               strcat(outpars,"+");
-                                       }
-                               }
-                               direction = 1;
-                       }
-                       else
-                       if (parameters[1][i] == '-')
-                       {
-                               if (direction != 0)
-                               {
-                                       int t = strlen(outpars)-1;
-                                       if ((outpars[t] == '+') || (outpars[t] == '-'))
-                                       {
-                                               outpars[t] = '-';
-                                       }
-                                       else
-                                       {
-                                               strcat(outpars,"-");
-                                       }
-                               }
-                               direction = 0;
-                       }
-                       else
-                       {
-                               can_change = 0;
-                               if (strchr(user->modes,'o'))
-                               {
-                                       can_change = 1;
-                               }
-                               else
-                               {
-                                       if ((parameters[1][i] == 'i') || (parameters[1][i] == 'w') || (parameters[1][i] == 's') || (ServerInstance->ModeGrok->AllowedUmode(parameters[1][i],user->modes,direction,false)))
-                                       {
-                                               can_change = 1;
-                                       }
-                               }
-                               if (can_change)
-                               {
-                                       if (direction == 1)
-                                       {
-                                               if ((!strchr(dmodes,parameters[1][i])) && (ServerInstance->ModeGrok->AllowedUmode(parameters[1][i],user->modes,true,false)))
-                                               {
-                                                       char umode = parameters[1][i];
-                                                       if ((ServerInstance->ModeGrok->ProcessModuleUmode(umode, user, dest, direction)) || (umode == 'i') || (umode == 's') || (umode == 'w') || (umode == 'o'))
-                                                       {
-                                                               int q = strlen(dmodes);
-                                                               int r = strlen(outpars);
-                                                               dmodes[q+1]='\0';
-                                                               dmodes[q] = parameters[1][i];
-                                                               outpars[r+1]='\0';
-                                                               outpars[r] = parameters[1][i];
-                                                               if (parameters[1][i] == 'o')
-                                                               {
-                                                                       FOREACH_MOD(I_OnGlobalOper,OnGlobalOper(dest));
-                                                               }
-                                                       }
-                                               }
-                                       }
-                                       else
-                                       {
-                                               if ((ServerInstance->ModeGrok->AllowedUmode(parameters[1][i],user->modes,false,false)) && (strchr(dmodes,parameters[1][i])))
-                                               {
-                                                       char umode = parameters[1][i];
-                                                       if ((ServerInstance->ModeGrok->ProcessModuleUmode(umode, user, dest, direction)) || (umode == 'i') || (umode == 's') || (umode == 'w') || (umode == 'o'))
-                                                       {
-                                                               unsigned int q = 0;
-                                                               char temp[MAXBUF];      
-                                                               char moo[MAXBUF];       
-
-                                                               unsigned int r = strlen(outpars);
-                                                               outpars[r+1]='\0';
-                                                               outpars[r] = parameters[1][i];
-                                                       
-                                                               *temp = 0;
-                                                               for (q = 0; q < strlen(dmodes); q++)
-                                                               {
-                                                                       if (dmodes[q] != parameters[1][i])
-                                                                       {
-                                                                               moo[0] = dmodes[q];
-                                                                               moo[1] = '\0';
-                                                                               strlcat(temp,moo,MAXBUF);
-                                                                       }
-                                                               }
-                                                               strlcpy(dmodes,temp,52);
-
-                                                               if (umode == 'o')
-                                                                       DeleteOper(dest);
-                                                       }
-                                               }
-                                       }
-                               }
-                       }
-               }
-               if (outpars[0])
-               {
-                       char b[MAXBUF];
-                       strlcpy(b,"",MAXBUF);
-                       unsigned int z = 0;
-                       unsigned int i = 0;
-                       while (i < strlen (outpars))
-                       {
-                               b[z++] = outpars[i++];
-                               b[z] = '\0';
-                               if (i<strlen(outpars)-1)
-                               {
-                                       if (((outpars[i] == '-') || (outpars[i] == '+')) && ((outpars[i+1] == '-') || (outpars[i+1] == '+')))
-                                       {
-                                               // someones playing silly buggers and trying
-                                               // to put a +- or -+ into the line...
-                                               i++;
-                                       }
-                               }
-                               if (i == strlen(outpars)-1)
-                               {
-                                       if ((outpars[i] == '-') || (outpars[i] == '+'))
-                                       {
-                                               i++;
-                                       }
-                               }
-                       }
-
-                       z = strlen(b)-1;
-                       if ((b[z] == '-') || (b[z] == '+'))
-                               b[z] = '\0';
-
-                       if ((!b[0]) || (!strcmp(b,"+")) || (!strcmp(b,"-")))
-                               return;
-
-                       if (strcmp(b,""))
-                       {
-                               WriteTo(user, dest, "MODE %s :%s", dest->nick, b);
-                               FOREACH_MOD(I_OnMode,OnMode(user, dest, TYPE_USER, b));
-                       }
-
-                       if (strlen(dmodes)>MAXMODES)
-                       {
-                               dmodes[MAXMODES-1] = '\0';
-                       }
-                       log(DEBUG,"Stripped mode line");
-                       log(DEBUG,"Line dest is now %s",dmodes);
-                       strlcpy(dest->modes,dmodes,52);
+       return;
+}
 
-               }
+void ModeParser::CleanMask(std::string &mask)
+{
+       std::string::size_type pos_of_pling = mask.find_first_of('!');
+       std::string::size_type pos_of_at = mask.find_first_of('@');
+       std::string::size_type pos_of_dot = mask.find_first_of('.');
+       std::string::size_type pos_of_colon = mask.find_first_of(':'); /* Because ipv6 addresses are colon delimited */
 
-               return;
-       }
-       
-       Ptr = FindChan(parameters[0]);
-       if (Ptr)
+       if ((pos_of_pling == std::string::npos) && (pos_of_at == std::string::npos))
        {
-               if (pcnt == 1)
+               /* Just a nick, or just a host */
+               if ((pos_of_dot == std::string::npos) && (pos_of_colon == std::string::npos))
                {
-                       /* just /modes #channel */
-                       WriteServ(user->fd,"324 %s %s +%s",user->nick, Ptr->name, chanmodes(Ptr));
-                        WriteServ(user->fd,"329 %s %s %d", user->nick, Ptr->name, Ptr->created);
-                       return;
+                       /* It has no '.' in it, it must be a nick. */
+                       mask.append("!*@*");
                }
                else
-               if (pcnt == 2)
                {
-                       char* mode = parameters[1];
-                       if (*mode == '+')
-                               mode++;
-                       int MOD_RESULT = 0;
-                        FOREACH_RESULT(I_OnRawMode,OnRawMode(user, Ptr, *mode, "", false, 0));
-                        if (!MOD_RESULT)
-                        {
-                               if (*mode == 'b')
-                               {
-
-                                       for (BanList::iterator i = Ptr->bans.begin(); i != Ptr->bans.end(); i++)
-                                       {
-                                               WriteServ(user->fd,"367 %s %s %s %s %d",user->nick, Ptr->name, i->data, i->set_by, i->set_time);
-                                       }
-                                       WriteServ(user->fd,"368 %s %s :End of channel ban list",user->nick, Ptr->name);
-                                       return;
-                               }
-                               if ((ModeDefined(*mode,MT_CHANNEL)) && (ModeIsListMode(*mode,MT_CHANNEL)))
-                               {
-                                       // list of items for an extmode
-                                       log(DEBUG,"Calling OnSendList for all modules, list output for mode %c",*mode);
-                                       FOREACH_MOD(I_OnSendList,OnSendList(user,Ptr,*mode));
-                                       return;
-                               }
-                       }
-               }
-
-                if (((Ptr) && (!has_channel(user,Ptr))) && (!is_uline(user->server)))
-                {
-                        WriteServ(user->fd,"442 %s %s :You're not on that channel!",user->nick, Ptr->name);
-                        return;
-                }
-
-               if (Ptr)
-               {
-                       int MOD_RESULT = 0;
-                       FOREACH_RESULT(I_OnAccessCheck,OnAccessCheck(user,NULL,Ptr,AC_GENERAL_MODE));
-                       
-                       if (MOD_RESULT == ACR_DENY)
-                               return;
-                       if (MOD_RESULT == ACR_DEFAULT)
-                       {
-                               if (cstatus(user,Ptr) < STATUS_HOP)
-                               {
-                                       WriteServ(user->fd,"482 %s %s :You must be at least a half-operator to change modes on this channel",user->nick, Ptr->name);
-                                       return;
-                               }
-                       }
-
-                       ServerInstance->ModeGrok->ProcessModes(parameters,user,Ptr,cstatus(user,Ptr),pcnt,false,false,false);
+                       /* Got a dot in it? Has to be a host */
+                       mask = "*!*@" + mask;
                }
        }
-       else
+       else if ((pos_of_pling == std::string::npos) && (pos_of_at != std::string::npos))
        {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, parameters[0]);
+               /* Has an @ but no !, its a user@host */
+                mask = "*!" + mask;
        }
-}
-
-
-
-
-void ModeParser::ServerMode(char **parameters, int pcnt, userrec *user)
-{
-       chanrec* Ptr;
-       userrec* dest;
-       int can_change;
-       int direction = 1;
-       char outpars[MAXBUF];
-
-       dest = Find(parameters[0]);
-       
-       // fix: ChroNiCk found this - we cant use this as debug if its null!
-       if (dest)
+       else if ((pos_of_pling != std::string::npos) && (pos_of_at == std::string::npos))
        {
-               log(DEBUG,"server_mode on %s",dest->nick);
+               /* Has a ! but no @, it must be a nick!ident */
+               mask.append("@*");
        }
 
-       if ((dest) && (pcnt > 1))
+       /* Check for dumb stuff like *@*!*
+        * swap the two items over so that at least the n!u@h ordering
+        * is correct even if the elements may not be
+        */
+       if (pos_of_pling > pos_of_at)
        {
-                std::string tidied = ServerInstance->ModeGrok->CompressModes(parameters[1],false);
-                parameters[1] = (char*)tidied.c_str();
-
-               char dmodes[MAXBUF];
-               strlcpy(dmodes,dest->modes,52);
-
-               outpars[0] = '+';
-               outpars[1] = 0;
-               direction = 1;
-
-               if ((parameters[1][0] != '+') && (parameters[1][0] != '-'))
-                       return;
-
-               for (unsigned int i = 0; i < strlen(parameters[1]); i++)
-               {
-                        if (parameters[1][i] == ' ')
-                                continue;
-                       if (parameters[1][i] == '+')
-                       {
-                               if (direction != 1)
-                               {
-                                       int t = strlen(outpars)-1;
-                                       if ((outpars[t] == '+') || (outpars[t] == '-'))
-                                       {
-                                               outpars[t] = '+';
-                                       }
-                                       else
-                                       {
-                                               strcat(outpars,"+");
-                                       }
-                               }
-                               direction = 1;
-                       }
-                       else
-                       if (parameters[1][i] == '-')
-                       {
-                               if (direction != 0)
-                               {
-                                       int t = strlen(outpars)-1;
-                                       if ((outpars[t] == '+') || (outpars[t] == '-'))
-                                       {
-                                               outpars[t] = '-';
-                                       }
-                                       else
-                                       {
-                                               strcat(outpars,"-");
-                                       }
-                               }
-                               direction = 0;
-                       }
-                       else
-                       {
-                               log(DEBUG,"begin mode processing entry");
-                               can_change = 1;
-                               if (can_change)
-                               {
-                                       if (direction == 1)
-                                       {
-                                               log(DEBUG,"umode %c being added",parameters[1][i]);
-                                               if ((!strchr(dmodes,parameters[1][i])) && (ServerInstance->ModeGrok->AllowedUmode(parameters[1][i],user->modes,true,true)))
-                                               {
-                                                       char umode = parameters[1][i];
-                                                       log(DEBUG,"umode %c is an allowed umode",umode);
-                                                       if ((ServerInstance->ModeGrok->ProcessModuleUmode(umode, user, dest, direction)) || (umode == 'i') || (umode == 's') || (umode == 'w') || (umode == 'o'))
-                                                       {
-                                                               int v1 = strlen(dmodes);
-                                                               int v2 = strlen(outpars);
-                                                               dmodes[v1+1]='\0';
-                                                               dmodes[v1] = parameters[1][i];
-                                                               outpars[v2+1]='\0';
-                                                               outpars[v2] = parameters[1][i];
-                                                       }
-                                               }
-                                       }
-                                       else
-                                       {
-                                               // can only remove a mode they already have
-                                               log(DEBUG,"umode %c being removed",parameters[1][i]);
-                                               if ((ServerInstance->ModeGrok->AllowedUmode(parameters[1][i],user->modes,false,true)) && (strchr(dmodes,parameters[1][i])))
-                                               {
-                                                       char umode = parameters[1][i];
-                                                       log(DEBUG,"umode %c is an allowed umode",umode);
-                                                       if ((ServerInstance->ModeGrok->ProcessModuleUmode(umode, user, dest, direction)) || (umode == 'i') || (umode == 's') || (umode == 'w') || (umode == 'o'))
-                                                       {
-                                                               unsigned int q = 0;
-                                                               char temp[MAXBUF];
-                                                               char moo[MAXBUF];       
-
-                                                               unsigned int v1 = strlen(outpars);
-                                                               outpars[v1+1]='\0';
-                                                               outpars[v1] = parameters[1][i];
-                                                               *temp = 0;
-                                                               for (q = 0; q < strlen(dmodes); q++)
-                                                               {
-                                                                       if (dmodes[q] != parameters[1][i])
-                                                                       {
-                                                                               moo[0] = dmodes[q];
-                                                                               moo[1] = '\0';
-                                                                               strlcat(temp,moo,MAXBUF);
-                                                                       }
-                                                               }
-                                                               strlcpy(dmodes,temp,52);
-                                                       }
-                                               }
-                                       }
-                               }
-                       }
-               }
-               if (outpars[0])
-               {
-                       char b[MAXBUF];
-                       strlcpy(b,"",MAXBUF);
-                       unsigned int z = 0;
-                       unsigned int i = 0;
-                       while (i < strlen (outpars))
-                       {
-                               b[z++] = outpars[i++];
-                               b[z] = '\0';
-                               if (i<strlen(outpars)-1)
-                               {
-                                       if (((outpars[i] == '-') || (outpars[i] == '+')) && ((outpars[i+1] == '-') || (outpars[i+1] == '+')))
-                                       {
-                                               // someones playing silly buggers and trying
-                                               // to put a +- or -+ into the line...
-                                               i++;
-                                       }
-                               }
-                               if (i == strlen(outpars)-1)
-                               {
-                                       if ((outpars[i] == '-') || (outpars[i] == '+'))
-                                       {
-                                               i++;
-                                       }
-                               }
-                       }
+               mask.replace(pos_of_pling, 1, "@");
+               mask.replace(pos_of_at, 1, "!");
+       }
+}
 
-                       z = strlen(b)-1;
-                       if ((b[z] == '-') || (b[z] == '+'))
-                               b[z] = '\0';
+bool ModeParser::AddMode(ModeHandler* mh, unsigned const char modeletter)
+{
+       unsigned char mask = 0;
+       unsigned char pos = 0;
+
+       /* Yes, i know, this might let people declare modes like '_' or '^'.
+        * If they do that, thats their problem, and if i ever EVER see an
+        * official InspIRCd developer do that, i'll beat them with a paddle!
+        */
+       if ((modeletter < 'A') || (modeletter > 'z'))
+               return false;
 
-                       if ((!b[0]) || (!strcmp(b,"+")) || (!strcmp(b,"-")))
-                               return;
+       mh->GetModeType() == MODETYPE_USER ? mask = MASK_USER : mask = MASK_CHANNEL;
+       pos = (modeletter-65) | mask;
 
-                       if (strcmp(b,""))
-                       {
-                               WriteTo(user, dest, "MODE %s :%s", dest->nick, b);
-                               FOREACH_MOD(I_OnMode,OnMode(user, dest, TYPE_USER, b));
-                       }
-                       
-                       if (strlen(dmodes)>MAXMODES)
-                       {
-                               dmodes[MAXMODES-1] = '\0';
-                       }
-                       log(DEBUG,"Stripped mode line");
-                       log(DEBUG,"Line dest is now %s",dmodes);
-                       strlcpy(dest->modes,dmodes,MAXMODES);
+       if (modehandlers[pos])
+               return false;
 
-               }
+       modehandlers[pos] = mh;
+       log(DEBUG,"ModeParser::AddMode: added mode %c",modeletter);
+       return true;
+}
 
-               return;
-       }
-       
-       Ptr = FindChan(parameters[0]);
-       if (Ptr)
-       {
-               ServerInstance->ModeGrok->ProcessModes(parameters,user,Ptr,STATUS_OP,pcnt,true,false,false);
-       }
-       else
-       {
-               WriteServ(user->fd,"401 %s %s :No such nick/channel",user->nick, parameters[0]);
-       }
+ModeParser::ModeParser()
+{
+       /* Clear mode list */
+       memset(modehandlers, 0, sizeof(modehandlers));
+       memset(modewatchers, 0, sizeof(modewatchers));
+
+       /* Initialise the RFC mode letters */
+       this->AddMode(new ModeChannelSecret, 's');
+       this->AddMode(new ModeChannelPrivate, 'p');
+       this->AddMode(new ModeChannelBan, 'b');
+       this->AddMode(new ModeChannelModerated, 'm');
 }
+