Pedantic clean

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

diff --git a/src/modules/m_operjoin.cpp b/src/modules/m_operjoin.cpp
index 76ba00ef21d90c12edb0810f49cd5a3bce102241..ae92a12551eed88c6766f61aad20d1e40a54e8a2 100644 (file)
--- a/src/modules/m_operjoin.cpp
+++ b/src/modules/m_operjoin.cpp
@@ -11,22 +11,15 @@
  * ---------------------------------------------------
  */
 
-#include "users.h"
-#include "channels.h"
-#include "modules.h"
-
 #include "inspircd.h"
 
 /* $ModDesc: Forces opers to join the specified channel(s) on oper-up */
 
-
-
 class ModuleOperjoin : public Module
 {
        private:
                std::string operChan;
-               ConfigReader* conf;
-               
+               std::vector<std::string> operChans;             
 
                int tokenize(const string &str, std::vector<std::string> &tokens)
                {
@@ -48,12 +41,9 @@ class ModuleOperjoin : public Module
                }
 
        public:
-               ModuleOperjoin(InspIRCd* Me)
-                       : Module::Module(Me)
+               ModuleOperjoin(InspIRCd* Me) : Module(Me)
                {
-                       
-                       conf = new ConfigReader(ServerInstance);
-                       operChan = conf->ReadValue("operjoin", "channel", 0);
+                       OnRehash(NULL, "");
                }
 
                void Implements(char* List)
@@ -61,15 +51,20 @@ class ModuleOperjoin : public Module
                        List[I_OnPostOper] = List[I_OnRehash] = 1;
                }
 
-               virtual void OnRehash(const std::string &parameter)
+               virtual void OnRehash(User* user, const std::string &parameter)
                {
-                       DELETE(conf);
-                       conf = new ConfigReader(ServerInstance);
+                       ConfigReader* conf = new ConfigReader(ServerInstance);
+
+                       operChan = conf->ReadValue("operjoin", "channel", 0);
+                       operChans.clear();
+                       if (!operChan.empty())
+                               tokenize(operChan,operChans);
+
+                       delete conf;
                }
 
                virtual ~ModuleOperjoin()
                {
-                       DELETE(conf);
                }
 
                virtual Version GetVersion()
@@ -77,38 +72,16 @@ class ModuleOperjoin : public Module
                        return Version(1,1,0,1,VF_VENDOR,API_VERSION);
                }
 
-               virtual void OnPostOper(userrec* user, const std::string &opertype)
-               {
-                       if(operChan != "")
-                       {
-                               std::vector<std::string> operChans;
-                               tokenize(operChan,operChans);
-                               for(std::vector<std::string>::iterator it = operChans.begin(); it != operChans.end(); it++)
-                                       chanrec::JoinUser(ServerInstance, user, it->c_str(), false);
-                       }
-
-               }
-
-};
-
-class ModuleOperjoinFactory : public ModuleFactory
-{
-       public:
-               ModuleOperjoinFactory()
+               virtual void OnPostOper(User* user, const std::string &opertype)
                {
-               }
+                       if (!IS_LOCAL(user))
+                               return;
 
-               ~ModuleOperjoinFactory()
-               {
+                       for(std::vector<std::string>::iterator it = operChans.begin(); it != operChans.end(); it++)
+                               if (ServerInstance->IsChannel(it->c_str()))
+                                       Channel::JoinUser(ServerInstance, user, it->c_str(), false, "", ServerInstance->Time(true));
                }
 
-               virtual Module * CreateModule(InspIRCd* Me)
-               {
-                       return new ModuleOperjoin(Me);
-               }
 };
 
-extern "C" void * init_module( void )
-{
-       return new ModuleOperjoinFactory;
-}
+MODULE_INIT(ModuleOperjoin)