Flummi/amxmodx
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix broken admin chat access consistency and apply general optimizations in adminchat plugin (bug 5822, r=arkshine)

Browse Source 
Former-commit-id: 99caf22f48f0da39f5cbfe04b847dfa576366f5c
This commit is contained in:
connorr 2013-08-05 18:44:23 +02:00
parent 16b2721c2a
commit f3f29e4769
1 changed files with 75 additions and 76 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

107
plugins/adminchat.sma
View File

@ -58,7 +58,7 @@ public plugin_init()
register_dictionary("adminchat.txt") register_dictionary("adminchat.txt")
register_dictionary("common.txt") register_dictionary("common.txt")
register_dictionary("antiflood.txt") register_dictionary("antiflood.txt")
register_clcmd("say", "cmdSayChat", ADMIN_CHAT, "@[@|@|@][w|r|g|b|y|m|c]<text> - displays hud message") register_clcmd("say", "cmdSayChat", ADMIN_CHAT, "@[@|@|@][w|r|g|b|y|m|c]<text> - displays hud message", 1) // forces FlagManager as it is a say command
register_clcmd("say_team", "cmdSayAdmin", 0, "@<text> - displays message to admins") register_clcmd("say_team", "cmdSayAdmin", 0, "@<text> - displays message to admins")
register_concmd("amx_say", "cmdSay", ADMIN_CHAT, "<message> - sends message to all players") register_concmd("amx_say", "cmdSay", ADMIN_CHAT, "<message> - sends message to all players")
admin_chat_id = register_concmd("amx_chat", "cmdChat", ADMIN_CHAT, "<message> - sends message to admins") admin_chat_id = register_concmd("amx_chat", "cmdChat", ADMIN_CHAT, "<message> - sends message to admins")
@ -90,15 +90,15 @@ public plugin_cfg()
} }
} }
public cmdSayChat(id) public cmdSayChat(id, level)
{ {
if (!access(id, g_AdminChatFlag)) if (!access(id, level))
{ {
return PLUGIN_CONTINUE return PLUGIN_CONTINUE
} }
new said[6], i = 0 new said[6], i = 0
read_argv(1, said, 5) read_argv(1, said, charsmax(said))
while (said[i] == '@') while (said[i] == '@')
{ {
@ -111,7 +111,7 @@ public cmdSayChat(id)
} }
new message[192], a = 0 new message[192], a = 0
read_args(message, 191) read_args(message, charsmax(message))
remove_quotes(message) remove_quotes(message)
switch (said[i]) switch (said[i])
@ -140,8 +140,8 @@ public cmdSayChat(id)
new name[32], authid[32], userid new name[32], authid[32], userid
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name, 31) get_user_name(id, name, charsmax(name))
userid = get_user_userid(id) userid = get_user_userid(id)
log_amx("Chat: ^"%s<%d><%s><>^" tsay ^"%s^"", name, userid, authid, message[i + n]) log_amx("Chat: ^"%s<%d><%s><>^" tsay ^"%s^"", name, userid, authid, message[i + n])
@ -160,11 +160,12 @@ public cmdSayChat(id)
{ {
case 3, 4: case 3, 4:
{ {
new maxpl = get_maxplayers(); new players[32], plrsnum, pl
for (new pl = 1; pl <= maxpl; pl++) get_players(players, plrsnum, "ch")
{ for(new j; j<plrsnum; j++)
if (is_user_connected(pl) && !is_user_bot(pl))
{ {
pl = players[j]
if (is_user_admin(pl)) if (is_user_admin(pl))
{ {
show_hudmessage(pl, "%s : %s", name, message[i + n]) show_hudmessage(pl, "%s : %s", name, message[i + n])
@ -177,7 +178,6 @@ public cmdSayChat(id)
} }
} }
} }
}
case 2: case 2:
{ {
show_hudmessage(0, "%s : %s", name, message[i + n]) show_hudmessage(0, "%s : %s", name, message[i + n])
@ -196,7 +196,7 @@ public cmdSayChat(id)
public cmdSayAdmin(id) public cmdSayAdmin(id)
{ {
new said[2] new said[2]
read_argv(1, said, 1) read_argv(1, said, charsmax(said))
if (said[0] != '@') if (said[0] != '@')
return PLUGIN_CONTINUE return PLUGIN_CONTINUE
@ -226,33 +226,31 @@ public cmdSayAdmin(id)
} }
new message[192], name[32], authid[32], userid new message[192], name[32], authid[32], userid
new players[32], inum new players[32], inum, pl
read_args(message, 191) read_args(message, charsmax(message))
remove_quotes(message) remove_quotes(message)
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name, 31) get_user_name(id, name, charsmax(name))
userid = get_user_userid(id) userid = get_user_userid(id)
log_amx("Chat: ^"%s<%d><%s><>^" chat ^"%s^"", name, userid, authid, message[1]) log_amx("Chat: ^"%s<%d><%s><>^" chat ^"%s^"", name, userid, authid, message[1])
log_message("^"%s<%d><%s><>^" triggered ^"amx_chat^" (text ^"%s^")", name, userid, authid, message[1]) log_message("^"%s<%d><%s><>^" triggered ^"amx_chat^" (text ^"%s^")", name, userid, authid, message[1])
if (is_user_admin(id)) if (is_user_admin(id)) // no diff here if admins have g_AdminChatFlag access or not, but we don't want to print "PLAYER"
format(message, 191, "(%L) %s : %s", id, "ADMIN", name, message[1]) format(message, charsmax(message), "(%L) %s : %s", id, "ADMIN", name, message[1])
else else
format(message, 191, "(%L) %s : %s", id, "PLAYER", name, message[1]) format(message, charsmax(message), "(%L) %s : %s", id, "PLAYER", name, message[1])
get_players(players, inum) get_players(players, inum, "ch")
for (new i = 0; i < inum; ++i) for (new i = 0; i < inum; ++i)
{ {
// dont print the message to the client that used the cmd if he has ADMIN_CHAT to avoid double printing pl = players[i]
if (players[i] != id && get_user_flags(players[i]) & g_AdminChatFlag) if (pl == id || get_user_flags(pl) & g_AdminChatFlag)
client_print(players[i], print_chat, "%s", message) client_print(pl, print_chat, "%s", message)
} }
client_print(id, print_chat, "%s", message)
return PLUGIN_HANDLED return PLUGIN_HANDLED
} }
@ -261,25 +259,26 @@ public cmdChat(id, level, cid)
if (!cmd_access(id, level, cid, 2)) if (!cmd_access(id, level, cid, 2))
return PLUGIN_HANDLED return PLUGIN_HANDLED
new message[192], name[32], players[32], inum, authid[32], userid new message[192], name[32], players[32], inum, authid[32], userid, pl
read_args(message, 191) read_args(message, charsmax(message))
remove_quotes(message) remove_quotes(message)
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name, 31) get_user_name(id, name, charsmax(name))
userid = get_user_userid(id) userid = get_user_userid(id)
get_players(players, inum) get_players(players, inum, "ch")
log_amx("Chat: ^"%s<%d><%s><>^" chat ^"%s^"", name, userid, authid, message) log_amx("Chat: ^"%s<%d><%s><>^" chat ^"%s^"", name, userid, authid, message)
log_message("^"%s<%d><%s><>^" triggered ^"amx_chat^" (text ^"%s^")", name, userid, authid, message) log_message("^"%s<%d><%s><>^" triggered ^"amx_chat^" (text ^"%s^")", name, userid, authid, message)
format(message, 191, "(ADMINS) %s : %s", name, message) format(message, charsmax(message), "(ADMINS) %s : %s", name, message)
console_print(id, "%s", message) console_print(id, "%s", message)
for (new i = 0; i < inum; ++i) for (new i = 0; i < inum; ++i)
{ {
if (access(players[i], g_AdminChatFlag)) pl = players[i]
client_print(players[i], print_chat, "%s", message) if (access(pl, g_AdminChatFlag))
client_print(pl, print_chat, "%s", message)
} }
return PLUGIN_HANDLED return PLUGIN_HANDLED
@ -292,10 +291,10 @@ public cmdSay(id, level, cid)
new message[192], name[32], authid[32], userid new message[192], name[32], authid[32], userid
read_args(message, 191) read_args(message, charsmax(message))
remove_quotes(message) remove_quotes(message)
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name, 31) get_user_name(id, name, charsmax(name))
userid = get_user_userid(id) userid = get_user_userid(id)
client_print(0, print_chat, "%L", LANG_PLAYER, "PRINT_ALL", name, message) client_print(0, print_chat, "%L", LANG_PLAYER, "PRINT_ALL", name, message)
console_print(id, "%L", LANG_PLAYER, "PRINT_ALL", name, message) console_print(id, "%L", LANG_PLAYER, "PRINT_ALL", name, message)
@ -312,7 +311,7 @@ public cmdPsay(id, level, cid)
return PLUGIN_HANDLED return PLUGIN_HANDLED
new name[32] new name[32]
read_argv(1, name, 31) read_argv(1, name, charsmax(name))
new priv = cmd_target(id, name, 0) new priv = cmd_target(id, name, 0)
if (!priv) if (!priv)
@ -322,10 +321,10 @@ public cmdPsay(id, level, cid)
new message[192], name2[32], authid[32], authid2[32], userid, userid2 new message[192], name2[32], authid[32], authid2[32], userid, userid2
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name2, 31) get_user_name(id, name2, charsmax(name2))
userid = get_user_userid(id) userid = get_user_userid(id)
read_args(message, 191) read_args(message, charsmax(message))
if (message[0] == '"' && message[length] == '"') // HLSW fix if (message[0] == '"' && message[length] == '"') // HLSW fix
{ {
@ -335,14 +334,14 @@ public cmdPsay(id, level, cid)
} }
remove_quotes(message[length]) remove_quotes(message[length])
get_user_name(priv, name, 31) get_user_name(priv, name, charsmax(name))
if (id && id != priv) if (id && id != priv)
client_print(id, print_chat, "(%s) %s : %s", name, name2, message[length]) client_print(id, print_chat, "(%s) %s : %s", name, name2, message[length])
client_print(priv, print_chat, "(%s) %s : %s", name, name2, message[length]) client_print(priv, print_chat, "(%s) %s : %s", name, name2, message[length])
console_print(id, "(%s) %s : %s", name, name2, message[length]) console_print(id, "(%s) %s : %s", name, name2, message[length])
get_user_authid(priv, authid2, 31) get_user_authid(priv, authid2, charsmax(authid2))
userid2 = get_user_userid(priv) userid2 = get_user_userid(priv)
log_amx("Chat: ^"%s<%d><%s><>^" psay ^"%s<%d><%s><>^" ^"%s^"", name2, userid, authid, name, userid2, authid2, message[length]) log_amx("Chat: ^"%s<%d><%s><>^" psay ^"%s<%d><%s><>^" ^"%s^"", name2, userid, authid, name, userid2, authid2, message[length])
@ -358,12 +357,12 @@ public cmdTsay(id, level, cid)
new cmd[16], color[16], color2[16], message[192], name[32], authid[32], userid = 0 new cmd[16], color[16], color2[16], message[192], name[32], authid[32], userid = 0
read_argv(0, cmd, 15) read_argv(0, cmd, charsmax(cmd))
new bool:tsay = (tolower(cmd[4]) == 't') new bool:tsay = (tolower(cmd[4]) == 't')
read_args(message, 191) read_args(message, charsmax(message))
remove_quotes(message) remove_quotes(message)
parse(message, color, 15) parse(message, color, charsmax(color))
new found = 0, a = 0 new found = 0, a = 0
new lang[3], langnum = get_langsnum() new lang[3], langnum = get_langsnum()
@ -373,7 +372,7 @@ public cmdTsay(id, level, cid)
for (new j = 0; j < langnum; j++) for (new j = 0; j < langnum; j++)
{ {
get_lang(j, lang) get_lang(j, lang)
format(color2, 15, "%L", lang, g_Colors[i]) formatex(color2, charsmax(color2), "%L", lang, g_Colors[i])
if (equali(color, color2)) if (equali(color, color2))
{ {
@ -393,8 +392,8 @@ public cmdTsay(id, level, cid)
new Float:verpos = (tsay ? 0.55 : 0.1) + float(g_msgChannel) / 35.0 new Float:verpos = (tsay ? 0.55 : 0.1) + float(g_msgChannel) / 35.0
get_user_authid(id, authid, 31) get_user_authid(id, authid, charsmax(authid))
get_user_name(id, name, 31) get_user_name(id, name, charsmax(name))
userid = get_user_userid(id) userid = get_user_userid(id)
set_hudmessage(g_Values[a][0], g_Values[a][1], g_Values[a][2], tsay ? 0.05 : -1.0, verpos, 0, 6.0, 6.0, 0.5, 0.15, -1) set_hudmessage(g_Values[a][0], g_Values[a][1], g_Values[a][2], tsay ? 0.05 : -1.0, verpos, 0, 6.0, 6.0, 0.5, 0.15, -1)
@ -402,11 +401,12 @@ public cmdTsay(id, level, cid)
{ {
case 3, 4: case 3, 4:
{ {
new maxpl = get_maxplayers(); new players[32], plrsnum, pl
for (new pl = 1; pl <= maxpl; pl++) get_players(players, plrsnum, "ch")
{ for(new i; i<plrsnum; i++)
if (is_user_connected(pl) && !is_user_bot(pl))
{ {
pl = players[i]
if (is_user_admin(pl)) if (is_user_admin(pl))
{ {
show_hudmessage(pl, "%s : %s", name, message[length]) show_hudmessage(pl, "%s : %s", name, message[length])
@ -418,7 +418,6 @@ public cmdTsay(id, level, cid)
client_print(pl, print_notify, "%s", message[length]) client_print(pl, print_notify, "%s", message[length])
} }
} }
}
console_print(id, "%s : %s", name, message[length]) console_print(id, "%s : %s", name, message[length])
} }
case 2: case 2: