Prettified command error messages
Using the magic of overlords (overloads) and standards compliance.
This commit is contained in:
parent
aaaa53ce9d
commit
b4ac4a14fa
|
@ -9,6 +9,7 @@
|
||||||
#include "../Root.h"
|
#include "../Root.h"
|
||||||
#include "../Server.h"
|
#include "../Server.h"
|
||||||
#include "../CommandOutput.h"
|
#include "../CommandOutput.h"
|
||||||
|
#include "../ChatColor.h"
|
||||||
|
|
||||||
#include "inifile/iniFile.h"
|
#include "inifile/iniFile.h"
|
||||||
#include "../Entities/Player.h"
|
#include "../Entities/Player.h"
|
||||||
|
@ -230,19 +231,25 @@ bool cPluginManager::CallHookBlockToPickups(
|
||||||
|
|
||||||
bool cPluginManager::CallHookChat(cPlayer * a_Player, AString & a_Message)
|
bool cPluginManager::CallHookChat(cPlayer * a_Player, AString & a_Message)
|
||||||
{
|
{
|
||||||
if (ExecuteCommand(a_Player, a_Message))
|
bool WasCommandForbidden = false;
|
||||||
|
if (HandleCommand(a_Player, a_Message, true, WasCommandForbidden)) // We use HandleCommand as opposed to ExecuteCommand to accomodate the need to the WasCommandForbidden bool
|
||||||
{
|
{
|
||||||
return true;
|
return true; // Chat message was handled as command
|
||||||
|
}
|
||||||
|
else if (WasCommandForbidden) // Couldn't be handled as command, was it because of insufficient permissions?
|
||||||
|
{
|
||||||
|
return true; // Yes - message was sent in HandleCommand, abort
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check if it was a standard command (starts with a slash)
|
// Check if it was a standard command (starts with a slash)
|
||||||
|
// If it was, we know that it was completely unrecognised (WasCommandForbidden == false)
|
||||||
if (!a_Message.empty() && (a_Message[0] == '/'))
|
if (!a_Message.empty() && (a_Message[0] == '/'))
|
||||||
{
|
{
|
||||||
AStringVector Split(StringSplit(a_Message, " "));
|
AStringVector Split(StringSplit(a_Message, " "));
|
||||||
ASSERT(!Split.empty()); // This should not happen - we know there's at least one char in the message so the split needs to be at least one item long
|
ASSERT(!Split.empty()); // This should not happen - we know there's at least one char in the message so the split needs to be at least one item long
|
||||||
a_Player->SendMessage(Printf("Unknown Command: \"%s\"", Split[0].c_str()));
|
a_Player->SendMessage(Printf("%s[INFO] %sUnknown command: \"%s\"", cChatColor::Yellow.c_str(), cChatColor::White.c_str(), Split[0].c_str()));
|
||||||
LOGINFO("Player \"%s\" issued an unknown command: \"%s\"", a_Player->GetName().c_str(), a_Message.c_str());
|
LOGINFO("Player %s issued an unknown command: \"%s\"", a_Player->GetName().c_str(), a_Message.c_str());
|
||||||
return true; // Cancel sending
|
return true; // Cancel sending
|
||||||
}
|
}
|
||||||
|
|
||||||
HookMap::iterator Plugins = m_Hooks.find(HOOK_CHAT);
|
HookMap::iterator Plugins = m_Hooks.find(HOOK_CHAT);
|
||||||
|
@ -1251,7 +1258,7 @@ bool cPluginManager::CallHookWorldTick(cWorld & a_World, float a_Dt, int a_LastT
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
bool cPluginManager::HandleCommand(cPlayer * a_Player, const AString & a_Command, bool a_ShouldCheckPermissions)
|
bool cPluginManager::HandleCommand(cPlayer * a_Player, const AString & a_Command, bool a_ShouldCheckPermissions, bool & a_WasCommandForbidden)
|
||||||
{
|
{
|
||||||
ASSERT(a_Player != NULL);
|
ASSERT(a_Player != NULL);
|
||||||
|
|
||||||
|
@ -1271,7 +1278,7 @@ bool cPluginManager::HandleCommand(cPlayer * a_Player, const AString & a_Command
|
||||||
// Ask plugins first if a command is okay to execute the command:
|
// Ask plugins first if a command is okay to execute the command:
|
||||||
if (CallHookExecuteCommand(a_Player, Split))
|
if (CallHookExecuteCommand(a_Player, Split))
|
||||||
{
|
{
|
||||||
LOGINFO("Player \"%s\" tried executing command \"%s\" that was stopped by the HOOK_EXECUTE_COMMAND hook", a_Player->GetName().c_str(), Split[0].c_str());
|
LOGINFO("Player %s tried executing command \"%s\" that was stopped by the HOOK_EXECUTE_COMMAND hook", a_Player->GetName().c_str(), Split[0].c_str());
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1281,7 +1288,9 @@ bool cPluginManager::HandleCommand(cPlayer * a_Player, const AString & a_Command
|
||||||
!a_Player->HasPermission(cmd->second.m_Permission)
|
!a_Player->HasPermission(cmd->second.m_Permission)
|
||||||
)
|
)
|
||||||
{
|
{
|
||||||
LOGINFO("Player \"%s\" tried to execute forbidden command \"%s\".", a_Player->GetName().c_str(), Split[0].c_str());
|
a_Player->SendMessage(Printf("%s[INFO] %sForbidden command; insufficient privileges: \"%s\"", cChatColor::Rose.c_str(), cChatColor::White.c_str(), Split[0].c_str()));
|
||||||
|
LOGINFO("Player %s tried to execute forbidden command: \"%s\"", a_Player->GetName().c_str(), Split[0].c_str());
|
||||||
|
a_WasCommandForbidden = true;
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -289,7 +289,12 @@ private:
|
||||||
bool AddPlugin(cPlugin * a_Plugin);
|
bool AddPlugin(cPlugin * a_Plugin);
|
||||||
|
|
||||||
/// Tries to match a_Command to the internal table of commands, if a match is found, the corresponding plugin is called. Returns true if the command is handled.
|
/// Tries to match a_Command to the internal table of commands, if a match is found, the corresponding plugin is called. Returns true if the command is handled.
|
||||||
bool HandleCommand(cPlayer * a_Player, const AString & a_Command, bool a_ShouldCheckPermissions);
|
bool HandleCommand(cPlayer * a_Player, const AString & a_Command, bool a_ShouldCheckPermissions, bool & a_WasCommandForbidden);
|
||||||
|
bool HandleCommand(cPlayer * a_Player, const AString & a_Command, bool a_ShouldCheckPermissions)
|
||||||
|
{
|
||||||
|
bool DummyBoolean = false;
|
||||||
|
return HandleCommand(a_Player, a_Command, a_ShouldCheckPermissions, DummyBoolean);
|
||||||
|
}
|
||||||
} ; // tolua_export
|
} ; // tolua_export
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user