Skip to content

Commit

Permalink
Remove "Server -!- " prefix from player messages
Browse files Browse the repository at this point in the history
  • Loading branch information
ShadowNinja committed Feb 27, 2014
1 parent 18fd913 commit 9a3b771
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 15 deletions.
2 changes: 1 addition & 1 deletion builtin/chatcommands.lua
Expand Up @@ -710,7 +710,7 @@ minetest.register_chatcommand("msg", {
if found then
if minetest.get_player_by_name(sendto) then
minetest.log("action", "PM from "..name.." to "..sendto..": "..message)
minetest.chat_send_player(sendto, "PM from "..name..": "..message, false)
minetest.chat_send_player(sendto, "PM from "..name..": "..message)
minetest.chat_send_player(name, "Message sent")
else
minetest.chat_send_player(name, "The player "..sendto.." is not online")
Expand Down
3 changes: 1 addition & 2 deletions doc/lua_api.txt
Expand Up @@ -1327,8 +1327,7 @@ minetest.get_player_ip(name) -> IP address string

Chat:
minetest.chat_send_all(text)
minetest.chat_send_player(name, text, prepend)
^ prepend: optional, if it is set to false "Server -!- " will not be prepended to the message
minetest.chat_send_player(name, text)

Environment access:

Expand Down
8 changes: 2 additions & 6 deletions src/script/lua_api/l_server.cpp
Expand Up @@ -52,21 +52,17 @@ int ModApiServer::l_chat_send_all(lua_State *L)
return 0;
}

// chat_send_player(name, text, prepend)
// chat_send_player(name, text)
int ModApiServer::l_chat_send_player(lua_State *L)
{
NO_MAP_LOCK_REQUIRED;
const char *name = luaL_checkstring(L, 1);
const char *text = luaL_checkstring(L, 2);
bool prepend = true;

if (lua_isboolean(L, 3))
prepend = lua_toboolean(L, 3);

// Get server from registry
Server *server = getServer(L);
// Send
server->notifyPlayer(name, narrow_to_wide(text), prepend);
server->notifyPlayer(name, narrow_to_wide(text));
return 0;
}

Expand Down
7 changes: 2 additions & 5 deletions src/server.cpp
Expand Up @@ -4385,7 +4385,7 @@ std::string Server::getBanDescription(const std::string &ip_or_name)
return m_banmanager->getBanDescription(ip_or_name);
}

void Server::notifyPlayer(const char *name, const std::wstring msg, const bool prepend = true)
void Server::notifyPlayer(const char *name, const std::wstring msg)
{
Player *player = m_env->getPlayer(name);
if(!player)
Expand All @@ -4394,10 +4394,7 @@ void Server::notifyPlayer(const char *name, const std::wstring msg, const bool p
if (player->peer_id == PEER_ID_INEXISTENT)
return;

if (prepend)
SendChatMessage(player->peer_id, std::wstring(L"Server -!- ")+msg);
else
SendChatMessage(player->peer_id, msg);
SendChatMessage(player->peer_id, msg);
}

bool Server::showFormspec(const char *playername, const std::string &formspec, const std::string &formname)
Expand Down
2 changes: 1 addition & 1 deletion src/server.h
Expand Up @@ -229,7 +229,7 @@ class Server : public con::PeerHandler, public MapEventReceiver,
void unsetIpBanned(const std::string &ip_or_name);
std::string getBanDescription(const std::string &ip_or_name);

void notifyPlayer(const char *name, const std::wstring msg, const bool prepend);
void notifyPlayer(const char *name, const std::wstring msg);
void notifyPlayers(const std::wstring msg);
void spawnParticle(const char *playername,
v3f pos, v3f velocity, v3f acceleration,
Expand Down

0 comments on commit 9a3b771

Please sign in to comment.