Remove CMD_LOCALONLY, enforce use of GetRouting for routed commands

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

diff --git a/src/modules/m_jumpserver.cpp b/src/modules/m_jumpserver.cpp
index 535edb641052f9ad0ca1007b2058846d96372829..d8492fd5e2aeda292c7b1308dbac241ec0315f8f 100644 (file)
--- a/src/modules/m_jumpserver.cpp
+++ b/src/modules/m_jumpserver.cpp
@@ -3,7 +3,7 @@
  *       +------------------------------------+
  *
  *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -27,10 +27,9 @@ class CommandJumpserver : public Command
        std::string reason;
        int port;
 
-       CommandJumpserver (InspIRCd* Instance) : Command(Instance, "JUMPSERVER", "o", 0)
+       CommandJumpserver (InspIRCd* Instance, Module* Creator) : Command(Instance, Creator, "JUMPSERVER", "o", 0, 4)
        {
-               this->source = "m_jumpserver.so";
-               syntax = "[<server> <port> <+/-a> :<reason>]";
+               syntax = "[<server> <port> <+/-an> <reason>]";
                redirect_to.clear();
                reason.clear();
                port = 0;
@@ -56,7 +55,7 @@ class CommandJumpserver : public Command
 
                        port = 0;
                        redirect_to.clear();
-                       return CMD_LOCALONLY;
+                       return CMD_SUCCESS;
                }
 
                port = 0;
@@ -80,9 +79,19 @@ class CommandJumpserver : public Command
                                        case 'n':
                                                redirect_new_users = direction;
                                        break;
+                                       default:
+                                               user->WriteServ("NOTICE %s :*** Invalid JUMPSERVER flag: %c", user->nick.c_str(), *n);
+                                               return CMD_FAILURE;
+                                       break;
                                }
                        }
 
+                       if (!atoi(parameters[1].c_str()))
+                       {
+                               user->WriteServ("NOTICE %s :*** Invalid port number", user->nick.c_str());
+                               return CMD_FAILURE;
+                       }
+
                        if (redirect_all_immediately)
                        {
                                /* Redirect everyone but the oper sending the command */
@@ -117,21 +126,19 @@ class CommandJumpserver : public Command
                                        reason.c_str());
                }
 
-               return CMD_LOCALONLY;
+               return CMD_SUCCESS;
        }
 };
 
 
 class ModuleJumpServer : public Module
 {
-       CommandJumpserver*      js;
+       CommandJumpserver js;
  public:
        ModuleJumpServer(InspIRCd* Me)
-               : Module(Me)
+               : Module(Me), js(Me, this)
        {
-
-               js = new CommandJumpserver(ServerInstance);
-               ServerInstance->AddCommand(js);
+               ServerInstance->AddCommand(&js);
                Implementation eventlist[] = { I_OnUserRegister };
                ServerInstance->Modules->Attach(eventlist, this, 1);
        }
@@ -140,15 +147,16 @@ class ModuleJumpServer : public Module
        {
        }
 
-       virtual int OnUserRegister(User* user)
+       virtual ModResult OnUserRegister(User* user)
        {
-               if (js->port && js->redirect_new_users)
+               if (js.port && js.redirect_new_users)
                {
-                       user->WriteNumeric(10, "%s %s %d :Please use this Server/Port instead", user->nick.c_str(), js->redirect_to.c_str(), js->port);
-                       ServerInstance->Users->QuitUser(user, js->reason);
-                       return 0;
+                       user->WriteNumeric(10, "%s %s %d :Please use this Server/Port instead",
+                               user->nick.c_str(), js.redirect_to.c_str(), js.port);
+                       ServerInstance->Users->QuitUser(user, js.reason);
+                       return MOD_RES_PASSTHRU;
                }
-               return 0;
+               return MOD_RES_PASSTHRU;
        }