Fix silly oversight discovered by tra26 (thanks!) where the core tries to handle...

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

diff --git a/src/modules/m_nokicks.cpp b/src/modules/m_nokicks.cpp
index de6eab1c7e9221065605955eea9a0cc15f970331..bb5db683c5a958dff2c77ad769afbe3b0119ffb7 100644 (file)
--- a/src/modules/m_nokicks.cpp
+++ b/src/modules/m_nokicks.cpp
@@ -23,28 +23,29 @@ class NoKicks : public SimpleChannelModeHandler
 
 class ModuleNoKicks : public Module
 {
-
        NoKicks* nk;
 
  public:
-
        ModuleNoKicks(InspIRCd* Me)
                : Module(Me)
        {
-
                nk = new NoKicks(ServerInstance);
                if (!ServerInstance->Modes->AddMode(nk))
                        throw ModuleException("Could not add new modes!");
-               Implementation eventlist[] = { I_OnAccessCheck };
-               ServerInstance->Modules->Attach(eventlist, this, 1);
+               Implementation eventlist[] = { I_OnAccessCheck, I_On005Numeric };
+               ServerInstance->Modules->Attach(eventlist, this, 2);
        }
 
+       virtual void On005Numeric(std::string &output)
+       {
+               ServerInstance->AddExtBanChar('Q');
+       }
 
        virtual int OnAccessCheck(User* source,User* dest,Channel* channel,int access_type)
        {
                if (access_type == AC_KICK)
                {
-                       if (channel->IsModeSet('Q'))
+                       if (channel->IsModeSet('Q') || channel->IsExtBanned(source, 'Q'))
                        {
                                if ((ServerInstance->ULine(source->nick.c_str())) || (ServerInstance->ULine(source->server)) || (!*source->server))
                                {
@@ -70,7 +71,7 @@ class ModuleNoKicks : public Module
 
        virtual Version GetVersion()
        {
-               return Version(1,2,0,0,VF_COMMON|VF_VENDOR,API_VERSION);
+               return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
        }
 };