Fix use of commasepstream on now space-separated items

[user/henk/code/inspircd.git] / src / modules / m_spanningtree / fmode.cpp

diff --git a/src/modules/m_spanningtree/fmode.cpp b/src/modules/m_spanningtree/fmode.cpp
index 2ab37017079131d6d257453c9664df87418760fa..598c9ab9ce0d3ea01e2a84b67acae2c8fd9b201e 100644 (file)
--- a/src/modules/m_spanningtree/fmode.cpp
+++ b/src/modules/m_spanningtree/fmode.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -14,40 +14,25 @@
 #include "inspircd.h"
 #include "xline.h"
 
-#include "m_spanningtree/treesocket.h"
-#include "m_spanningtree/treeserver.h"
-#include "m_spanningtree/utils.h"
+#include "treesocket.h"
+#include "treeserver.h"
+#include "utils.h"
 
 /* $ModDep: m_spanningtree/utils.h m_spanningtree/treeserver.h m_spanningtree/treesocket.h */
 
 
 /** FMODE command - server mode with timestamp checks */
-bool TreeSocket::ForceMode(const std::string &source, std::deque<std::string> &params)
+void TreeSocket::ForceMode(User* who, parameterlist &params)
 {
        /* Chances are this is a 1.0 FMODE without TS */
        if (params.size() < 3)
        {
                /* No modes were in the command, probably a channel with no modes set on it */
-               return true;
+               return;
        }
 
-       bool smode = false;
-       std::string sourceserv;
+       std::string sourceserv = who->server;
 
-       /* Are we dealing with an FMODE from a user, or from a server? */
-       User* who = this->Instance->FindNick(source);
-       if (who)
-       {
-               /* FMODE from a user, set sourceserv to the users server name */
-               sourceserv = who->server;
-       }
-       else
-       {
-               /* FMODE from a server, use a fake user to receive mode feedback */
-               who = this->Instance->FakeClient;
-               smode = true;                   /* Setting this flag tells us it is a server mode*/
-               sourceserv = source;    /* Set sourceserv to the actual source string */
-       }
        std::vector<std::string> modelist;
        time_t TS = 0;
        for (unsigned int q = 0; (q < params.size()) && (q < 64); q++)
@@ -68,7 +53,7 @@ bool TreeSocket::ForceMode(const std::string &source, std::deque<std::string> &p
 
        }
        /* Extract the TS value of the object, either User or Channel */
-       User* dst = this->Instance->FindNick(params[0]);
+       User* dst = ServerInstance->FindNick(params[0]);
        Channel* chan = NULL;
        time_t ourTS = 0;
 
@@ -78,41 +63,34 @@ bool TreeSocket::ForceMode(const std::string &source, std::deque<std::string> &p
        }
        else
        {
-               chan = this->Instance->FindChan(params[0]);
+               chan = ServerInstance->FindChan(params[0]);
                if (chan)
                {
                        ourTS = chan->age;
                }
                else
                        /* Oops, channel doesnt exist! */
-                       return true;
+                       return;
        }
 
        if (!TS)
        {
-               Instance->Logs->Log("m_spanningtree",DEFAULT,"*** BUG? *** TS of 0 sent to FMODE. Are some services authors smoking craq, or is it 1970 again?. Dropped.");
-               Instance->SNO->WriteToSnoMask('d', "WARNING: The server %s is sending FMODE with a TS of zero. Total craq. Mode was dropped.", sourceserv.c_str());
-               return true;
+               ServerInstance->Logs->Log("m_spanningtree",DEFAULT,"*** BUG? *** TS of 0 sent to FMODE. Are some services authors smoking craq, or is it 1970 again?. Dropped.");
+               ServerInstance->SNO->WriteToSnoMask('d', "WARNING: The server %s is sending FMODE with a TS of zero. Total craq. Mode was dropped.", sourceserv.c_str());
+               return;
        }
 
        /* TS is equal or less: Merge the mode changes into ours and pass on.
         */
        if (TS <= ourTS)
        {
-               if (smode)
-               {
-                       this->Instance->SendMode(modelist, who);
-               }
-               else
-               {
-                       this->Instance->CallCommandHandler("MODE", modelist, who);
-               }
+               ServerInstance->Modes->Process(modelist, who, IS_SERVER(who));
+
                /* HOT POTATO! PASS IT ON! */
-               Utils->DoOneToAllButSender(source,"FMODE",params,sourceserv);
+               Utils->DoOneToAllButSender(sourceserv,"FMODE",params,sourceserv);
        }
        /* If the TS is greater than ours, we drop the mode and dont pass it anywhere.
         */
-       return true;
 }