add complete input validation, better result msgs

This commit is contained in:
cora 2022-05-24 19:50:22 +02:00
parent f89780695e
commit 2499fe1e83
1 changed files with 45 additions and 11 deletions

View File

@ -186,26 +186,60 @@ function minetest.is_creative_enabled(name)
if mt_is_creative_enabled(name) then return true end if mt_is_creative_enabled(name) then return true end
local p = minetest.get_player_by_name(name) local p = minetest.get_player_by_name(name)
if p then if p then
return p:get_meta():get_string("gamemode_creative") == "true" return p:get_meta():get_string("gamemode") == "creative"
end end
return false return false
end end
local function in_table(n,h)
for k,v in pairs(h) do
if v == n then return true end
end
return false
end
function str_split(s,d)
if d == nil then d = "%s" end
local t={}
for v in string.gmatch(s, "([^"..d.."]+)") do
table.insert(t, v)
end
return t
end
local gamemodes = {
"survival",
"creative"
}
local function player_set_gamemode(p,g)
local m = p:get_meta()
m:set_string("gamemode",g)
set_inventory(p)
end
minetest.register_chatcommand("gamemode",{ minetest.register_chatcommand("gamemode",{
privs = { server = true }, privs = { server = true },
func = function(n,param) func = function(n,param)
-- Full input validation ( just for @erlehmann <3 )
local p = minetest.get_player_by_name(n) local p = minetest.get_player_by_name(n)
if not p then return end local args = str_split(param)
local m = p:get_meta() if args[2] ~= nil then
local gm = "survival" p = minetest.get_player_by_name(args[2])
if param == "creative" then
m:set_string("gamemode_creative","true")
gm = "creative"
set_inventory(p)
elseif param == "survival" then
m:set_string("gamemode_creative","")
set_inventory(p)
end end
if not p then
minetest.chat_send_player(n,S("Player not online"))
return
end
if args[1] ~= nil and not in_table(args[1],gamemodes) then
minetest.chat_send_player(n,S("Gamemode "..tostring(args[1]).." does not exist." ))
return
elseif args[1] ~= nil then
player_set_gamemode(p,args[1])
end
--Result message - show effective game mode
local gm = p:get_meta():get_string("gamemode")
if gm == "" then gm = gamemodes[1] end
minetest.chat_send_player(n,S("Gamemode for player ")..n..S(": "..gm)) minetest.chat_send_player(n,S("Gamemode for player ")..n..S(": "..gm))
end end
}) })