Change the wording of configure, to indicate that building as ipv6 is no longer an...

[user/henk/code/inspircd.git] / src / cmd_nick.cpp

diff --git a/src/cmd_nick.cpp b/src/cmd_nick.cpp
index 10d95ed469696835c39284b6be0aa8e10052910f..0bba6af0d2a32533aff15694b5932292bf7fef16 100644 (file)
--- a/src/cmd_nick.cpp
+++ b/src/cmd_nick.cpp
@@ -2,12 +2,9 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *                <Craig@chatspike.net>
+ *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ * See: http://www.inspircd.org/wiki/index.php/Credits
  *
- * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
  *
@@ -33,25 +30,21 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
        char oldnick[NICKMAX];
 
        if (!*parameters[0])
-       {
-               ServerInstance->Log(DEBUG,"zero length new nick passed to handle_nick");
                return CMD_FAILURE;
-       }
+
        if (!*user->nick)
-       {
-               ServerInstance->Log(DEBUG,"invalid old nick passed to handle_nick");
                return CMD_FAILURE;
-       }
+
        if (irc::string(user->nick) == irc::string(parameters[0]))
        {
                /* If its exactly the same, even case, dont do anything. */
                if (!strcmp(user->nick,parameters[0]))
                        return CMD_SUCCESS;
+
                /* Its a change of case. People insisted that they should be
                 * able to do silly things like this even though the RFC says
                 * the nick AAA is the same as the nick aaa.
                 */
-               ServerInstance->Log(DEBUG,"old nick is new nick, not updating hash (case change only)");
                strlcpy(oldnick, user->nick, NICKMAX - 1);
                int MOD_RESULT = 0;
                FOREACH_RESULT(I_OnUserPreNick,OnUserPreNick(user,parameters[0]));
@@ -60,20 +53,17 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
                if (user->registered == REG_ALL)
                        user->WriteCommon("NICK %s",parameters[0]);
                strlcpy(user->nick, parameters[0], NICKMAX - 1);
+               user->InvalidateCache();
                FOREACH_MOD(I_OnUserPostNick,OnUserPostNick(user,oldnick));
                return CMD_SUCCESS;
        }
        else
        {
-               if ((*parameters[0] == ':') && (*(parameters[0]+1) != 0))
-               {
-                       parameters[0]++;
-               }
-               char* mq = ServerInstance->XLines->matches_qline(parameters[0]);
+               QLine* mq = ServerInstance->XLines->matches_qline(parameters[0]);
                if (mq)
                {
-                       ServerInstance->WriteOpers("*** Q-Lined nickname %s from %s!%s@%s: %s",parameters[0],user->nick,user->ident,user->host,mq);
-                       user->WriteServ("432 %s %s :Invalid nickname: %s",user->nick,parameters[0],mq);
+                       ServerInstance->SNO->WriteToSnoMask('x', "Q-Lined nickname %s from %s!%s@%s: %s", parameters[0], user->nick, user->ident, user->host, mq->reason);
+                       user->WriteServ("432 %s %s :Invalid nickname: %s",user->nick,parameters[0], mq->reason);
                        return CMD_FAILURE;
                }
                if ((ServerInstance->FindNick(parameters[0])) && (ServerInstance->FindNick(parameters[0]) != user) && (ServerInstance->IsNick(parameters[0])))
@@ -85,7 +75,7 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
                        }
                        else
                        {
-                               user->WriteServ("433 %s %s :Nickname is already in use.",user->nick,parameters[0]);
+                               user->WriteServ("433 %s %s :Nickname is already in use.", user->registered >= REG_NICK ? user->nick : "*", parameters[0]);
                                return CMD_FAILURE;
                        }
                }
@@ -106,39 +96,44 @@ CmdResult cmd_nick::Handle (const char** parameters, int pcnt, userrec *user)
                }
 
                user->WriteCommon("NICK %s",parameters[0]);
-               
+
        }
 
        strlcpy(oldnick, user->nick, NICKMAX - 1);
 
        /* change the nick of the user in the users_hash */
        user = user->UpdateNickHash(parameters[0]);
+
        /* actually change the nick within the record */
        if (!user) return CMD_FAILURE;
        if (!*user->nick) return CMD_FAILURE;
 
        strlcpy(user->nick, parameters[0], NICKMAX - 1);
 
-       ServerInstance->Log(DEBUG,"new nick set: %s",user->nick);
-       
+       user->InvalidateCache();
+
        if (user->registered < REG_NICKUSER)
        {
                user->registered = (user->registered | REG_NICK);
-               // dont attempt to look up the dns until they pick a nick... because otherwise their pointer WILL change
-               // and unless we're lucky we'll get a duff one later on.
-               //user->dns_done = (!lookup_dns(user->nick));
-               //if (user->dns_done)
-               //      ServerInstance->Log(DEBUG,"Aborting dns lookup of %s because dns server experienced a failure.",user->nick);
 
                if (ServerInstance->Config->NoUserDns)
                {
                        user->dns_done = true;
+                       ServerInstance->next_call = ServerInstance->Time();
                }
                else
                {
                        user->StartDNSLookup();
                        if (user->dns_done)
-                               ServerInstance->Log(DEBUG,"Aborting dns lookup of %s because dns server experienced a failure.",user->nick);
+                       {
+                               /* Cached result or instant failure - fall right through if possible */
+                               ServerInstance->next_call = ServerInstance->Time();
+                       }
+                       else
+                       {
+                               if (ServerInstance->next_call > ServerInstance->Time() + ServerInstance->Config->dns_timeout)
+                                       ServerInstance->next_call = ServerInstance->Time() + ServerInstance->Config->dns_timeout;
+                       }
                }
        }
        if (user->registered == REG_NICKUSER)