X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_redirect.cpp;h=5fc97d3c5bd577a5b1dd439555f6e57afb542715;hb=357d190074ee58809b31ea0c08543566168bddf6;hp=735b27adaf1ffe15d4de018738810925910f8402;hpb=abc57eddfb56462ac3e433601d010abf1942e611;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_redirect.cpp b/src/modules/m_redirect.cpp
index 735b27ada..5fc97d3c5 100644
--- a/src/modules/m_redirect.cpp
+++ b/src/modules/m_redirect.cpp
@@ -39,8 +39,7 @@ class Redirect : public ModeHandler
 			{
 				if (!ServerInstance->IsChannel(parameter))
 				{
-					source->WriteNumeric(403, "%s %s :Invalid channel name", source->nick.c_str(), parameter.c_str());
-					parameter.clear();
+					source->WriteNumeric(ERR_NOSUCHCHANNEL, "%s :Invalid channel name", parameter.c_str());
 					return MODEACTION_DENY;
 				}
 			}
@@ -50,14 +49,12 @@ class Redirect : public ModeHandler
 				Channel* c = ServerInstance->FindChan(parameter);
 				if (!c)
 				{
-					source->WriteNumeric(690, "%s :Target channel %s must exist to be set as a redirect.",source->nick.c_str(),parameter.c_str());
-					parameter.clear();
+					source->WriteNumeric(690, ":Target channel %s must exist to be set as a redirect.",parameter.c_str());
 					return MODEACTION_DENY;
 				}
 				else if (c->GetPrefixValue(source) < OP_VALUE)
 				{
-					source->WriteNumeric(690, "%s :You must be opped on %s to set it as a redirect.",source->nick.c_str(),parameter.c_str());
-					parameter.clear();
+					source->WriteNumeric(690, ":You must be opped on %s to set it as a redirect.",parameter.c_str());
 					return MODEACTION_DENY;
 				}
 			}
@@ -88,7 +85,11 @@ class Redirect : public ModeHandler
 class AntiRedirect : public SimpleUserModeHandler
 {
 	public:
-		AntiRedirect(Module* Creator) : SimpleUserModeHandler(Creator, "antiredirect", 'L') {}
+		AntiRedirect(Module* Creator) : SimpleUserModeHandler(Creator, "antiredirect", 'L')
+		{
+			if (!ServerInstance->Config->ConfValue("redirect")->getBool("antiredirect"))
+				DisableAutoRegister();
+		}
 };
 
 class ModuleRedirect : public Module
@@ -110,22 +111,6 @@ class ModuleRedirect : public Module
 	{
 		/* Setting this here so it isn't changable by rehasing the config later. */
 		UseUsermode = ServerInstance->Config->ConfValue("redirect")->getBool("antiredirect");
-
-		/* Channel mode */
-		ServerInstance->Modules->AddService(re);
-
-		/* Check to see if the usermode is enabled in the config */
-		if (UseUsermode)
-		{
-			/* Log noting that this breaks compatability. */
-			ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "REDIRECT: Enabled usermode +L. This breaks linking with servers that do not have this enabled. This is disabled by default in the 2.0 branch but will be enabled in the next version.");
-
-			/* Try to add the usermode */
-			ServerInstance->Modules->AddService(re_u);
-		}
-
-		Implementation eventlist[] = { I_OnUserPreJoin };
-		ServerInstance->Modules->Attach(eventlist, this, sizeof(eventlist)/sizeof(Implementation));
 	}
 
 	ModResult OnUserPreJoin(LocalUser* user, Channel* chan, const std::string& cname, std::string& privs, const std::string& keygiven) CXX11_OVERRIDE
@@ -142,19 +127,19 @@ class ModuleRedirect : public Module
 					Channel* destchan = ServerInstance->FindChan(channel);
 					if (destchan && destchan->IsModeSet(re))
 					{
-						user->WriteNumeric(470, "%s %s * :You may not join this channel. A redirect is set, but you may not be redirected as it is a circular loop.", user->nick.c_str(), cname.c_str());
+						user->WriteNumeric(470, "%s * :You may not join this channel. A redirect is set, but you may not be redirected as it is a circular loop.", cname.c_str());
 						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))
 					{
-						user->WriteNumeric(470, "%s %s %s :Force redirection stopped.", user->nick.c_str(), cname.c_str(), channel.c_str());
+						user->WriteNumeric(470, "%s %s :Force redirection stopped.", cname.c_str(), channel.c_str());
 						return MOD_RES_DENY;
 					}
 					else
 					{
-						user->WriteNumeric(470, "%s %s %s :You may not join this channel, so you are automatically being transferred to the redirect channel.", user->nick.c_str(), cname.c_str(), channel.c_str());
+						user->WriteNumeric(470, "%s %s :You may not join this channel, so you are automatically being transferred to the redirect channel.", cname.c_str(), channel.c_str());
 						Channel::JoinUser(user, channel);
 						return MOD_RES_DENY;
 					}