Convert the account login event to use the new cross-module event system

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

diff --git a/src/modules/m_ircv3.cpp b/src/modules/m_ircv3.cpp
index 4eb54d2a6f7af6413999b3968868ff23740a6d0a..b80c110f4a60cbfb568a63258f4b6dc5241a55d7 100644 (file)
--- a/src/modules/m_ircv3.cpp
+++ b/src/modules/m_ircv3.cpp
@@ -20,7 +20,27 @@
 #include "modules/account.h"
 #include "modules/cap.h"
 
-class ModuleIRCv3 : public Module
+class WriteNeighboursWithExt : public User::ForEachNeighborHandler
+{
+       const LocalIntExt& ext;
+       const std::string& msg;
+
+       void Execute(LocalUser* user) CXX11_OVERRIDE
+       {
+               if (ext.get(user))
+                       user->Write(msg);
+       }
+
+ public:
+       WriteNeighboursWithExt(User* user, const std::string& message, const LocalIntExt& extension)
+               : ext(extension)
+               , msg(message)
+       {
+               user->ForEachNeighbor(*this, false);
+       }
+};
+
+class ModuleIRCv3 : public Module, public AccountEventListener
 {
        GenericCap cap_accountnotify;
        GenericCap cap_awaynotify;
@@ -31,46 +51,10 @@ class ModuleIRCv3 : public Module
 
        CUList last_excepts;
 
-       void WriteNeighboursWithExt(User* user, const std::string& line, const LocalIntExt& ext)
-       {
-               IncludeChanList chans(user->chans.begin(), user->chans.end());
-
-               std::map<User*, bool> exceptions;
-               FOREACH_MOD(OnBuildNeighborList, (user, chans, exceptions));
-
-               // Send it to all local users who were explicitly marked as neighbours by modules and have the required ext
-               for (std::map<User*, bool>::const_iterator i = exceptions.begin(); i != exceptions.end(); ++i)
-               {
-                       LocalUser* u = IS_LOCAL(i->first);
-                       if ((u) && (i->second) && (ext.get(u)))
-                               u->Write(line);
-               }
-
-               // Now consider sending it to all other users who has at least a common channel with the user
-               std::set<User*> already_sent;
-               for (IncludeChanList::const_iterator i = chans.begin(); i != chans.end(); ++i)
-               {
-                       const Channel::MemberMap& userlist = (*i)->chan->GetUsers();
-                       for (Channel::MemberMap::const_iterator m = userlist.begin(); m != userlist.end(); ++m)
-                       {
-                               /*
-                                * Send the line if the channel member in question meets all of the following criteria:
-                                * - local
-                                * - not the user who is doing the action (i.e. whose channels we're iterating)
-                                * - has the given extension
-                                * - not on the except list built by modules
-                                * - we haven't sent the line to the member yet
-                                *
-                                */
-                               LocalUser* member = IS_LOCAL(m->first);
-                               if ((member) && (member != user) && (ext.get(member)) && (exceptions.find(member) == exceptions.end()) && (already_sent.insert(member).second))
-                                       member->Write(line);
-                       }
-               }
-       }
-
  public:
-       ModuleIRCv3() : cap_accountnotify(this, "account-notify"),
+       ModuleIRCv3()
+               : AccountEventListener(this)
+               , cap_accountnotify(this, "account-notify"),
                                        cap_awaynotify(this, "away-notify"),
                                        cap_extendedjoin(this, "extended-join")
        {
@@ -92,25 +76,21 @@ class ModuleIRCv3 : public Module
                        cap_extendedjoin.HandleEvent(ev);
 
                if (accountnotify)
-               {
                        cap_accountnotify.HandleEvent(ev);
+       }
 
-                       if (ev.id == "account_login")
-                       {
-                               AccountEvent* ae = static_cast<AccountEvent*>(&ev);
-
-                               // :nick!user@host ACCOUNT account
-                               // or
-                               // :nick!user@host ACCOUNT *
-                               std::string line =  ":" + ae->user->GetFullHost() + " ACCOUNT ";
-                               if (ae->account.empty())
-                                       line += "*";
-                               else
-                                       line += std::string(ae->account);
-
-                               WriteNeighboursWithExt(ae->user, line, cap_accountnotify.ext);
-                       }
-               }
+       void OnAccountChange(User* user, const std::string& newaccount) CXX11_OVERRIDE
+       {
+               // :nick!user@host ACCOUNT account
+               // or
+               // :nick!user@host ACCOUNT *
+               std::string line = ":" + user->GetFullHost() + " ACCOUNT ";
+               if (newaccount.empty())
+                       line += "*";
+               else
+                       line += newaccount;
+
+               WriteNeighboursWithExt(user, line, cap_accountnotify.ext);
        }
 
        void OnUserJoin(Membership* memb, bool sync, bool created, CUList& excepts) CXX11_OVERRIDE