- Modify ServerConfig::GetFullProgDir to just return cwd, previously it did a lot...

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

diff --git a/src/cmd_kill.cpp b/src/cmd_kill.cpp
index d0de52b8d18eba2a80ac49165712f1a90ee3fbd4..82ca9744053889fa33aa85374ad7769a7d1153d3 100644 (file)
--- a/src/cmd_kill.cpp
+++ b/src/cmd_kill.cpp
@@ -18,7 +18,7 @@
 #include "wildcard.h"
 #include "commands/cmd_kill.h"
 
-extern "C" command_t* init_command(InspIRCd* Instance)
+extern "C" DllExport command_t* init_command(InspIRCd* Instance)
 {
        return new cmd_kill(Instance);
 }
@@ -33,6 +33,7 @@ CmdResult cmd_kill::Handle (const char** parameters, int pcnt, userrec *user)
 
        userrec *u = ServerInstance->FindNick(parameters[0]);
        char killreason[MAXBUF];
+       char killoperreason[MAXBUF];
        int MOD_RESULT = 0;
 
        if (u)
@@ -42,44 +43,45 @@ CmdResult cmd_kill::Handle (const char** parameters, int pcnt, userrec *user)
                if (MOD_RESULT)
                        return CMD_FAILURE;
 
+               // generate two reasons here, one for users, one for opers. first, the user visible reason, which may change.
+               if (*ServerInstance->Config->HideKillsServer)
+               {
+                       // hidekills is on, use it
+                       snprintf(killreason, MAXQUIT, "Killed (%s (%s))", ServerInstance->Config->HideKillsServer, parameters[1]);
+               }
+               else
+               {
+                       // hidekills is off, do nothing
+                       snprintf(killreason, MAXQUIT, "Killed (%s (%s))", user->nick, parameters[1]);
+               }
+
+               // opers are lucky ducks, they always see the real reason
+               snprintf(killoperreason, MAXQUIT, "Killed (%s (%s))", user->nick, parameters[1]);
+
                if (!IS_LOCAL(u))
                {
                        // remote kill
                        ServerInstance->SNO->WriteToSnoMask('k',"Remote kill by %s: %s!%s@%s (%s)", user->nick, u->nick, u->ident, u->host, parameters[1]);
-                       snprintf(killreason, MAXQUIT,"[%s] Killed (%s (%s))", ServerInstance->Config->ServerName, user->nick, parameters[1]);
-                       u->WriteCommonExcept("QUIT :%s", killreason);
                        FOREACH_MOD(I_OnRemoteKill, OnRemoteKill(user, u, killreason));
-                       
-                       user_hash::iterator iter = ServerInstance->clientlist->find(u->nick);
 
-                       if (iter != ServerInstance->clientlist->end())
-                               ServerInstance->clientlist->erase(iter);
+                       /*
+                        * IMPORTANT SHIT:
+                        *  There used to be a WriteCommonExcept() of the QUIT here. It seems to be unnecessary with QuitUser() right below, so it's gone.
+                        *  If it explodes painfully, put it back!
+                        */
 
-                       if (u->registered == REG_ALL)
-                       {
-                               u->PurgeEmptyChannels();
-                       }
-
-                       if (u == user)
-                       {
-                               std::string original_command = std::string("KILL ") + u->nick + " :"+parameters[1];
-                               FOREACH_MOD(I_OnPostCommand,OnPostCommand("KILL", parameters, pcnt, user, CMD_SUCCESS,original_command));
-                               return CMD_USER_DELETED;
-                       }
-                       DELETE(u);
+                       userrec::QuitUser(ServerInstance, u, killreason);
                }
                else
                {
                        // local kill
+                       ServerInstance->SNO->WriteToSnoMask('k',"Local Kill by %s: %s!%s@%s (%s)", user->nick, u->nick, u->ident, u->host, parameters[1]);
                        ServerInstance->Log(DEFAULT,"LOCAL KILL: %s :%s!%s!%s (%s)", u->nick, ServerInstance->Config->ServerName, user->dhost, user->nick, parameters[1]);
                        user->WriteTo(u, "KILL %s :%s!%s!%s (%s)", u->nick, ServerInstance->Config->ServerName, user->dhost, user->nick, parameters[1]);
-                       ServerInstance->SNO->WriteToSnoMask('k',"Local Kill by %s: %s!%s@%s (%s)", user->nick, u->nick, u->ident, u->host, parameters[1]);
-                       snprintf(killreason,MAXQUIT,"Killed (%s (%s))", user->nick, parameters[1]);
-                       userrec::QuitUser(ServerInstance, u, killreason);
-
-                       if (u == user)
-                               return CMD_USER_DELETED;
                }
+
+               // send the quit out
+               userrec::QuitUser(ServerInstance, u, killreason, killoperreason);
        }
        else
        {