Only assign NewServices once the duplicate check is done.

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

diff --git a/src/modules/m_spanningtree/idle.cpp b/src/modules/m_spanningtree/idle.cpp
index 972707ca23ad4efb30f51892e5b6a77564842a23..8e075e312f9a6e96d96c2884c9d04591cd853103 100644 (file)
--- a/src/modules/m_spanningtree/idle.cpp
+++ b/src/modules/m_spanningtree/idle.cpp
@@ -1,7 +1,12 @@
 /*
  * InspIRCd -- Internet Relay Chat Daemon
  *
- *   Copyright (C) 2011 Adam <Adam@anope.org>
+ *   Copyright (C) 2018-2019 Sadie Powell <sadie@witchery.services>
+ *   Copyright (C) 2012-2015 Attila Molnar <attilamolnar@hush.com>
+ *   Copyright (C) 2012 Robby <robby@chatbelgie.be>
+ *   Copyright (C) 2009 Uli Schlachter <psychon@inspircd.org>
+ *   Copyright (C) 2009 Daniel De Graaf <danieldg@inspircd.org>
+ *   Copyright (C) 2008 Robin Burchell <robin+git@viroteck.net>
  *
  * 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
@@ -60,9 +65,9 @@ CmdResult CommandIdle::HandleRemote(RemoteUser* issuer, Params& params)
                        idle = ((unsigned int) (ServerInstance->Time() - localtarget->idle_lastmsg));
 
                CmdBuilder reply(target, "IDLE");
-               reply.push_back(issuer->uuid);
-               reply.push_back(ConvToStr(target->signon));
-               reply.push_back(ConvToStr(idle));
+               reply.push(issuer->uuid);
+               reply.push(ConvToStr(target->signon));
+               reply.push(ConvToStr(idle));
                reply.Unicast(issuer);
        }