]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_chanprotect.cpp
Add option to allow +q to +q users, +a to +a users.
[user/henk/code/inspircd.git] / src / modules / m_chanprotect.cpp
index cd84d9c5f2629cc5cb497a58ef8dc03a2fbbc809..a88f5de384a71cbad4f24a2160a666c507918961 100644 (file)
@@ -31,9 +31,10 @@ class FounderProtectBase
  protected:
        bool& remove_own_privs;
        bool& remove_other_privs;
+       bool& add_other_privs;
  public:
-       FounderProtectBase(InspIRCd* Instance, const std::string &ext, const std::string &mtype, int l, int e, bool &remove_own, bool &remove_others) :
-               MyInstance(Instance), extend(ext), type(mtype), list(l), end(e), remove_own_privs(remove_own), remove_other_privs(remove_others)
+       FounderProtectBase(InspIRCd* Instance, const std::string &ext, const std::string &mtype, int l, int e, bool &remove_own, bool &remove_others, bool &add_others) :
+               MyInstance(Instance), extend(ext), type(mtype), list(l), end(e), remove_own_privs(remove_own), remove_other_privs(remove_others), add_other_privs(add_others)
        {
        }
 
@@ -123,7 +124,13 @@ class FounderProtectBase
        bool CanRemoveOthers(User* u1, User* u2, Channel* c)
        {
                std::string item = extend+std::string(c->name);
-               return (u1->GetExt(item) && u2->GetExt(item));
+               return (remove_other_privs && u1->GetExt(item) && u2->GetExt(item));
+       }
+
+       bool CanAddOthers(User* u1, User* u2, Channel* c)
+       {
+               std::string item = extend+std::string(c->name);
+               return (add_other_privs && u1->GetExt(item));
        }
 
        ModeAction HandleChange(User* source, User* theuser, bool adding, Channel* channel, std::string &parameter)
