X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_redirect.cpp;h=b71a2f3db608570e591876f46fa4af83ebb27901;hb=3d0d64933da3d37866fadfa042e34a1125315db6;hp=b14de9ff91eeecda7d8eda473c4a858f2354db7f;hpb=761e6d75ba37b984998952940ed681e79e456142;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_redirect.cpp b/src/modules/m_redirect.cpp
index b14de9ff9..b71a2f3db 100644
--- 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;