From 6075913736e98354c007728658506c93e3503f0b Mon Sep 17 00:00:00 2001 From: special Date: Wed, 13 May 2009 01:23:52 +0000 Subject: Fix m_sakick to properly send local/global snomasks like other modules, instead of repeating a global snomask once per linked server. Found by SnoFox git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@11370 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/modules/m_sakick.cpp | 29 ++++++++++++----------------- 1 file changed, 12 insertions(+), 17 deletions(-) (limited to 'src') diff --git a/src/modules/m_sakick.cpp b/src/modules/m_sakick.cpp index af70c5800..cc8628c50 100644 --- a/src/modules/m_sakick.cpp +++ b/src/modules/m_sakick.cpp @@ -47,7 +47,7 @@ class CommandSakick : public Command if (ServerInstance->ULine(dest->server)) { - user->WriteNumeric(ERR_NOPRIVILEGES, "%s :Cannot use an SA command on a u-lined client",user->nick.c_str()); + user->WriteNumeric(ERR_NOPRIVILEGES, "%s :Cannot use an SA command on a u-lined client", user->nick.c_str()); return CMD_FAILURE; } @@ -60,29 +60,24 @@ class CommandSakick : public Command if (!channel->ServerKickUser(dest, reason, servername)) delete channel; - Channel* n = ServerInstance->FindChan(parameters[1]); - if (!n) + Channel *n = ServerInstance->FindChan(parameters[1]); + if (!n || !n->HasUser(dest)) { - ServerInstance->SNO->WriteGlobalSno('a', std::string(user->nick)+" SAKICKed "+dest->nick+" on "+parameters[0]); - return CMD_SUCCESS; + /* Success; send the global snomask */ + ServerInstance->PI->SendSNONotice("A", std::string(user->nick) + " SAKICKed " + dest->nick + " on " + parameters[0]); } else { - if (!n->HasUser(dest)) - { - ServerInstance->SNO->WriteGlobalSno('a', std::string(user->nick)+" SAKICKed "+dest->nick+" on "+parameters[0]); - return CMD_SUCCESS; - } - else - { - user->WriteServ("NOTICE %s :*** Unable to kick %s from %s",user->nick.c_str(), dest->nick.c_str(), parameters[0].c_str()); - return CMD_FAILURE; - } + /* Sort-of-bug: If the command was issued remotely, this message won't be sent */ + user->WriteServ("NOTICE %s :*** Unable to kick %s from %s", user->nick.c_str(), dest->nick.c_str(), parameters[0].c_str()); + return CMD_FAILURE; } } - else + + if (IS_LOCAL(user)) { - ServerInstance->SNO->WriteGlobalSno('a', std::string(user->nick)+" sent remote SAKICK to kick "+dest->nick+" from "+parameters[0]); + /* Locally issued command; send the local snomask */ + ServerInstance->SNO->WriteToSnoMask('a', std::string(user->nick) + " SAKICKed " + dest->nick + " on " + parameters[0]); } return CMD_SUCCESS; -- cgit v1.2.3