diff options
author | special <special@e03df62e-2008-0410-955e-edbf42e46eb7> | 2007-05-24 23:36:39 +0000 |
---|---|---|
committer | special <special@e03df62e-2008-0410-955e-edbf42e46eb7> | 2007-05-24 23:36:39 +0000 |
commit | a338648303c208517a3261d9aa3908472a2546e3 (patch) | |
tree | 7394077612a82c3544d521d99b2035fbc5c5c50f /src | |
parent | 6427e2077e3881845650e0dc9bb37839c17dfca3 (diff) |
Propagate the gunloadmodule command even if the module isn't loaded locally; bug #299
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@7132 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src')
-rw-r--r-- | src/modules/m_globalload.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/modules/m_globalload.cpp b/src/modules/m_globalload.cpp index 1d5e8f18b..e8ea11378 100644 --- a/src/modules/m_globalload.cpp +++ b/src/modules/m_globalload.cpp @@ -42,7 +42,10 @@ class cmd_gloadmodule : public command_t else { user->WriteServ("974 %s %s :Failed to load module: %s",user->nick, parameters[0],ServerInstance->ModuleError()); - /* XXX - returning CMD_FAILURE here could potentially mean half the net loads it, half doesn't. pass it on anyway? -- w00t */ + /* XXX - returning CMD_FAILURE here could potentially mean half the net loads it, half doesn't. pass it on anyway? -- w00t + * + * Returning CMD_SUCCESS would have the same effect, just with less servers. Someone should update this module to properly + * pass the success/failure for each server to the caller (or to all opers) -Special */ return CMD_FAILURE; } } @@ -65,15 +68,13 @@ class cmd_gunloadmodule : public command_t { ServerInstance->WriteOpers("*** MODULE '%s' GLOBALLY UNLOADED BY '%s'",parameters[0],user->nick); user->WriteServ("973 %s %s :Module successfully unloaded.",user->nick, parameters[0]); - /* route it! */ - return CMD_SUCCESS; } else { - /* XXX - see above note about returning CMD_FAILURE here -- w00t */ + /* Return CMD_SUCCESS so the module will be unloaded on any servers it is loaded on - this is a seperate case entirely from loading -Special */ user->WriteServ("972 %s %s :Failed to unload module: %s",user->nick, parameters[0],ServerInstance->ModuleError()); - return CMD_FAILURE; } + return CMD_SUCCESS; } }; |