Replace OnAccessCheck with OnPreMode to remove a number of redundant checks

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

diff --git a/src/modules/m_chgident.cpp b/src/modules/m_chgident.cpp
index 58a06cbd8366104ed05439dc9b43e2d4ead3bdd8..d5269105434fa9b2c5ca419d5c2961d4c672d9e7 100644 (file)
--- a/src/modules/m_chgident.cpp
+++ b/src/modules/m_chgident.cpp
@@ -54,14 +54,24 @@ class CommandChgident : public Command
                        return CMD_FAILURE;
                }
 
-               dest->ChangeIdent(parameters[1].c_str());
+               if (IS_LOCAL(dest))
+               {
+                       dest->ChangeIdent(parameters[1].c_str());
 
-               if (!ServerInstance->ULine(user->server))
-                       ServerInstance->SNO->WriteToSnoMask(IS_LOCAL(dest) ? 'a' : 'A', "%s used CHGIDENT to change %s's ident to '%s'", user->nick.c_str(), dest->nick.c_str(), dest->ident.c_str());
+                       if (!ServerInstance->ULine(user->server))
+                               ServerInstance->SNO->WriteGlobalSno('a', "%s used CHGIDENT to change %s's ident to '%s'", user->nick.c_str(), dest->nick.c_str(), dest->ident.c_str());
+               }
 
-               /* route it! */
                return CMD_SUCCESS;
        }
+
+       RouteDescriptor GetRouting(User* user, const std::vector<std::string>& parameters)
+       {
+               User* dest = ServerInstance->FindNick(parameters[0]);
+               if (dest)
+                       return ROUTE_OPT_UCAST(dest->server);
+               return ROUTE_LOCALONLY;
+       }
 };
 
 
@@ -81,7 +91,7 @@ public:
 
        virtual Version GetVersion()
        {
-               return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
+               return Version("$Id$", VF_OPTCOMMON | VF_VENDOR, API_VERSION);
        }
 
 };