forked from mff/names_per_ip
Workaround for get_player_information (nil), improve save function
This commit is contained in:
parent
846e1848ea
commit
680c0b3743
45
init.lua
45
init.lua
@ -10,7 +10,6 @@ ipnames = {}
|
|||||||
ipnames.data = {}
|
ipnames.data = {}
|
||||||
ipnames.whitelist = {}
|
ipnames.whitelist = {}
|
||||||
ipnames.changes = false
|
ipnames.changes = false
|
||||||
ipnames.save_time = 0
|
|
||||||
ipnames.file = minetest.get_worldpath().."/ipnames.data"
|
ipnames.file = minetest.get_worldpath().."/ipnames.data"
|
||||||
ipnames.whitelist_file = minetest.get_worldpath().."/ipnames_whitelist.data"
|
ipnames.whitelist_file = minetest.get_worldpath().."/ipnames_whitelist.data"
|
||||||
|
|
||||||
@ -79,33 +78,41 @@ minetest.register_on_prejoinplayer(function(name, ip)
|
|||||||
end)
|
end)
|
||||||
|
|
||||||
-- Save IP if player joined
|
-- Save IP if player joined
|
||||||
minetest.register_on_joinplayer(function(player)
|
local function update_player_address(name, recursive)
|
||||||
local name = player:get_player_name()
|
local info = minetest.get_player_information(name)
|
||||||
local time = os.time()
|
local address = info and info.address
|
||||||
local player_info = minetest.get_player_information(name)
|
|
||||||
if not player_info.address then
|
if not address then
|
||||||
minetest.log("warning", "[names_per_ip] Failed to get the IP address for " ..
|
minetest.log("warning", "[names_per_ip] minetest.get_player_information(\"" ..
|
||||||
name .. ". This should not happen.")
|
name .. "\"): " .. dump(info) .. ". This is probably an engine bug.")
|
||||||
|
if not recursive then
|
||||||
|
-- Delay, hope it works next time
|
||||||
|
minetest.after(0.5, update_address, name, true)
|
||||||
|
return
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
ipnames.data[name] = {
|
ipnames.data[name] = {
|
||||||
player_info.address or "??",
|
address or "??",
|
||||||
time
|
os.time()
|
||||||
}
|
}
|
||||||
|
|
||||||
ipnames.changes = true
|
ipnames.changes = true
|
||||||
|
end
|
||||||
|
|
||||||
|
minetest.register_on_joinplayer(function(player)
|
||||||
|
update_player_address(player:get_player_name())
|
||||||
end)
|
end)
|
||||||
|
|
||||||
minetest.register_globalstep(function(t)
|
-- Save changes at a fixed interval
|
||||||
ipnames.save_time = ipnames.save_time + t
|
local function save_data_job()
|
||||||
if ipnames.save_time < ipnames.save_interval then
|
|
||||||
return
|
|
||||||
end
|
|
||||||
ipnames.save_time = 0
|
|
||||||
ipnames.save_data()
|
ipnames.save_data()
|
||||||
end)
|
minetest.after(ipnames.save_interval, save_data_job)
|
||||||
|
end
|
||||||
|
minetest.after(ipnames.save_interval, save_data_job)
|
||||||
|
|
||||||
minetest.register_on_shutdown(ipnames.save_data)
|
minetest.register_on_shutdown(ipnames.save_data)
|
||||||
|
|
||||||
minetest.after(3, ipnames.load_data)
|
|
||||||
minetest.after(3, ipnames.load_whitelist)
|
ipnames.load_data()
|
||||||
|
ipnames.load_whitelist()
|
||||||
|
Loading…
Reference in New Issue
Block a user