X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_cap.h;h=409671f482aeb5301cf3e62e3c6c75979424b917;hb=a5fe50aca04ca554d313e7361c571c6a497a9c4e;hp=0fc782b97c6f9b53bf9c08837700eb3aba4cf740;hpb=553a8da754c8cd308bad2008018849714e70f9b7;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_cap.h b/src/modules/m_cap.h
index 0fc782b97..409671f48 100644
--- a/src/modules/m_cap.h
+++ b/src/modules/m_cap.h
@@ -21,18 +21,22 @@
 #ifndef M_CAP_H
 #define M_CAP_H
 
-#include <map>
-#include <string>
-
 class CapEvent : public Event
 {
  public:
-	irc::string type;
+	enum CapEventType
+	{
+		CAPEVENT_REQ,
+		CAPEVENT_LS,
+		CAPEVENT_LIST,
+		CAPEVENT_CLEAR
+	};
+
+	CapEventType type;
 	std::vector<std::string> wanted;
 	std::vector<std::string> ack;
 	User* user;
-	Module* creator;
-	CapEvent(Module* sender, const std::string& t) : Event(sender, t) {}
+	CapEvent(Module* sender, User* u, CapEventType capevtype) : Event(sender, "cap_request"), type(capevtype), user(u) {}
 };
 
 class GenericCap
@@ -42,36 +46,42 @@ class GenericCap
 	const std::string cap;
 	GenericCap(Module* parent, const std::string &Cap) : ext("cap_" + Cap, parent), cap(Cap)
 	{
-		ServerInstance->Extensions.Register(&ext);
+		ServerInstance->Modules->AddService(ext);
 	}
 
 	void HandleEvent(Event& ev)
 	{
+		if (ev.id != "cap_request")
+			return;
+
 		CapEvent *data = static_cast<CapEvent*>(&ev);
-		if (ev.id == "cap_req")
+		if (data->type == CapEvent::CAPEVENT_REQ)
 		{
-			std::vector<std::string>::iterator it;
-			if ((it = std::find(data->wanted.begin(), data->wanted.end(), cap)) != data->wanted.end())
+			for (std::vector<std::string>::iterator it = data->wanted.begin(); it != data->wanted.end(); ++it)
 			{
-				// we can handle this, so ACK it, and remove it from the wanted list
-				data->ack.push_back(*it);
-				data->wanted.erase(it);
-				ext.set(data->user, 1);
+				if (it->empty())
+					continue;
+				bool enablecap = ((*it)[0] != '-');
+				if (((enablecap) && (*it == cap)) || (*it == "-" + cap))
+				{
+					// we can handle this, so ACK it, and remove it from the wanted list
+					data->ack.push_back(*it);
+					data->wanted.erase(it);
+					ext.set(data->user, enablecap ? 1 : 0);
+					break;
+				}
 			}
 		}
-
-		if (ev.id == "cap_ls")
+		else if (data->type == CapEvent::CAPEVENT_LS)
 		{
 			data->wanted.push_back(cap);
 		}
-
-		if (ev.id == "cap_list")
+		else if (data->type == CapEvent::CAPEVENT_LIST)
 		{
 			if (ext.get(data->user))
 				data->wanted.push_back(cap);
 		}
-
-		if (ev.id == "cap_clear")
+		else if (data->type == CapEvent::CAPEVENT_CLEAR)
 		{
 			data->ack.push_back("-" + cap);
 			ext.set(data->user, 0);