git/newplayer-cd2025
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't crash if a command is run by a player that's not actually online

Browse Source 
This can happen, for example, if the command is run by a command block owned by a player that has since left.
This commit is contained in:
cheapie 2020-04-26 21:04:13 -05:00
parent e744e27e65
commit f46262b691
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
init.lua
View File

@ -228,7 +228,9 @@ minetest.register_chatcommand("set_no_interact_spawn",{
description = "Set the spawn point for players without interact to your current position", description = "Set the spawn point for players without interact to your current position",
privs = {server=true}, privs = {server=true},
func = function(name) func = function(name)
local pos = minetest.get_player_by_name(name):getpos() local player = minetest.get_player_by_name(name)
if not player then return end
local pos = player:get_pos()
minetest.setting_set("spawnpoint_no_interact",string.format("%s,%s,%s",pos.x,pos.y,pos.z)) minetest.setting_set("spawnpoint_no_interact",string.format("%s,%s,%s",pos.x,pos.y,pos.z))
minetest.setting_save() minetest.setting_save()
return true, newplayer.colorize("#55FF55","Success: ").."Spawn point for players without interact set to: "..newplayer.colorize("#00FFFF",minetest.pos_to_string(pos)) return true, newplayer.colorize("#55FF55","Success: ").."Spawn point for players without interact set to: "..newplayer.colorize("#00FFFF",minetest.pos_to_string(pos))
@ -240,7 +242,9 @@ minetest.register_chatcommand("set_interact_spawn",{
description = "Set the spawn point for players with interact to your current position", description = "Set the spawn point for players with interact to your current position",
privs = {server=true}, privs = {server=true},
func = function(name) func = function(name)
local pos = minetest.get_player_by_name(name):getpos() local player = minetest.get_player_by_name(name)
if not player then return end
local pos = player:get_pos()
minetest.setting_set("spawnpoint_interact",string.format("%s,%s,%s",pos.x,pos.y,pos.z)) minetest.setting_set("spawnpoint_interact",string.format("%s,%s,%s",pos.x,pos.y,pos.z))
minetest.setting_save() minetest.setting_save()
return true, newplayer.colorize("#55FF55","Success: ").."Spawn point for players with interact set to: "..newplayer.colorize("#00FFFF",minetest.pos_to_string(pos)) return true, newplayer.colorize("#55FF55","Success: ").."Spawn point for players with interact set to: "..newplayer.colorize("#00FFFF",minetest.pos_to_string(pos))
@ -302,12 +306,13 @@ minetest.register_chatcommand("spawn",{
params = "", params = "",
description = "Teleport to the spawn", description = "Teleport to the spawn",
func = function(name) func = function(name)
local hasinteract = minetest.check_player_privs(name,{interact=true})
local player = minetest.get_player_by_name(name) local player = minetest.get_player_by_name(name)
if not player then return end
local hasinteract = minetest.check_player_privs(name,{interact=true})
if hasinteract then if hasinteract then
local pos = minetest.setting_get_pos("spawnpoint_interact") local pos = minetest.setting_get_pos("spawnpoint_interact")
if pos then if pos then
player:setpos(pos) player:set_pos(pos)
return true, "Teleporting to spawn..." return true, "Teleporting to spawn..."
else else
return true, newplayer.colorize("#FF0000","ERROR: ").."The spawn point is not set!" return true, newplayer.colorize("#FF0000","ERROR: ").."The spawn point is not set!"
@ -315,7 +320,7 @@ minetest.register_chatcommand("spawn",{
else else
local pos = minetest.setting_get_pos("spawnpoint_no_interact") local pos = minetest.setting_get_pos("spawnpoint_no_interact")
if pos then if pos then
player:setpos(pos) player:set_pos(pos)
return true, "Teleporting to spawn..." return true, "Teleporting to spawn..."
else else
return true, newplayer.colorize("#FF0000","ERROR: ").."The spawn point is not set!" return true, newplayer.colorize("#FF0000","ERROR: ").."The spawn point is not set!"