Move all local-only fields to LocalUser

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

diff --git a/src/usermanager.cpp b/src/usermanager.cpp
index db2d3b08e6a55899950fe513d486711b38b243af..2d824c6b2ff188417a971ec57f54fd57ecebe26c 100644 (file)
--- a/src/usermanager.cpp
+++ b/src/usermanager.cpp
@@ -23,10 +23,10 @@ void UserManager::AddUser(int socket, ClientListenSocket* via, irc::sockets::soc
        /* NOTE: Calling this one parameter constructor for User automatically
         * allocates a new UUID and places it in the hash_map.
         */
-       User* New = NULL;
+       LocalUser* New = NULL;
        try
        {
-               New = new User();
+               New = new LocalUser();
        }
        catch (...)
        {
@@ -63,7 +63,7 @@ void UserManager::AddUser(int socket, ClientListenSocket* via, irc::sockets::soc
        /* The users default nick is their UUID */
        New->nick.assign(New->uuid, 0, ServerInstance->Config->Limits.NickMax);
 
-       New->server = ServerInstance->FindServerNamePtr(ServerInstance->Config->ServerName);
+       New->server = ServerInstance->Config->ServerName;
        New->ident.assign("unknown");
 
        New->registered = REG_NONE;
@@ -81,13 +81,7 @@ void UserManager::AddUser(int socket, ClientListenSocket* via, irc::sockets::soc
         * First class check. We do this again in FullConnect after DNS is done, and NICK/USER is recieved.
         * See my note down there for why this is required. DO NOT REMOVE. :) -- w00t
         */
-       ConnectClass* i = New->SetClass();
-
-       if (!i)
-       {
-               this->QuitUser(New, "Access denied by configuration");
-               return;
-       }
+       New->SetClass();
 
        /*
         * Check connect class settings and initialise settings into User.
@@ -117,8 +111,8 @@ void UserManager::AddUser(int socket, ClientListenSocket* via, irc::sockets::soc
                {
                        /* user banned */
                        ServerInstance->Logs->Log("BANCACHE", DEBUG, std::string("BanCache: Positive hit for ") + New->GetIPString());
-                       if (*ServerInstance->Config->MoronBanner)
-                               New->WriteServ("NOTICE %s :*** %s", New->nick.c_str(), ServerInstance->Config->MoronBanner);
+                       if (!ServerInstance->Config->MoronBanner.empty())
+                               New->WriteServ("NOTICE %s :*** %s", New->nick.c_str(), ServerInstance->Config->MoronBanner.c_str());
                        this->QuitUser(New, b->Reason);
                        return;
                }
@@ -171,7 +165,7 @@ void UserManager::QuitUser(User *user, const std::string &quitreason, const char
                return;
        }
 
-       if (IS_FAKE(user))
+       if (IS_SERVER(user))
        {
                ServerInstance->Logs->Log("CULLLIST",DEBUG, "*** Warning *** - You tried to quit a fake user (%s)", user->nick.c_str());
                return;
@@ -195,18 +189,16 @@ void UserManager::QuitUser(User *user, const std::string &quitreason, const char
        if (user->registered == REG_ALL)
        {
                FOREACH_MOD(I_OnUserQuit,OnUserQuit(user, reason, oper_reason));
-               user->PurgeEmptyChannels();
                user->WriteCommonQuit(reason, oper_reason);
        }
 
-       FOREACH_MOD(I_OnUserDisconnect,OnUserDisconnect(user));
-
        if (user->registered != REG_ALL)
                if (ServerInstance->Users->unregistered_count)
                        ServerInstance->Users->unregistered_count--;
 
        if (IS_LOCAL(user))
        {
+               FOREACH_MOD(I_OnUserDisconnect,OnUserDisconnect(IS_LOCAL(user)));
                user->DoWrite();
                if (user->GetIOHook())
                {
@@ -245,7 +237,7 @@ void UserManager::QuitUser(User *user, const std::string &quitreason, const char
                        if ((!ServerInstance->SilentULine(user->server)) && (!user->quietquit))
                        {
                                ServerInstance->SNO->WriteToSnoMask('Q',"Client exiting on server %s: %s!%s@%s [%s]",
-                                       user->server, user->nick.c_str(), user->ident.c_str(), user->host.c_str(), oper_reason.c_str());
+                                       user->server.c_str(), user->nick.c_str(), user->ident.c_str(), user->host.c_str(), oper_reason.c_str());
                        }
                }
                user->AddToWhoWas();
@@ -422,9 +414,9 @@ void UserManager::ServerNoticeAll(const char* text, ...)
        vsnprintf(textbuffer, MAXBUF, text, argsPtr);
        va_end(argsPtr);
 
-       snprintf(formatbuffer,MAXBUF,"NOTICE $%s :%s", ServerInstance->Config->ServerName, textbuffer);
+       snprintf(formatbuffer,MAXBUF,"NOTICE $%s :%s", ServerInstance->Config->ServerName.c_str(), textbuffer);
 
-       for (std::vector<User*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
+       for (std::vector<LocalUser*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
        {
                User* t = *i;
                t->WriteServ(std::string(formatbuffer));
@@ -443,9 +435,9 @@ void UserManager::ServerPrivmsgAll(const char* text, ...)
        vsnprintf(textbuffer, MAXBUF, text, argsPtr);
        va_end(argsPtr);
 
-       snprintf(formatbuffer,MAXBUF,"PRIVMSG $%s :%s", ServerInstance->Config->ServerName, textbuffer);
+       snprintf(formatbuffer,MAXBUF,"PRIVMSG $%s :%s", ServerInstance->Config->ServerName.c_str(), textbuffer);
 
-       for (std::vector<User*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
+       for (std::vector<LocalUser*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
        {
                User* t = *i;
                t->WriteServ(std::string(formatbuffer));
@@ -471,7 +463,7 @@ void UserManager::WriteMode(const char* modes, int flags, const char* text, ...)
 
        if (flags == WM_AND)
        {
-               for (std::vector<User*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
+               for (std::vector<LocalUser*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
                {
                        User* t = *i;
                        bool send_to_user = true;
@@ -492,7 +484,7 @@ void UserManager::WriteMode(const char* modes, int flags, const char* text, ...)
        }
        else if (flags == WM_OR)
        {
-               for (std::vector<User*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
+               for (std::vector<LocalUser*>::const_iterator i = local_users.begin(); i != local_users.end(); i++)
                {
                        User* t = *i;
                        bool send_to_user = false;