- if (ServerInstance->LoadModule(parameters[0]))
- {
- WriteOpers("*** NEW MODULE '%s' GLOBALLY LOADED BY '%s'",parameters[0],user->nick);
- WriteServ(user->fd,"975 %s %s :Module successfully loaded.",user->nick, parameters[0]);
- }
- else
- {
- WriteServ(user->fd,"974 %s %s :Failed to load module: %s",user->nick, parameters[0],ServerInstance->ModuleError());
- }
+ if (ServerInstance->LoadModule(parameters[0]))
+ {
+ ServerInstance->WriteOpers("*** NEW MODULE '%s' GLOBALLY LOADED BY '%s'",parameters[0],user->nick);
+ user->WriteServ("975 %s %s :Module successfully loaded.",user->nick, parameters[0]);
+
+ /* route it! */
+ return CMD_SUCCESS;
+ }
+ 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
+ *
+ * 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;
+ }