Make rehash work more than once per run, and fix some uninitialized values in connect...

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

diff --git a/src/modules/m_censor.cpp b/src/modules/m_censor.cpp
index c980a1f454eaa67a7418c6d3f075b139d60f442b..0f1c3f61c72a0cfc3dd73053363516867305538f 100644 (file)
--- a/src/modules/m_censor.cpp
+++ b/src/modules/m_censor.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -49,7 +49,7 @@ class ModuleCensor : public Module
        {
                /* Read the configuration file on startup.
                 */
-               OnRehash(NULL,"");
+               OnRehash(NULL);
                cu = new CensorUser(ServerInstance);
                cc = new CensorChannel(ServerInstance);
                if (!ServerInstance->Modes->AddMode(cu) || !ServerInstance->Modes->AddMode(cc))
@@ -71,49 +71,6 @@ class ModuleCensor : public Module
                delete cc;
        }
 
-       virtual void OnRunTestSuite()
-       {
-               std::cout << "Test suite for m_censor:" << std::endl;
-
-               irc::string text = "original text";
-               irc::string pattern = "text";
-               irc::string replace = "new";
-               std::cout << (ReplaceLine(text, pattern, replace) == "original new" ? "\nSUCCESS!\n" : "\nFAILURE '" + text + "' \n");
-               text = "original text here";
-               pattern = "text";
-               replace = "texts";
-               std::cout << (ReplaceLine(text, pattern, replace) == "original texts here" ? "\nSUCCESS!\n" : "\nFAILURE: '" + text + "' \n");
-               text = "original text";
-               pattern = "original";
-               replace = "new";
-               std::cout << (ReplaceLine(text, pattern, replace) == "new text" ? "\nSUCCESS!\n" : "\nFAILURE '" + text + "' \n");
-               std::cout << "end of test suite for m_censor" << std::endl;
-       }
-
-       /* This version of ReplaceLine won't loop forever if the replacement string contains the source pattern */
-       virtual const irc::string& ReplaceLine(irc::string &text, irc::string pattern, irc::string replace)
-       {
-               irc::string replacement;
-               if ((!pattern.empty()) && (!text.empty()))
-               {
-                       for (std::string::size_type n = 0; n != text.length(); ++n)
-                       {
-                               if (text.length() >= pattern.length() && text.substr(n, pattern.length()) == pattern)
-                               {
-                                       /* Found the pattern in the text, replace it, and advance */
-                                       replacement.append(replace);
-                                       n = n + pattern.length() - 1;
-                               }
-                               else
-                               {
-                                       replacement += text[n];
-                               }
-                       }
-               }
-               text = replacement;
-               return text;
-       }
-
        // format of a config entry is <badword text="shit" replace="poo">
        virtual int OnUserPreMessage(User* user,void* dest,int target_type, std::string &text, char status, CUList &exempt_list)
        {
@@ -148,7 +105,7 @@ class ModuleCensor : public Module
                                        return 1;
                                }
 
-                               this->ReplaceLine(text2,index->first,index->second);
+                               SearchAndReplace(text2, index->first, index->second);
                        }
                }
                text = text2.c_str();
@@ -160,7 +117,7 @@ class ModuleCensor : public Module
                return OnUserPreMessage(user,dest,target_type,text,status,exempt_list);
        }
 
-       virtual void OnRehash(User* user, const std::string &parameter)
+       virtual void OnRehash(User* user)
        {
                /*
                 * reload our config file on rehash - we must destroy and re-allocate the classes