Move the <disabled> tag out of the core to a new module.

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

diff --git a/src/modules/m_redirect.cpp b/src/modules/m_redirect.cpp
index b14de9ff91eeecda7d8eda473c4a858f2354db7f..b71a2f3db608570e591876f46fa4af83ebb27901 100644 (file)
--- a/src/modules/m_redirect.cpp
+++ b/src/modules/m_redirect.cpp
@@ -32,13 +32,13 @@ class Redirect : public ParamMode<Redirect, LocalStringExt>
        Redirect(Module* Creator)
                : ParamMode<Redirect, LocalStringExt>(Creator, "redirect", 'L') { }
 
-       ModeAction OnSet(User* source, Channel* channel, std::string& parameter)
+       ModeAction OnSet(User* source, Channel* channel, std::string& parameter) CXX11_OVERRIDE
        {
                if (IS_LOCAL(source))
                {
                        if (!ServerInstance->IsChannel(parameter))
                        {
-                               source->WriteNumeric(ERR_NOSUCHCHANNEL, parameter, "Invalid channel name");
+                               source->WriteNumeric(Numerics::NoSuchChannel(parameter));
                                return MODEACTION_DENY;
                        }
                }
@@ -72,46 +72,27 @@ class Redirect : public ParamMode<Redirect, LocalStringExt>
        }
 };
 
-/** Handles usermode +L to stop forced redirection and print an error.
-*/
-class AntiRedirect : public SimpleUserModeHandler
-{
-       public:
-               AntiRedirect(Module* Creator) : SimpleUserModeHandler(Creator, "antiredirect", 'L')
-               {
-                       if (!ServerInstance->Config->ConfValue("redirect")->getBool("antiredirect"))
-                               DisableAutoRegister();
-               }
-};
-
 class ModuleRedirect : public Module
 {
        Redirect re;
-       AntiRedirect re_u;
+       SimpleUserModeHandler antiredirectmode;
        ChanModeReference limitmode;
-       bool UseUsermode;
 
  public:
        ModuleRedirect()
                : re(this)
-               , re_u(this)
+               , antiredirectmode(this, "antiredirect", 'L')
                , limitmode(this, "limit")
        {
        }
 
-       void init() CXX11_OVERRIDE
-       {
-               /* Setting this here so it isn't changable by rehasing the config later. */
-               UseUsermode = ServerInstance->Config->ConfValue("redirect")->getBool("antiredirect");
-       }
-
        ModResult OnUserPreJoin(LocalUser* user, Channel* chan, const std::string& cname, std::string& privs, const std::string& keygiven) CXX11_OVERRIDE
        {
                if (chan)
                {
                        if (chan->IsModeSet(re) && chan->IsModeSet(limitmode))
                        {
-                               if (chan->GetUserCounter() >= ConvToInt(chan->GetModeParameter(limitmode)))
+                               if (chan->GetUserCounter() >= ConvToNum<size_t>(chan->GetModeParameter(limitmode)))
                                {
                                        const std::string& channel = *re.ext.get(chan);
 
@@ -122,9 +103,8 @@ class ModuleRedirect : public Module
                                                user->WriteNumeric(470, cname, '*', "You may not join this channel. A redirect is set, but you may not be redirected as it is a circular loop.");
                                                return MOD_RES_DENY;
                                        }
-                                       /* We check the bool value here to make sure we have it enabled, if we don't then
-                                               usermode +L might be assigned to something else. */
-                                       if (UseUsermode && user->IsModeSet(re_u))
+
+                                       if (user->IsModeSet(antiredirectmode))
                                        {
                                                user->WriteNumeric(470, cname, channel, "Force redirection stopped.");
                                                return MOD_RES_DENY;