Change to use std::string::iterator rather than making a copy of the pointer and...

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

diff --git a/src/modules/m_knock.cpp b/src/modules/m_knock.cpp
index 0121f04f4e130e09863b85570e62a5cbc0695394..5018899d208533773711db7e1e01a268872164bd 100644 (file)
--- a/src/modules/m_knock.cpp
+++ b/src/modules/m_knock.cpp
@@ -2,50 +2,44 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *               <Craig@chatspike.net>
- *     
- * Written by Craig Edwards, Craig McLure, and others.
+ *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ * See: http://www.inspircd.org/wiki/index.php/Credits
+ *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
  *
  * ---------------------------------------------------
  */
 
-using namespace std;
-
 #include <stdio.h>
 #include <string>
 #include "users.h"
 #include "channels.h"
 #include "modules.h"
-#include "helperfuncs.h"
 #include "configreader.h"
 #include "inspircd.h"
 
 /* $ModDesc: Provides support for /KNOCK and mode +K */
 
-extern InspIRCd* ServerInstance;
-
+/** Handles the /KNOCK command
+ */
 class cmd_knock : public command_t
 {
  public:
- cmd_knock (InspIRCd* Instance) : command_t(Instance,"KNOCK", 0, 2)
+       cmd_knock (InspIRCd* Instance) : command_t(Instance,"KNOCK", 0, 2)
        {
                this->source = "m_knock.so";
                syntax = "<channel> <reason>";
        }
        
-       void Handle (const char** parameters, int pcnt, userrec *user)
+       CmdResult Handle (const char** parameters, int pcnt, userrec *user)
        {
                chanrec* c = ServerInstance->FindChan(parameters[0]);
 
                if (!c)
                {
                        user->WriteServ("401 %s %s :No such channel",user->nick, parameters[0]);
-                       return;
+                       return CMD_FAILURE;
                }
 
                std::string line = "";
@@ -53,7 +47,7 @@ class cmd_knock : public command_t
                if (c->IsModeSet('K'))
                {
                        user->WriteServ("480 %s :Can't KNOCK on %s, +K is set.",user->nick, c->name);
-                       return;
+                       return CMD_FAILURE;
                }
 
                for (int i = 1; i < pcnt - 1; i++)
@@ -66,16 +60,20 @@ class cmd_knock : public command_t
                {
                        c->WriteChannelWithServ((char*)ServerInstance->Config->ServerName,  "NOTICE %s :User %s is KNOCKing on %s (%s)", c->name, user->nick, c->name, line.c_str());
                        user->WriteServ("NOTICE %s :KNOCKing on %s",user->nick,c->name);
-                       return;
+                       return CMD_SUCCESS;
                }
                else
                {
                        user->WriteServ("480 %s :Can't KNOCK on %s, channel is not invite only so knocking is pointless!",user->nick, c->name);
-                       return;
+                       return CMD_FAILURE;
                }
+
+               return CMD_SUCCESS;
        }
 };
 
+/** Handles channel mode +K
+ */
 class Knock : public ModeHandler
 {
  public:
@@ -120,22 +118,17 @@ class ModuleKnock : public Module
 
        void Implements(char* List)
        {
-               List[I_On005Numeric] = 1;
-       }
-
-       virtual void On005Numeric(std::string &output)
-       {
-               ServerInstance->ModeGrok->InsertMode(output,"K",4);
        }
 
        virtual ~ModuleKnock()
        {
+               ServerInstance->Modes->DelMode(kn);
                DELETE(kn);
        }
 
        virtual Version GetVersion()
        {
-               return Version(1,0,0,1,VF_STATIC|VF_VENDOR);
+               return Version(1,1,0,1,VF_COMMON|VF_VENDOR,API_VERSION);
        }
 };