Configurable prefixes for chanmodes +qa: prefixes can be turned on or off individually and seperately, per server as well as per mode, and prefixes are no longer limited to ~&. *** SERVICES AUTHORS *** This involves a protocol change. FJOIN now sends the userlist with format modes,nick instead of prefixes,nick.

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@9241 e03df62e-2008-0410-955e-edbf42e46eb7

3 files changed, 48 insertions, 66 deletions

diff --git a/src/modules/m_chanprotect.cpp b/src/modules/m_chanprotect.cpp
index 853d65159..c811d985f 100644
--- a/src/modules/m_chanprotect.cpp
+++ b/src/modules/m_chanprotect.cpp
@@ -164,8 +164,8 @@ class ChanFounder : public ModeHandler, public FounderProtectBase
 {
 	char* dummyptr;
  public:
-	ChanFounder(InspIRCd* Instance, bool using_prefixes, bool &depriv_self, bool &depriv_others)
-		: ModeHandler(Instance, 'q', 1, 1, true, MODETYPE_CHANNEL, false, using_prefixes ? '~' : 0, 0),
+	ChanFounder(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others)
+		: ModeHandler(Instance, 'q', 1, 1, true, MODETYPE_CHANNEL, false, my_prefix, 0),
 		  FounderProtectBase(Instance, "cm_founder_", "founder", 386, 387, depriv_self, depriv_others) { }
 
 	unsigned int GetPrefixRank()
@@ -226,8 +226,8 @@ class ChanProtect : public ModeHandler, public FounderProtectBase
 {
 	char* dummyptr;
  public:
-	ChanProtect(InspIRCd* Instance, bool using_prefixes, bool &depriv_self, bool &depriv_others)
-		: ModeHandler(Instance, 'a', 1, 1, true, MODETYPE_CHANNEL, false, using_prefixes ? '&' : 0, 0),
+	ChanProtect(InspIRCd* Instance, char my_prefix, bool &depriv_self, bool &depriv_others)
+		: ModeHandler(Instance, 'a', 1, 1, true, MODETYPE_CHANNEL, false, my_prefix, 0),
 		  FounderProtectBase(Instance,"cm_protect_","protected user", 388, 389, depriv_self, depriv_others) { }
 
 	unsigned int GetPrefixRank()
@@ -286,7 +286,8 @@ class ModuleChanProtect : public Module
 {
 	
 	bool FirstInGetsFounder;
-	bool QAPrefixes;
+	char QPrefix;
+	char APrefix;
 	bool DeprivSelf;
 	bool DeprivOthers;
 	bool booting;
@@ -297,7 +298,7 @@ class ModuleChanProtect : public Module
  public:
  
 	ModuleChanProtect(InspIRCd* Me)
-		: Module(Me), FirstInGetsFounder(false), QAPrefixes(false), DeprivSelf(false), DeprivOthers(false), booting(true)
+		: Module(Me), FirstInGetsFounder(false), QPrefix(0), APrefix(0), DeprivSelf(false), DeprivOthers(false), booting(true)
 	{	
 		/* Load config stuff */
 		OnRehash(NULL,"");
@@ -305,8 +306,8 @@ class ModuleChanProtect : public Module
 
 		/* Initialise module variables */
 
-		cp = new ChanProtect(ServerInstance,QAPrefixes,DeprivSelf,DeprivOthers);
-		cf = new ChanFounder(ServerInstance,QAPrefixes,DeprivSelf,DeprivOthers);
+		cp = new ChanProtect(ServerInstance, APrefix, DeprivSelf, DeprivOthers);
+		cf = new ChanFounder(ServerInstance, QPrefix, DeprivSelf, DeprivOthers);
 
 		if (!ServerInstance->Modes->AddMode(cp) || !ServerInstance->Modes->AddMode(cf))
 		{
@@ -315,8 +316,8 @@ class ModuleChanProtect : public Module
 			throw ModuleException("Could not add new modes!");
 		}
 
-		Implementation eventlist[] = { I_OnUserKick, I_OnUserPart, I_OnRehash, I_OnUserPreJoin, I_OnPostJoin, I_OnAccessCheck, I_OnSyncChannel };
-		ServerInstance->Modules->Attach(eventlist, this, 7);
+		Implementation eventlist[] = { I_OnUserKick, I_OnUserPart, I_OnRehash, I_OnUserPreJoin, I_OnPostJoin, I_OnAccessCheck };
+		ServerInstance->Modules->Attach(eventlist, this, 6);
 	}
 
 	virtual void OnUserKick(User* source, User* user, Channel* chan, const std::string &reason, bool &silent)
@@ -342,30 +343,44 @@ class ModuleChanProtect : public Module
 		 */
 		ConfigReader Conf(ServerInstance);
 
-		bool old_qa = QAPrefixes;
+		char old_q = QPrefix;
+		char old_a = APrefix;
+
+		FirstInGetsFounder = Conf.ReadFlag("options", "noservices", 0);
+
+		std::string qpre = Conf.ReadValue("options", "qprefix", 0);
+		QPrefix = qpre.empty() ? 0 : qpre[0];
+
+		std::string apre = Conf.ReadValue("options", "aprefix", 0);
+		APrefix = apre.empty() ? 0 : apre[0];
 
-		FirstInGetsFounder = Conf.ReadFlag("options","noservices",0);
-		QAPrefixes = Conf.ReadFlag("options","qaprefixes",0);
 		DeprivSelf = Conf.ReadFlag("options","deprotectself",0);
 		DeprivOthers = Conf.ReadFlag("options","deprotectothers",0);
 
+		ServerInstance->Logs->Log("chanprotect", DEBUG, "qprefix is %c and aprefix is %c", QPrefix, APrefix);
+
 		/* Did the user change the QA prefixes on the fly?
 		 * If so, remove all instances of the mode, and reinit
 		 * the module with prefixes enabled.
 		 */
-		if ((old_qa != QAPrefixes) && (!booting))
+		if ((old_q != QPrefix) && (!booting))
 		{
-			ServerInstance->Modes->DelMode(cp);
 			ServerInstance->Modes->DelMode(cf);
-			delete cp;
 			delete cf;
-			cp = new ChanProtect(ServerInstance,QAPrefixes,DeprivSelf,DeprivOthers);
-			cf = new ChanFounder(ServerInstance,QAPrefixes,DeprivSelf,DeprivOthers);
+			cf = new ChanFounder(ServerInstance, QPrefix, DeprivSelf, DeprivOthers);
 			/* These wont fail, we already owned the mode characters before */
-			ServerInstance->Modes->AddMode(cp);
 			ServerInstance->Modes->AddMode(cf);
 			ServerInstance->SNO->WriteToSnoMask('A', "WARNING: +qa prefixes were enabled or disabled via a REHASH. Clients will probably need to reconnect to pick up this change.");
 		}
+
+		if ((old_a != APrefix) && (!booting))
+		{
+			ServerInstance->Modes->DelMode(cp);
+			delete cp;
+			cp = new ChanProtect(ServerInstance, APrefix, DeprivSelf, DeprivOthers);
+			ServerInstance->Modes->AddMode(cp);
+			ServerInstance->SNO->WriteToSnoMask('A', "WARNING: +qa prefixes were enabled or disabled via a REHASH. Clients will probably need to reconnect to pick up this change.");
+		}
 	}
 	
 	virtual int OnUserPreJoin(User *user, Channel *chan, const char *cname, std::string &privs)
@@ -374,7 +389,7 @@ class ModuleChanProtect : public Module
 		// the config option for it is set
 
 		if (FirstInGetsFounder && !chan)
-			privs = "~@";
+			privs = QPrefix + "@";
 		
 		return 0;
 	}
@@ -488,43 +503,6 @@ class ModuleChanProtect : public Module
 	{
 		return Version(1, 2, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
 	}
-	
-	virtual void OnSyncChannel(Channel* chan, Module* proto, void* opaque)
-	{
-		/* NOTE: If +qa prefix is on, this is propagated by the channel join,
-		 * so we dont need to propagate it manually
-		 */
-		if (!QAPrefixes)
-		{
-			// this is called when the server is linking into a net and wants to sync channel data.
-			// we should send our mode changes for the channel here to ensure that other servers
-			// know whos +q/+a on the channel.
-			CUList* cl = chan->GetUsers();
-			string_list commands;
-			std::string founder = "cm_founder_"+std::string(chan->name);
-			std::string protect = "cm_protect_"+std::string(chan->name);
-			irc::modestacker modestack(true);
-			std::deque<std::string> stackresult;
-			for (CUList::iterator i = cl->begin(); i != cl->end(); i++)
-			{
-				if (i->first->GetExt(founder,dummyptr))
-				{
-					modestack.Push('q',i->first->nick);
-				}
-				if (i->first->GetExt(protect,dummyptr))
-				{
-					modestack.Push('a',i->first->nick);
-				}
-			}
-			while (modestack.GetStackedLine(stackresult))
-			{
-				irc::stringjoiner mode_join(" ", stackresult, 0, stackresult.size() - 1);
-				std::string line = mode_join.GetJoined();
-				proto->ProtoSendMode(opaque,TYPE_CHANNEL,chan, line);
-			}
-		}
-	}
-
 };
 
 MODULE_INIT(ModuleChanProtect)
diff --git a/src/modules/m_spanningtree/fjoin.cpp b/src/modules/m_spanningtree/fjoin.cpp
index a4fe5836a..0f33d65bd 100644
--- a/src/modules/m_spanningtree/fjoin.cpp
+++ b/src/modules/m_spanningtree/fjoin.cpp
@@ -112,22 +112,26 @@ bool TreeSocket::ForceJoin(const std::string &source, std::deque<std::string> &p
 		const char* usr = item.c_str();
 		if (usr && *usr)
 		{
-			const char* permissions = usr;
-			/* Iterate through all the prefix values, convert them from prefixes to mode letters */
+			const char* unparsedmodes = usr;
 			std::string modes;
-			while ((*permissions) && (*permissions != ','))
+
+
+			/* Iterate through all modes for this user and check they are valid. */
+			while ((*unparsedmodes) && (*unparsedmodes != ','))
 			{
-				ModeHandler* mh = Instance->Modes->FindPrefix(*permissions);
+				ModeHandler *mh = Instance->Modes->FindMode(*unparsedmodes, MODETYPE_CHANNEL);
 				if (mh)
-					modes = modes + mh->GetModeChar();
+					modes += *unparsedmodes;
 				else
 				{
-					this->SendError(std::string("Invalid prefix '")+(*permissions)+"' in FJOIN");
+					this->SendError(std::string("Invalid prefix '")+(*unparsedmodes)+"' in FJOIN");
 					return false;
 				}
+
 				usr++;
-				permissions++;
+				unparsedmodes++;
 			}
+
 			/* Advance past the comma, to the nick */
 			usr++;
 			
@@ -140,7 +144,7 @@ bool TreeSocket::ForceJoin(const std::string &source, std::deque<std::string> &p
 				if ((!route_back_again) || (route_back_again->GetSocket() != this))
 					continue;
 
-				/* Add any permissions this user had to the mode stack */
+				/* Add any modes this user had to the mode stack */
 				for (std::string::iterator x = modes.begin(); x != modes.end(); ++x)
 					modestack.Push(*x, who->nick);
 
diff --git a/src/modules/m_spanningtree/netburst.cpp b/src/modules/m_spanningtree/netburst.cpp
index d1fcc928b..4e6c1dfd1 100644
--- a/src/modules/m_spanningtree/netburst.cpp
+++ b/src/modules/m_spanningtree/netburst.cpp
@@ -97,7 +97,7 @@ void TreeSocket::SendFJoins(TreeServer* Current, Channel* c)
 	for (CUList::iterator i = ulist->begin(); i != ulist->end(); i++)
 	{
 		// The first parameter gets a : before it
-		size_t ptrlen = snprintf(ptr, MAXBUF, " %s%s,%s", !numusers ? ":" : "", c->GetAllPrefixChars(i->first), i->first->uuid);
+		size_t ptrlen = snprintf(ptr, MAXBUF, " %s%s,%s", !numusers ? ":" : "", c->GetAllPrefixModes(i->first), i->first->uuid);
 
 		curlen += ptrlen;
 		ptr += ptrlen;