]> git.netwichtig.de Git - user/henk/code/inspircd.git/commitdiff
Send ERR_CANTUNLOADMODULE when unloading a module on reload fails.
authorSadie Powell <sadie@witchery.services>
Tue, 19 Jan 2021 07:19:48 +0000 (07:19 +0000)
committerSadie Powell <sadie@witchery.services>
Tue, 19 Jan 2021 07:19:48 +0000 (07:19 +0000)
src/coremods/core_reloadmodule.cpp
src/modules/m_globalload.cpp

index 98bf2ab405b17f55665c2c0fc11230b65c530ae8..34eaeae186ee04f378dbda557967c52583157dfc 100644 (file)
@@ -736,7 +736,10 @@ class ReloadAction : public ActionBase
                ServerInstance->SNO->WriteGlobalSno('a', "RELOAD MODULE: %s %ssuccessfully reloaded", passedname.c_str(), result ? "" : "un");
                User* user = ServerInstance->FindUUID(uuid);
                if (user)
-                       user->WriteNumeric(RPL_LOADEDMODULE, passedname, InspIRCd::Format("Module %ssuccessfully reloaded.", (result ? "" : "un")));
+               {
+                       int numeric = result ? RPL_LOADEDMODULE : ERR_CANTUNLOADMODULE;
+                       user->WriteNumeric(numeric, passedname, InspIRCd::Format("Module %ssuccessfully reloaded.", (result ? "" : "un")));
+               }
 
                ServerInstance->GlobalCulls.AddItem(this);
        }
@@ -747,7 +750,7 @@ CmdResult CommandReloadmodule::Handle(User* user, const Params& parameters)
        Module* m = ServerInstance->Modules->Find(parameters[0]);
        if (m == creator)
        {
-               user->WriteNumeric(RPL_LOADEDMODULE, parameters[0], "You cannot reload core_reloadmodule (unload and load it)");
+               user->WriteNumeric(ERR_CANTUNLOADMODULE, parameters[0], "You cannot reload core_reloadmodule (unload and load it)");
                return CMD_FAILURE;
        }
 
@@ -761,7 +764,7 @@ CmdResult CommandReloadmodule::Handle(User* user, const Params& parameters)
        }
        else
        {
-               user->WriteNumeric(RPL_LOADEDMODULE, parameters[0], "Could not find module by that name");
+               user->WriteNumeric(ERR_CANTUNLOADMODULE, parameters[0], "Could not find module by that name");
                return CMD_FAILURE;
        }
 }
index f1961f6b45294114a003c7fbc7976a8506fd5e50..15c41502f29865f0dc47ea212111f5f3946d7f43 100644 (file)
@@ -143,7 +143,7 @@ class CommandGReloadModule : public Command
                        }
                        else
                        {
-                               user->WriteRemoteNumeric(RPL_LOADEDMODULE, parameters[0], "Could not find module by that name");
+                               user->WriteRemoteNumeric(ERR_CANTUNLOADMODULE, parameters[0], "Could not find module by that name");
                                return CMD_FAILURE;
                        }
                }