Improve log callbacks

* Remove load logging
* Remove save logging
* Log on set spawnpoint
* Log on configure variable
* Fix get configuration for `do_not_move`
This commit is contained in:
octacian 2017-07-08 17:13:32 -07:00
parent 7a6d676b36
commit f7a102396f

View File

@ -76,7 +76,6 @@ function spawnpoint.load()
if storage then if storage then
os.remove(path) os.remove(path)
end end
minetest.log("Loaded via Settings")
elseif storage and not is_empty(storage:to_table()) then elseif storage and not is_empty(storage:to_table()) then
local pos = storage:get_string("pos") local pos = storage:get_string("pos")
if pos then if pos then
@ -91,7 +90,6 @@ function spawnpoint.load()
end end
spawnpoint.time = storage:get_float("time") spawnpoint.time = storage:get_float("time")
minetest.log("Loaded via modstorage")
else else
local f = io.open(path, "r") local f = io.open(path, "r")
if f then if f then
@ -126,7 +124,6 @@ function spawnpoint.save()
storage:set_string("pos", minetest.pos_to_string(spawnpoint.pos)) storage:set_string("pos", minetest.pos_to_string(spawnpoint.pos))
end end
minetest.log("Saved via modstorage")
return true return true
elseif data then elseif data then
data:set("time", tostring(spawnpoint.time)) data:set("time", tostring(spawnpoint.time))
@ -137,7 +134,6 @@ function spawnpoint.save()
end end
data:write() data:write()
minetest.log("Saved via settings")
return true return true
end end
end end
@ -151,6 +147,7 @@ function spawnpoint.set(pos)
if type(pos) == "table" then if type(pos) == "table" then
spawnpoint.pos = pos spawnpoint.pos = pos
spawnpoint.save() spawnpoint.save()
spawnpoint.log("Set spawnpoint to "..minetest.pos_to_string(pos))
end end
end end
@ -323,15 +320,17 @@ minetest.register_chatcommand("spawnpoint", {
else else
spawnpoint.time = num spawnpoint.time = num
spawnpoint.save() spawnpoint.save()
spawnpoint.log("Set time to "..dump(num))
return true, "Set time to "..dump(num) return true, "Set time to "..dump(num)
end end
elseif p[1] == "do_not_move" then elseif p[1] == "do_not_move" then
local move = minetest.is_yes(p[2]) local move = minetest.is_yes(p[2])
if move == nil then if move == nil or not p[2] then
return true, "SpawnPoint->do_not_move: "..dump(spawnpoint.do_not_move) return true, "SpawnPoint->do_not_move: "..dump(spawnpoint.do_not_move)
else else
spawnpoint.do_not_move = move spawnpoint.do_not_move = move
spawnpoint.save() spawnpoint.save()
spawnpoint.log("Set do_not_move to "..dump(move))
return true, "Set do_not_move to "..dump(move) return true, "Set do_not_move to "..dump(move)
end end
end end