Rewrite clone counting to use one map instead of two

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

diff --git a/src/modules/m_clones.cpp b/src/modules/m_clones.cpp
index 097227d39bd55f957b1abab681d3aa9e430516a1..c51c8d3b4f13e7d7427f017d60fb435ac951d557 100644 (file)
--- a/src/modules/m_clones.cpp
+++ b/src/modules/m_clones.cpp
@@ -1,35 +1,40 @@
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *   Copyright (C) 2007 Dennis Friis <peavey@inspircd.org>
+ *   Copyright (C) 2007 Robin Burchell <robin+git@viroteck.net>
+ *   Copyright (C) 2007 Craig Edwards <craigedwards@brainbox.cc>
  *
- * This program is free but copyrighted software; see
- *            the file COPYING for details.
+ * This file is part of InspIRCd.  InspIRCd is free software: you can
+ * redistribute it and/or modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation, version 2.
  *
- * ---------------------------------------------------
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "inspircd.h"
 
-/* $ModDesc: Provides the /clones command to retrieve information on clones. */
+#include "inspircd.h"
 
-/** Handle /CHECK
+/** Handle /CLONES
  */
 class CommandClones : public Command
 {
  public:
-       CommandClones (InspIRCd* Instance) : Command(Instance,"CLONES", "o", 1)
+       CommandClones(Module* Creator) : Command(Creator,"CLONES", 1)
        {
-               this->source = "m_clones.so";
-               syntax = "<limit>";
+               flags_needed = 'o'; syntax = "<limit>";
        }
 
        CmdResult Handle (const std::vector<std::string> &parameters, User *user)
        {
 
-               std::string clonesstr = "304 " + std::string(user->nick) + " :CLONES";
+               std::string clonesstr = "304 " + user->nick + " :CLONES";
 
                unsigned long limit = atoi(parameters[0].c_str());
 
@@ -37,49 +42,38 @@ class CommandClones : public Command
                 * Syntax of a /clones reply:
                 *  :server.name 304 target :CLONES START
                 *  :server.name 304 target :CLONES <count> <ip>
-                *  :server.name 304 target :CHECK END
+                *  :server.name 304 target :CLONES END
                 */
 
                user->WriteServ(clonesstr + " START");
 
                /* hostname or other */
-               // XXX I really don't like marking global_clones public for this. at all. -- w00t
-               for (clonemap::iterator x = ServerInstance->Users->global_clones.begin(); x != ServerInstance->Users->global_clones.end(); x++)
+               const UserManager::CloneMap& clonemap = ServerInstance->Users->GetCloneMap();
+               for (UserManager::CloneMap::const_iterator i = clonemap.begin(); i != clonemap.end(); ++i)
                {
-                       if (x->second >= limit)
-                               user->WriteServ(clonesstr + " "+ ConvToStr(x->second) + " " + assign(x->first));
+                       const UserManager::CloneCounts& counts = i->second;
+                       if (counts.global >= limit)
+                               user->WriteServ(clonesstr + " " + ConvToStr(counts.global) + " " + i->first.str());
                }
 
                user->WriteServ(clonesstr + " END");
 
-               return CMD_LOCALONLY;
+               return CMD_SUCCESS;
        }
 };
 
-
 class ModuleClones : public Module
 {
- private:
-       CommandClones *mycommand;
+       CommandClones cmd;
  public:
-       ModuleClones(InspIRCd* Me) : Module(Me)
+       ModuleClones() : cmd(this)
        {
-
-               mycommand = new CommandClones(ServerInstance);
-               ServerInstance->AddCommand(mycommand);
-
        }
 
-       virtual ~ModuleClones()
+       Version GetVersion() CXX11_OVERRIDE
        {
+               return Version("Provides the /CLONES command to retrieve information on clones.", VF_VENDOR);
        }
-
-       virtual Version GetVersion()
-       {
-               return Version(1, 2, 0, 0, VF_VENDOR, API_VERSION);
-       }
-
-
 };
 
 MODULE_INIT(ModuleClones)