X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_timedbans.cpp;h=497ac2569d94ad80528656a1c4a1de345ab339d4;hb=8d172c077e41e08e3864615538c6b06f07f24d84;hp=5023f4c01bd9a695e9c5712ac222eff9dc79a577;hpb=84a1569cd60daa64b1ae52a1fff62c0dc4d78850;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp
index 5023f4c01..497ac2569 100644
--- a/src/modules/m_timedbans.cpp
+++ b/src/modules/m_timedbans.cpp
@@ -62,12 +62,8 @@ class CommandTban : public Command
 			user->WriteNumeric(482, "%s %s :You do not have permission to set bans on this channel",
 				user->nick.c_str(), channel->name.c_str());
 			return CMD_FAILURE;
-		}
-		if (!ServerInstance->IsValidMask(parameters[2]))
-		{
-			user->WriteServ("NOTICE "+user->nick+" :Invalid ban mask");
-			return CMD_FAILURE;
-		}
+		}		
+
 		TimedBan T;
 		std::string channelname = parameters[0];
 		long duration = ServerInstance->Duration(parameters[1]);
@@ -81,7 +77,15 @@ class CommandTban : public Command
 		std::vector<std::string> setban;
 		setban.push_back(parameters[0]);
 		setban.push_back("+b");
-		setban.push_back(parameters[2]);
+		bool isextban = ((mask.size() > 2) && (mask[1] == ':'));
+		if (!isextban && !ServerInstance->IsValidMask(mask))
+			mask.append("!*@*");
+		if ((mask.length() > 250) || (!ServerInstance->IsValidMask(mask) && !isextban))
+		{
+			user->WriteServ("NOTICE "+user->nick+" :Invalid ban mask");
+			return CMD_FAILURE;
+		}
+		setban.push_back(mask);
 		// use CallHandler to make it so that the user sets the mode
 		// themselves
 		ServerInstance->Parser->CallHandler("MODE",setban,user);
@@ -95,7 +99,12 @@ found:
 		T.mask = mask;
 		T.expire = expire + (IS_REMOTE(user) ? 5 : 0);
 		TimedBanList.push_back(T);
-		channel->WriteAllExcept(ServerInstance->FakeClient, true, '@', tmp, "NOTICE %s :%s added a timed ban on %s lasting for %ld seconds.", channel->name.c_str(), user->nick.c_str(), mask.c_str(), duration);
+
+		// If halfop is loaded, send notice to halfops and above, otherwise send to ops and above
+		ModeHandler* mh = ServerInstance->Modes->FindMode('h', MODETYPE_CHANNEL);
+		char pfxchar = (mh && mh->name == "halfop") ? '%' : '@';
+
+		channel->WriteAllExcept(ServerInstance->FakeClient, true, pfxchar, tmp, "NOTICE %s :%s added a timed ban on %s lasting for %ld seconds.", channel->name.c_str(), user->nick.c_str(), mask.c_str(), duration);
 		return CMD_SUCCESS;
 	}
 
@@ -116,7 +125,7 @@ class ModuleTimedBans : public Module
 
 	void init()
 	{
-		ServerInstance->AddCommand(&cmd);
+		ServerInstance->Modules->AddService(cmd);
 		Implementation eventlist[] = { I_OnDelBan, I_OnBackgroundTimer };
 		ServerInstance->Modules->Attach(eventlist, this, sizeof(eventlist)/sizeof(Implementation));
 	}