Move all the parameters around AGAIN. See docs shortly

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

3 files changed, 33 insertions, 22 deletions

diff --git a/src/modules/m_spanningtree/netburst.cpp b/src/modules/m_spanningtree/netburst.cpp
index 26ff4b209..6d068c5e2 100644
--- a/src/modules/m_spanningtree/netburst.cpp
+++ b/src/modules/m_spanningtree/netburst.cpp
@@ -223,10 +223,18 @@ void TreeSocket::SendUsers(TreeServer* Current)
 			TreeServer* theirserver = Utils->FindServer(u->second->server);
 			if (theirserver)
 			{
-				snprintf(data,MAXBUF,":%s UID %s %lu %s %s %s %s +%s +%s %s %lu :%s", theirserver->GetID().c_str(), u->second->uuid.c_str(),
-						(unsigned long)u->second->age, u->second->nick.c_str(), u->second->host.c_str(), u->second->dhost.c_str(),
-						u->second->ident.c_str(), u->second->FormatModes(), u->second->FormatNoticeMasks(), u->second->GetIPString(),
-						(unsigned long)u->second->signon, u->second->fullname.c_str());
+				snprintf(data,MAXBUF,":%s UID %s %lu %s %s %s %s %s %lu +%s :%s",
+						theirserver->GetID().c_str(),	/* Prefix: SID */
+						u->second->uuid.c_str(),	/* 0: UUID */
+						(unsigned long)u->second->age,	/* 1: TS */
+						u->second->nick.c_str(),	/* 2: Nick */
+						u->second->host.c_str(),	/* 3: Displayed Host */
+						u->second->dhost.c_str(),	/* 4: Real host */
+						u->second->ident.c_str(),	/* 5: Ident */
+						u->second->GetIPString(),	/* 6: IP string */
+						(unsigned long)u->second->signon, /* 7: Signon time for WHOWAS */
+						u->second->FormatModes(true),	/* 8...n: Modes and params */
+						u->second->fullname.c_str());	/* size-1: GECOS */
 				this->WriteLine(data);
 				if (IS_OPER(u->second))
 				{
diff --git a/src/modules/m_spanningtree/protocolinterface.cpp b/src/modules/m_spanningtree/protocolinterface.cpp
index 410ccfdec..84a19b705 100644
--- a/src/modules/m_spanningtree/protocolinterface.cpp
+++ b/src/modules/m_spanningtree/protocolinterface.cpp
@@ -184,10 +184,9 @@ void SpanningTreeProtocolInterface::Introduce(User* user)
 		params.push_back(user->host);
 		params.push_back(user->dhost);
 		params.push_back(user->ident);
-		params.push_back("+"+std::string(user->FormatModes()));
-		params.push_back("+"+std::string(user->FormatNoticeMasks()));
 		params.push_back(user->GetIPString());
 		params.push_back(ConvToStr(user->signon));
+		params.push_back("+"+std::string(user->FormatModes(true)));
 		params.push_back(":"+std::string(user->fullname));
 		Utils->DoOneToMany(ServerInstance->Config->GetSID(), "UID", params);
 	}
diff --git a/src/modules/m_spanningtree/uid.cpp b/src/modules/m_spanningtree/uid.cpp
index 3e76c9725..abde297a2 100644
--- a/src/modules/m_spanningtree/uid.cpp
+++ b/src/modules/m_spanningtree/uid.cpp
@@ -33,19 +33,19 @@
 bool TreeSocket::ParseUID(const std::string &source, std::deque<std::string> &params)
 {
 	/** Do we have enough parameters:
-	 *           1    2    3    4    5      6         7         8        9      10
-	 * UID uuid age nick host dhost ident +modestr +snomasks ip.string signon :gecos
+	 *      0    1    2    3    4    5        6        7     8        9       (n-1)
+	 * UID uuid age nick host dhost ident ip.string signon +modes (modepara) :gecos
 	 */
-	if (params.size() != 11)
+	if (params.size() < 10)
 	{
 		if (!params.empty())
 			this->WriteLine(std::string(":")+this->Instance->Config->GetSID()+" KILL "+params[0]+" :Invalid client introduction ("+params[0]+" with only "+
-					ConvToStr(params.size())+" of 11 parameters?)");
+					ConvToStr(params.size())+" of 10 or more parameters?)");
 		return true;
 	}
 
 	time_t age_t = ConvToInt(params[1]);
-	time_t signon = ConvToInt(params[9]);
+	time_t signon = ConvToInt(params[7]);
 	std::string empty;
 
 	TreeServer* remoteserver = Utils->FindServer(source);
@@ -101,40 +101,44 @@ bool TreeSocket::ParseUID(const std::string &source, std::deque<std::string> &pa
 	_new->dhost.assign(params[4], 0, 64);
 	_new->server = this->Instance->FindServerNamePtr(remoteserver->GetName().c_str());
 	_new->ident.assign(params[5], 0, MAXBUF);
-	_new->fullname.assign(params[10], 0, MAXBUF);
+	_new->fullname.assign(params[params.size() - 1], 0, MAXBUF);
 	_new->registered = REG_ALL;
 	_new->signon = signon;
 	_new->age = age_t;
 
 	/* we need to remove the + from the modestring, so we can do our stuff */
-	std::string::size_type pos_after_plus = params[6].find_first_not_of('+');
+	std::string::size_type pos_after_plus = params[8].find_first_not_of('+');
 	if (pos_after_plus != std::string::npos)
-	params[6] = params[6].substr(pos_after_plus);
+	params[8] = params[8].substr(pos_after_plus);
 
-	for (std::string::iterator v = params[6].begin(); v != params[6].end(); v++)
+	unsigned int paramptr = 9;
+	for (std::string::iterator v = params[8].begin(); v != params[8].end(); v++)
 	{
 		/* For each mode thats set, increase counter */
 		ModeHandler* mh = Instance->Modes->FindMode(*v, MODETYPE_USER);
 
 		if (mh)
 		{
-			mh->OnModeChange(_new, _new, NULL, empty, true);
+			if (mh->GetNumParams(true) && (paramptr < params.size()))
+				mh->OnModeChange(_new, _new, NULL, params[paramptr++], true);
+			else
+				mh->OnModeChange(_new, _new, NULL, empty, true);
 			_new->SetMode(*v, true);
 			mh->ChangeCount(1);
 		}
 	}
 
-	_new->ProcessNoticeMasks(params[7].c_str());
+	//_new->ProcessNoticeMasks(params[7].c_str());
 
 	/* now we've done with modes processing, put the + back for remote servers */
-	params[6] = "+" + params[6];
+	params[8] = "+" + params[8];
 
 #ifdef SUPPORT_IP6LINKS
-	if (params[8].find_first_of(":") != std::string::npos)
-		_new->SetSockAddr(AF_INET6, params[8].c_str(), 0);
+	if (params[6].find_first_of(":") != std::string::npos)
+		_new->SetSockAddr(AF_INET6, params[6].c_str(), 0);
 	else
 #endif
-		_new->SetSockAddr(AF_INET, params[8].c_str(), 0);
+		_new->SetSockAddr(AF_INET, params[6].c_str(), 0);
 
 	Instance->Users->AddGlobalClone(_new);
 
@@ -146,7 +150,7 @@ bool TreeSocket::ParseUID(const std::string &source, std::deque<std::string> &pa
 	if (dosend)
 		this->Instance->SNO->WriteToSnoMask('C',"Client connecting at %s: %s!%s@%s [%s] [%s]", _new->server, _new->nick.c_str(), _new->ident.c_str(), _new->host.c_str(), _new->GetIPString(), _new->fullname.c_str());
 
-	params[10] = ":" + params[10];
+	params[params.size() - 1] = ":" + params[params.size() - 1];
 	Utils->DoOneToAllButSender(source, "UID", params, source);
 
 	Instance->PI->Introduce(_new);