@@ -157,9 +164,9 @@ class FounderProtectBase
 class ChanFounder : public ModeHandler, public FounderProtectBase
 {
  public:
-       ChanFounder(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others)
+       ChanFounder(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others, bool &priv_others)
                : ModeHandler(Instance, 'q', 1, 1, true, MODETYPE_CHANNEL, false, my_prefix, 0),
-                 FounderProtectBase(Instance, "cm_founder_", "founder", 386, 387, depriv_self, depriv_others) { }
+                 FounderProtectBase(Instance, "cm_founder_", "founder", 386, 387, depriv_self, depriv_others, priv_others) { }
 
        unsigned int GetPrefixRank()
        {
@@ -203,13 +210,17 @@ class ChanFounder : public ModeHandler, public FounderProtectBase
                {
                        return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
                }
-               else
+
+               // If they have it, allow them to give it.
+               if (adding && FounderProtectBase::CanAddOthers(source, theuser, channel))
                {
-                       // whoops, someones being naughty!
-                       source->WriteNumeric(468, "%s %s :Only servers may set channel mode +q", source->nick.c_str(), channel->name.c_str());
-                       parameter.clear();
-                       return MODEACTION_DENY;
+                       return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
                }
+
+               // whoops, someones being naughty!
+               source->WriteNumeric(468, "%s %s :You are not permitted to set additional founders", source->nick.c_str(), channel->name.c_str());
+               parameter.clear();
+               return MODEACTION_DENY;
        }
 
        void DisplayList(User* user, Channel* channel)
@@ -223,9 +234,9 @@ class ChanFounder : public ModeHandler, public FounderProtectBase
 class ChanProtect : public ModeHandler, public FounderProtectBase
 {
  public:
-       ChanProtect(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others)
+       ChanProtect(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others, bool &priv_others)
                : ModeHandler(Instance, 'a', 1, 1, true, MODETYPE_CHANNEL, false, my_prefix, 0),
-                 FounderProtectBase(Instance,"cm_protect_","protected user", 388, 389, depriv_self, depriv_others) { }
+                 FounderProtectBase(Instance,"cm_protect_","protected user", 388, 389, depriv_self, depriv_others, priv_others) { }
 
        unsigned int GetPrefixRank()
        {
@@ -255,10 +266,11 @@ class ChanProtect : public ModeHandler, public FounderProtectBase
 
                std::string founder = "cm_founder_"+std::string(channel->name);
 
-               if ((!adding) && FounderProtectBase::CanRemoveOthers(source, theuser, channel))
+               if (!adding && FounderProtectBase::CanRemoveOthers(source, theuser, channel))
                {
                        return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
                }
+
                // source has +q, is a server, or ulined, we'll let them +-a the user.
                if (source == ServerInstance->FakeClient ||
                        ((source == theuser) && (!adding) && (FounderProtectBase::remove_own_privs)) ||
@@ -270,12 +282,16 @@ class ChanProtect : public ModeHandler, public FounderProtectBase
                {
                        return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
                }
-               else
+
+               // If they have it, allow them to give it.
+               if (adding && FounderProtectBase::CanAddOthers(source, theuser, channel))
                {
-                       // bzzzt, wrong answer!
-                       source->WriteNumeric(482, "%s %s :You are not a channel founder", source->nick.c_str(), channel->name.c_str());
-                       return MODEACTION_DENY;
+                       return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
                }
+               
+               // bzzzt, wrong answer!
+               source->WriteNumeric(482, "%s %s :You are not a channel founder", source->nick.c_str(), channel->name.c_str());
+               return MODEACTION_DENY;
        }
 
        virtual void DisplayList(User* user, Channel* channel)
@@ -293,6 +309,7 @@ class ModuleChanProtect : public Module
        char APrefix;
        bool DeprivSelf;
        bool DeprivOthers;
+       bool PrivOthers;
        bool booting;
        ChanProtect* cp;
        ChanFounder* cf;
@@ -308,8 +325,8 @@ class ModuleChanProtect : public Module
 
                /* Initialise module variables */
 
-               cp = new ChanProtect(ServerInstance, APrefix, DeprivSelf, DeprivOthers);
-               cf = new ChanFounder(ServerInstance, QPrefix, DeprivSelf, DeprivOthers);
+               cp = new ChanProtect(ServerInstance, APrefix, DeprivSelf, DeprivOthers, PrivOthers);
+               cf = new ChanFounder(ServerInstance, QPrefix, DeprivSelf, DeprivOthers, PrivOthers);
 
                if (!ServerInstance->Modes->AddMode(cp) || !ServerInstance->Modes->AddMode(cf))
                {
@@ -329,7 +346,7 @@ class ModuleChanProtect : public Module
                user->Shrink("cm_protect_"+std::string(chan->name));
        }
 
-       virtual void OnUserPart(User* user, Channel* channel, const std::string &partreason, bool &silent)
+       virtual void OnUserPart(User* user, Channel* channel, std::string &partreason, bool &silent)
        {
                // FIX: when someone parts a channel we must remove their Extensibles!
                user->Shrink("cm_founder_"+std::string(channel->name));
@@ -362,8 +379,9 @@ class ModuleChanProtect : public Module
                if (cf && ServerInstance->Modes->FindPrefix(QPrefix) == cf)
                        throw ModuleException("Looks like the +q prefix you picked for m_chanprotect is already in use. Pick another.");
 
-               DeprivSelf = Conf.ReadFlag("options","deprotectself",0);
-               DeprivOthers = Conf.ReadFlag("options","deprotectothers",0);
+               DeprivSelf = Conf.ReadFlag("options","deprotectself", "yes", 0);
+               DeprivOthers = Conf.ReadFlag("options","deprotectothers", "yes", 0);
+               DeprivOthers = Conf.ReadFlag("options","setprivsonothers", "yes", 0);
        }
 
        virtual int OnUserPreJoin(User *user, Channel *chan, const char *cname, std::string &privs, const std::string &keygiven)
@@ -372,7 +390,7 @@ class ModuleChanProtect : public Module
                // the config option for it is set
 
                if (FirstInGetsFounder && !chan)
-                       privs = QPrefix + "@";
+                       privs = std::string(1, QPrefix) + "@";
 
                return 0;
        }
@@ -398,11 +416,10 @@ class ModuleChanProtect : public Module
                // a relatively small number of them relevent to our module using a switch statement.
                // don't allow action if:
                // (A) Theyre founder (no matter what)
-               // (B) Theyre protected, and you're not
+               // (B) Theyre protected, unless you're founder or are protected and DeprivOthers is enabled
                // always allow the action if:
                // (A) The source is ulined
 
-
                // firstly, if a ulined nick, or a server, is setting the mode, then allow them to set the mode
                // without any access checks, we're not worthy :p
                if ((ServerInstance->ULine(source->nick.c_str())) || (ServerInstance->ULine(source->server)) || (!*source->server))
@@ -411,16 +428,23 @@ class ModuleChanProtect : public Module
                std::string founder("cm_founder_"+channel->name);
                std::string protect("cm_protect_"+channel->name);
 
+               // Can do anything to yourself if deprotectself is enabled.
+               if (DeprivSelf && source == dest)
+                       return ACR_DEFAULT;
+
+               bool candepriv_founder = (DeprivOthers && source->GetExt(founder));
+               bool candepriv_protected = (source->GetExt(founder) || (DeprivOthers && source->GetExt(protect))); // Can the source remove +a?
+
                switch (access_type)
                {
                        // a user has been deopped. Do we let them? hmmm...
                        case AC_DEOP:
-                               if (dest->GetExt(founder))
+                               if (dest->GetExt(founder) && !candepriv_founder)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't deop "+dest->nick+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt(protect)) && (!source->GetExt(protect)))
+                               if ((dest->GetExt(protect)) && !candepriv_protected)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't deop "+dest->nick+" as they're protected (+a)");
                                        return ACR_DENY;
@@ -429,12 +453,12 @@ class ModuleChanProtect : public Module
 
                        // a user is being kicked. do we chop off the end of the army boot?
                        case AC_KICK:
-                               if (dest->GetExt(founder))
+                               if (dest->GetExt(founder) && !candepriv_founder)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't kick "+dest->nick+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt(protect)) && (!source->GetExt(protect)))
+                               if ((dest->GetExt(protect)) && !candepriv_protected)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't kick "+dest->nick+" as they're protected (+a)");
                                        return ACR_DENY;
@@ -443,12 +467,12 @@ class ModuleChanProtect : public Module
 
                        // a user is being dehalfopped. Yes, we do disallow -h of a +ha user
                        case AC_DEHALFOP:
-                               if (dest->GetExt(founder))
+                               if (dest->GetExt(founder) && !candepriv_founder)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't de-halfop "+dest->nick+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt(protect)) && (!source->GetExt(protect)))
+                               if ((dest->GetExt(protect)) && !candepriv_protected)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't de-halfop "+dest->nick+" as they're protected (+a)");
                                        return ACR_DENY;
@@ -457,12 +481,12 @@ class ModuleChanProtect : public Module
 
                        // same with devoice.
                        case AC_DEVOICE:
-                               if (dest->GetExt(founder))
+                               if (dest->GetExt(founder) && !candepriv_founder)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't devoice "+dest->nick+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt(protect)) && (!source->GetExt(protect)))
+                               if ((dest->GetExt(protect)) && !candepriv_protected)
                                {
                                        source->WriteNumeric(484, source->nick+" "+channel->name+" :Can't devoice "+dest->nick+" as they're protected (+a)");
                                        return ACR_DENY;
@@ -484,7 +508,7 @@ class ModuleChanProtect : public Module
 
        virtual Version GetVersion()
        {
-               return Version(1, 2, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
+               return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
        }
 };