X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcmd_nick.cpp;h=93fdd560cca9b03143475dc4234b316dcafb6656;hb=2b3394855d5adddb16285b905503d9ffe5a1d963;hp=2725f1a3c46a11b4b64ed85db642df869491c210;hpb=80b7db360de994d4509ea87c4325634f53d9d4ff;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/cmd_nick.cpp b/src/cmd_nick.cpp
index 2725f1a3c..93fdd560c 100644
--- a/src/cmd_nick.cpp
+++ b/src/cmd_nick.cpp
@@ -11,14 +11,11 @@
  * ---------------------------------------------------
  */
 
-#include "configreader.h"
-#include "users.h"
-#include "modules.h"
 #include "inspircd.h"
 #include "xline.h"
 #include "commands/cmd_nick.h"
 
-extern "C" command_t* init_command(InspIRCd* Instance)
+extern "C" DllExport command_t* init_command(InspIRCd* Instance)
 {
 	return new cmd_nick(Instance);
 }
@@ -34,8 +31,8 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
 
 	if (!*parameters[0] || !*user->nick)
 	{
-		/* We cant put blanks in the parameters, so for this (extremely rare) issue we just put '* *' here. */
-		user->WriteServ("432 * * :Erroneous Nickname");
+		/* We cant put blanks in the parameters, so for this (extremely rare) issue we just put '*' here. */
+		user->WriteServ("432 %s * :Erroneous Nickname", *user->nick ? user->nick : "*");
 		return CMD_FAILURE;
 	}
 
@@ -76,39 +73,37 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
 		 * the nickname too, we force a nickchange on the older user (Simply the one who was
 		 * here first, no TS checks need to take place here)
 		 */
-		userrec* InUse = ServerInstance->FindNick(parameters[0]);
-		if (InUse && (InUse != user) && (ServerInstance->IsNick(parameters[0])))
+		userrec* InUse = ServerInstance->FindNickOnly(parameters[0]);
+		ServerInstance->Log(DEBUG,"Nick in use");
+		if (InUse && (InUse != user) && ((ServerInstance->IsNick(parameters[0]) || allowinvalid)))
 		{
 			if (InUse->registered != REG_ALL)
 			{
-				/* change the nick of the older user to nnn-overruled,
-				 * where nnn is their file descriptor. We know this to be unique.
-				 * NOTE: We must do this and not quit the user, even though we do
-				 * not have UID support yet. This is because if we set this user
-				 * as quitting and then introduce the new user before the old one
-				 * has quit, then the user hash gets totally buggered.
-				 * (Yes, that is a technical term). -- Brain
+				/* change the nick of the older user to its UUID
 				 */
-				std::string changeback = ConvToStr(InUse->GetFd()) + "-overruled";
-				InUse->WriteTo(InUse, "NICK %s", changeback.c_str());
+				InUse->WriteTo(InUse, "NICK %s", InUse->uuid);
 				InUse->WriteServ("433 %s %s :Nickname overruled.", InUse->nick, InUse->nick);
-				InUse->UpdateNickHash(changeback.c_str());
-				strlcpy(InUse->nick, changeback.c_str(), NICKMAX - 1);
+				InUse->UpdateNickHash(InUse->uuid);
+				strlcpy(InUse->nick, InUse->uuid, NICKMAX - 1);
 				InUse->InvalidateCache();
 				/* Take away their nickname-sent state forcing them to send a nick again */
 				InUse->registered &= ~REG_NICK;
 			}
 			else
 			{
+				ServerInstance->Log(DEBUG,"Nick in use and user REG_ALL");
 				user->WriteServ("433 %s %s :Nickname is already in use.", user->registered >= REG_NICK ? user->nick : "*", parameters[0]);
 				return CMD_FAILURE;
 			}
 		}
 	}
-	if ((!ServerInstance->IsNick(parameters[0])) && (IS_LOCAL(user)))
+	if (((!ServerInstance->IsNick(parameters[0]))) && (IS_LOCAL(user)))
 	{
-		user->WriteServ("432 %s %s :Erroneous Nickname",user->nick,parameters[0]);
-		return CMD_FAILURE;
+		if (!allowinvalid)
+		{
+			user->WriteServ("432 %s %s :Erroneous Nickname",user->nick,parameters[0]);
+			return CMD_FAILURE;
+		}
 	}
 
 	if (user->registered == REG_ALL)
@@ -137,6 +132,15 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
 
 	user->InvalidateCache();
 
+	/* Update display nicks */
+	for (UCListIter v = user->chans.begin(); v != user->chans.end(); v++)
+	{
+		CUList* ulist = v->first->GetUsers();
+		CUList::iterator i = ulist->find(user);
+		if (i != ulist->end())
+			i->second = user->nick;
+	}
+
 	if (user->registered < REG_NICKUSER)
 	{
 		user->registered = (user->registered | REG_NICK);
@@ -178,3 +182,9 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
 
 }
 
+CmdResult cmd_nick::HandleInternal(const unsigned int id, const std::deque<classbase*> &parameters)
+{
+	allowinvalid = (id != 0);
+	return CMD_SUCCESS;
+}
+