Get rid of a bunch of memory-wasting C-style strings

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

diff --git a/src/modules/m_hidechans.cpp b/src/modules/m_hidechans.cpp
index 2637968a9d9750097c929c3aca4efefa027bcc16..0d405186b0e60e18aca512c7bbf1e9471c6b4c54 100644 (file)
--- a/src/modules/m_hidechans.cpp
+++ b/src/modules/m_hidechans.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2007 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.
@@ -20,14 +20,10 @@
 class HideChans : public ModeHandler
 {
  public:
-       HideChans(InspIRCd* Instance) : ModeHandler(Instance, 'I', 0, 0, false, MODETYPE_USER, false) { }
+       HideChans(Module* Creator) : ModeHandler(Creator, 'I', PARAM_NONE, MODETYPE_USER) { }
 
        ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string &parameter, bool adding)
        {
-               /* Only opers can change other users modes */
-               if (source != dest)
-                       return MODEACTION_DENY;
-
                if (adding)
                {
                        if (!dest->IsModeSet('I'))
@@ -44,47 +40,65 @@ class HideChans : public ModeHandler
                                return MODEACTION_ALLOW;
                        }
                }
-               
+
                return MODEACTION_DENY;
        }
 };
 
 class ModuleHideChans : public Module
 {
-       
-       HideChans* hm;
+       bool AffectsOpers;
+       HideChans hm;
  public:
-       ModuleHideChans(InspIRCd* Me)
-               : Module(Me)
+       ModuleHideChans() : hm(this)
        {
-               
-               hm = new HideChans(ServerInstance);
-               if (!ServerInstance->AddMode(hm, 'I'))
+               if (!ServerInstance->Modes->AddMode(&hm))
                        throw ModuleException("Could not add new modes!");
+               Implementation eventlist[] = { I_OnWhoisLine, I_OnRehash };
+               ServerInstance->Modules->Attach(eventlist, this, 2);
+               OnRehash(NULL);
        }
 
-       void Implements(char* List)
-       {
-               List[I_OnWhoisLine] = 1;
-       }
-       
        virtual ~ModuleHideChans()
        {
-               ServerInstance->Modes->DelMode(hm);
-               DELETE(hm);
+               ServerInstance->Modes->DelMode(&hm);
        }
-       
+
        virtual Version GetVersion()
        {
-               return Version(1,1,0,0,VF_COMMON|VF_VENDOR,API_VERSION);
+               return Version("Provides support for hiding channels with user mode +I", VF_COMMON | VF_VENDOR, API_VERSION);
        }
 
-       int OnWhoisLine(User* user, User* dest, int &numeric, std::string &text)
+       virtual void OnRehash(User* user)
        {
-               /* Dont display channels if they have +I set and the
-                * person doing the WHOIS is not an oper
-                */
-               return ((user != dest) && (!IS_OPER(user)) && (numeric == 319) && dest->IsModeSet('I'));
+               ConfigReader conf;
+               AffectsOpers = conf.ReadFlag("hidechans", "affectsopers", 0);
+       }
+
+       ModResult OnWhoisLine(User* user, User* dest, int &numeric, std::string &text)
+       {
+               /* always show to self */
+               if (user == dest)
+                       return MOD_RES_PASSTHRU;
+
+               /* don't touch anything except 319 */
+               if (numeric != 319)
+                       return MOD_RES_PASSTHRU;
+
+               /* don't touch if -I */
+               if (!dest->IsModeSet('I'))
+                       return MOD_RES_PASSTHRU;
+
+               /* if it affects opers, we don't care if they are opered */
+               if (AffectsOpers)
+                       return MOD_RES_DENY;
+
+               /* doesn't affect opers, sender is opered */
+               if (user->HasPrivPermission("users/auspex"))
+                       return MOD_RES_PASSTHRU;
+
+               /* user must be opered, boned. */
+               return MOD_RES_DENY;
        }
 };