PROPAGATE this fix

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

diff --git a/src/modules/m_operlevels.cpp b/src/modules/m_operlevels.cpp
index 2793241117a1bf8450c9a57267fe52163a2f96cd..4c4047caae8cc3b4676c55de4fcbf2c5a1298e9b 100644 (file)
--- a/src/modules/m_operlevels.cpp
+++ b/src/modules/m_operlevels.cpp
@@ -1,9 +1,15 @@
-using namespace std;
-
-#include "users.h"
-#include "channels.h"
-#include "modules.h"
-#include <string>
+/*       +------------------------------------+
+ *       | Inspire Internet Relay Chat Daemon |
+ *       +------------------------------------+
+ *
+ *  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.
+ *
+ * ---------------------------------------------------
+ */
 
 #include "inspircd.h"
 
@@ -22,7 +28,7 @@ class ModuleOperLevels : public Module
        public:
 
                ModuleOperLevels(InspIRCd* Me)
-                       : Module::Module(Me)
+                       : Module(Me)
                {
 
                        
@@ -39,7 +45,7 @@ class ModuleOperLevels : public Module
                        List[I_OnRehash] = List[I_OnKill] = 1;
                }
 
-               virtual void OnRehash(const std::string &parameter)
+               virtual void OnRehash(userrec* user, const std::string &parameter)
                {
                        DELETE(conf);
                        conf = new ConfigReader(ServerInstance);
@@ -47,14 +53,15 @@ class ModuleOperLevels : public Module
 
                virtual Version GetVersion()
                {
-                       return Version(1,0,0,1,VF_VENDOR,API_VERSION);
+                       return Version(1,1,0,1,VF_VENDOR,API_VERSION);
                }
 
                virtual int OnKill(userrec* source, userrec* dest, const std::string &reason)
                {
                        long dest_level = 0,source_level = 0;
+
                        // oper killing an oper?
-                       if (*dest->oper)
+                       if (IS_OPER(dest) && IS_OPER(source))
                        {
                                for (int j =0; j < conf->Enumerate("type"); j++)
                                {
@@ -78,7 +85,7 @@ class ModuleOperLevels : public Module
                                {
                                        ServerInstance->WriteOpers("Oper %s (level %d) attempted to /kill a higher oper: %s (level %d): Reason: %s",source->nick,source_level,dest->nick,dest_level,reason.c_str());
                                        dest->WriteServ("NOTICE %s :Oper %s attempted to /kill you!",dest->nick,source->nick);
-                                       source->WriteServ("481 %s :Permission Denied- Oper %s is a higher level than you",source->nick,dest->nick);
+                                       source->WriteServ("481 %s :Permission Denied - Oper %s is a higher level than you",source->nick,dest->nick);
                                        return 1;
                                }
                        }
@@ -87,26 +94,5 @@ class ModuleOperLevels : public Module
 
 };
 
-class ModuleOperLevelsFactory : public ModuleFactory
-{
- public:
-       ModuleOperLevelsFactory()
-       {
-       }
-
-       ~ModuleOperLevelsFactory()
-       {
-       }
-
-       virtual Module * CreateModule(InspIRCd* Me)
-       {
-               return new ModuleOperLevels(Me);
-       }
-
-};
-
-extern "C" void * init_module( void )
-{
-       return new ModuleOperLevelsFactory;
-}
+MODULE_INIT(ModuleOperLevels)