Add {To,From}{Human,Internal,Network} to ExtensionItem.

[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 479dd6d0f38c6715371d166143567c4393fa829b..3d10663aa4557d6dd23e5ac351ef733127a52563 100644 (file)
--- a/src/modules/m_redirect.cpp
+++ b/src/modules/m_redirect.cpp
@@ -30,7 +30,10 @@ class Redirect : public ParamMode<Redirect, LocalStringExt>
 {
  public:
        Redirect(Module* Creator)
-               : ParamMode<Redirect, LocalStringExt>(Creator, "redirect", 'L') { }
+               : ParamMode<Redirect, LocalStringExt>(Creator, "redirect", 'L')
+       {
+               syntax = "<target>";
+       }
 
        ModeAction OnSet(User* source, Channel* channel, std::string& parameter) CXX11_OVERRIDE
        {
@@ -38,7 +41,7 @@ class Redirect : public ParamMode<Redirect, LocalStringExt>
                {
                        if (!ServerInstance->IsChannel(parameter))
                        {
-                               source->WriteNumeric(ERR_NOSUCHCHANNEL, parameter, "Invalid channel name");
+                               source->WriteNumeric(Numerics::NoSuchChannel(parameter));
                                return MODEACTION_DENY;
                        }
                }
@@ -72,39 +75,20 @@ 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)
@@ -122,16 +106,15 @@ 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;
                                        }
                                        else
                                        {
-                                               user->WriteNumeric(470, cname, channel, "You may not join this channel, so you are automatically being transferred to the redirect channel.");
+                                               user->WriteNumeric(470, cname, channel, "You may not join this channel, so you are automatically being transferred to the redirected channel.");
                                                Channel::JoinUser(user, channel);
                                                return MOD_RES_DENY;
                                        }