X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_timedbans.cpp;h=d290ec18d260831f55a8100d02d7db03f2508685;hb=cb4c516ace8fef75b8a54a141c3644af9697ac0a;hp=c0816419f1f1880a9c5a3f6dad7ee2fcf5232c64;hpb=6d03943426dcce76ba66567a9b18425a5ebb4c0c;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp
index c0816419f..d290ec18d 100644
--- a/src/modules/m_timedbans.cpp
+++ b/src/modules/m_timedbans.cpp
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
  * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
@@ -17,7 +17,7 @@
 
 /** Holds a timed ban
  */
-class TimedBan : public classbase
+class TimedBan
 {
  public:
 	std::string channel;
@@ -42,70 +42,53 @@ class CommandTban : public Command
 	CmdResult Handle (const std::vector<std::string> &parameters, User *user)
 	{
 		Channel* channel = ServerInstance->FindChan(parameters[0]);
-		if (channel)
+		if (!channel)
 		{
-			int cm = channel->GetPrefixValue(user);
-			if ((cm == HALFOP_VALUE) || (cm == OP_VALUE))
-			{
-				if (!ServerInstance->IsValidMask(parameters[2]))
-				{
-					user->WriteServ("NOTICE "+std::string(user->nick)+" :Invalid ban mask");
-					return CMD_FAILURE;
-				}
-				for (BanList::iterator i = channel->bans.begin(); i != channel->bans.end(); i++)
-				{
-					if (!strcasecmp(i->data.c_str(), parameters[2].c_str()))
-					{
-						user->WriteServ("NOTICE "+std::string(user->nick)+" :The ban "+parameters[2]+" is already on the banlist of "+parameters[0]);
-						return CMD_FAILURE;
-					}
-				}
-				TimedBan T;
-				std::string channelname = parameters[0];
-				long duration = ServerInstance->Duration(parameters[1]);
-				unsigned long expire = duration + ServerInstance->Time();
-				if (duration < 1)
-				{
-					user->WriteServ("NOTICE "+std::string(user->nick)+" :Invalid ban time");
-					return CMD_FAILURE;
-				}
-				std::string mask = parameters[2];
-				std::vector<std::string> setban;
-				setban.push_back(parameters[0]);
-				setban.push_back("+b");
-				setban.push_back(parameters[2]);
-				// use CallCommandHandler to make it so that the user sets the mode
-				// themselves
-				ServerInstance->CallCommandHandler("MODE",setban,user);
-				/* Check if the ban was actually added (e.g. banlist was NOT full) */
-				bool was_added = false;
-				for (BanList::iterator i = channel->bans.begin(); i != channel->bans.end(); i++)
-					if (!strcasecmp(i->data.c_str(), mask.c_str()))
-						was_added = true;
-				if (was_added)
-				{
-					CUList tmp;
-					T.channel = channelname;
-					T.mask = mask;
-					T.expire = expire;
-					TimedBanList.push_back(T);
-					channel->WriteAllExcept(user, 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);
-					ServerInstance->PI->SendChannelNotice(channel, '@', user->nick + " added a timed ban on " + mask + " lasting for " + ConvToStr(duration) + " seconds.");
-					if (ServerInstance->Config->AllowHalfop)
-					{
-						channel->WriteAllExcept(user, 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);
-						ServerInstance->PI->SendChannelNotice(channel, '%', user->nick + " added a timed ban on " + mask + " lasting for " + ConvToStr(duration) + " seconds.");
-					}
-					return CMD_SUCCESS;
-				}
-				return CMD_FAILURE;
-			}
-			else user->WriteNumeric(482, "%s %s :You must be at least a%soperator to change modes on this channel",user->nick.c_str(), channel->name.c_str(),
-					ServerInstance->Config->AllowHalfop ? " half-" : " channel ");
+			user->WriteNumeric(401, "%s %s :No such channel",user->nick.c_str(), parameters[0].c_str());
 			return CMD_FAILURE;
 		}
-		user->WriteNumeric(401, "%s %s :No such channel",user->nick.c_str(), parameters[0].c_str());
+		int cm = channel->GetPrefixValue(user);
+		if (cm < HALFOP_VALUE)
+		{
+			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 "+std::string(user->nick)+" :Invalid ban mask");
+			return CMD_FAILURE;
+		}
+		TimedBan T;
+		std::string channelname = parameters[0];
+		long duration = ServerInstance->Duration(parameters[1]);
+		unsigned long expire = duration + ServerInstance->Time();
+		if (duration < 1)
+		{
+			user->WriteServ("NOTICE "+std::string(user->nick)+" :Invalid ban time");
+			return CMD_FAILURE;
+		}
+		std::string mask = parameters[2];
+		std::vector<std::string> setban;
+		setban.push_back(parameters[0]);
+		setban.push_back("+b");
+		setban.push_back(parameters[2]);
+		// use CallCommandHandler to make it so that the user sets the mode
+		// themselves
+		ServerInstance->CallCommandHandler("MODE",setban,user);
+		for (BanList::iterator i = channel->bans.begin(); i != channel->bans.end(); i++)
+			if (!strcasecmp(i->data.c_str(), mask.c_str()))
+				goto found;
 		return CMD_FAILURE;
+found:
+		CUList tmp;
+		T.channel = channelname;
+		T.mask = mask;
+		T.expire = expire;
+		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);
+		ServerInstance->PI->SendChannelNotice(channel, '@', user->nick + " added a timed ban on " + mask + " lasting for " + ConvToStr(duration) + " seconds.");
+		return CMD_SUCCESS;
 	}
 
 	RouteDescriptor GetRouting(User* user, const std::vector<std::string>& parameters)
@@ -170,14 +153,8 @@ class ModuleTimedBans : public Module
 					std::string expiry = "*** Timed ban on " + chan + " expired.";
 					cr->WriteAllExcept(ServerInstance->FakeClient, true, '@', empty, "NOTICE %s :%s", cr->name.c_str(), expiry.c_str());
 					ServerInstance->PI->SendChannelNotice(cr, '@', expiry);
-					if (ServerInstance->Config->AllowHalfop)
-					{
-						cr->WriteAllExcept(ServerInstance->FakeClient, true, '%', empty, "NOTICE %s :%s", cr->name.c_str(), expiry.c_str());
-						ServerInstance->PI->SendChannelNotice(cr, '%', expiry);
-					}
-
-					ServerInstance->SendMode(setban, ServerInstance->FakeClient);
-					ServerInstance->PI->SendMode(chan, ServerInstance->Modes->GetLastParseParams(), ServerInstance->Modes->GetLastParseTranslate());
+
+					ServerInstance->SendGlobalMode(setban, ServerInstance->FakeClient);
 				}
 			}
 			else
@@ -187,7 +164,7 @@ class ModuleTimedBans : public Module
 
 	virtual Version GetVersion()
 	{
-		return Version("Adds timed bans", VF_COMMON | VF_VENDOR, API_VERSION);
+		return Version("Adds timed bans", VF_COMMON | VF_VENDOR);
 	}
 };