Convert more modules

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

diff --git a/src/cmd_privmsg.cpp b/src/cmd_privmsg.cpp
index abf46814a1dd8f87c1aca242a088afd1d4f691c7..f3295df07e88f30d55a4d252bce3d8fb0d668b26 100644 (file)
--- a/src/cmd_privmsg.cpp
+++ b/src/cmd_privmsg.cpp
@@ -18,9 +18,7 @@
 #include "wildcard.h"
 #include "commands/cmd_privmsg.h"
 
-
-
-extern "C" command_t* init_command(InspIRCd* Instance)
+extern "C" DllExport  command_t* init_command(InspIRCd* Instance)
 {
        return new cmd_privmsg(Instance);
 }
@@ -29,7 +27,6 @@ CmdResult cmd_privmsg::Handle (const char** parameters, int pcnt, userrec *user)
 {
        userrec *dest;
        chanrec *chan;
-
        CUList except_list;
 
        user->idle_lastmsg = ServerInstance->Time();
@@ -37,19 +34,19 @@ CmdResult cmd_privmsg::Handle (const char** parameters, int pcnt, userrec *user)
        if (ServerInstance->Parser->LoopCall(user, this, parameters, pcnt, 0))
                return CMD_SUCCESS;
 
-       if ((parameters[0][0] == '$') && ((*user->oper) || (ServerInstance->ULine(user->server))))
+       if ((parameters[0][0] == '$') && (IS_OPER(user) || ServerInstance->ULine(user->server)))
        {
                int MOD_RESULT = 0;
                std::string temp = parameters[1];
                FOREACH_RESULT(I_OnUserPreMessage,OnUserPreMessage(user,(void*)parameters[0],TYPE_SERVER,temp,0,except_list));
                if (MOD_RESULT)
                        return CMD_FAILURE;
-               parameters[1] = (char*)temp.c_str();
+               parameters[1] = temp.c_str();
                // notice to server mask
                const char* servermask = parameters[0] + 1;
                if (match(ServerInstance->Config->ServerName,servermask))
                {
-                       ServerInstance->ServerPrivmsgAll("%s",parameters[1]);
+                       user->SendAll("PRIVMSG", "%s", parameters[1]);
                }
                FOREACH_MOD(I_OnUserMessage,OnUserMessage(user,(void*)parameters[0],TYPE_SERVER,parameters[1],0,except_list));
                return CMD_SUCCESS;
@@ -64,7 +61,7 @@ CmdResult cmd_privmsg::Handle (const char** parameters, int pcnt, userrec *user)
        {
                chan = ServerInstance->FindChan(parameters[0]);
 
-               except_list[user] = user;
+               except_list[user] = user->nick;
 
                if (chan)
                {
@@ -90,9 +87,10 @@ CmdResult cmd_privmsg::Handle (const char** parameters, int pcnt, userrec *user)
                        }
                        parameters[1] = temp.c_str();
 
-                       if (temp == "")
+                       /* Check again, a module may have zapped the input string */
+                       if (temp.empty())
                        {
-                               user->WriteServ("412 %s No text to send", user->nick);
+                               user->WriteServ("412 %s :No text to send", user->nick);
                                return CMD_FAILURE;
                        }
 
@@ -126,7 +124,13 @@ CmdResult cmd_privmsg::Handle (const char** parameters, int pcnt, userrec *user)
        dest = ServerInstance->FindNick(parameters[0]);
        if (dest)
        {
-               if ((IS_LOCAL(user)) && (*dest->awaymsg))
+               if (!*parameters[1])
+               {
+                       user->WriteServ("412 %s :No text to send", user->nick);
+                       return CMD_FAILURE;
+               }
+
+               if (IS_AWAY(dest))
                {
                        /* auto respond with aweh msg */
                        user->WriteServ("301 %s %s :%s",user->nick,dest->nick,dest->awaymsg);