mtcontrib/minetest-toolranks
1
0
Fork 1
mirror of https://github.com/lisacvuk/minetest-toolranks.git synced 2025-01-08 15:10:16 +01:00
Code Issues Releases Wiki Activity

Add nil check afteruse

Browse Source 
Add player nil check to toolranks.new_afteruse() to fix error.
This commit is contained in:
tenplus1 2020-11-22 14:32:59 +00:00 committed by GitHub
parent 12af618f9b
commit 8010feb5bf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
init.lua
View File

@ -63,11 +63,13 @@ end
function toolranks.new_afteruse(itemstack, user, node, digparams) function toolranks.new_afteruse(itemstack, user, node, digparams)
local itemmeta = itemstack:get_meta() local itemmeta = itemstack:get_meta()
local itemdef = itemstack:get_definition() local itemdef = itemstack:get_definition()
local itemdesc = itemdef.original_description local itemdesc = itemdef.original_description or ""
local dugnodes = tonumber(itemmeta:get_string("dug")) or 0 local dugnodes = tonumber(itemmeta:get_string("dug")) or 0
local lastlevel = tonumber(itemmeta:get_string("lastlevel")) or 0 local lastlevel = tonumber(itemmeta:get_string("lastlevel")) or 0
local most_digs = mod_storage:get_int("most_digs") or 0 local most_digs = mod_storage:get_int("most_digs") or 0
local most_digs_user = mod_storage:get_string("most_digs_user") or 0 local most_digs_user = mod_storage:get_string("most_digs_user") or 0
local pname = user:get_player_name()
if not pname then return itemstack end -- player nil check
if digparams.wear > 0 then -- Only count nodes that spend the tool if digparams.wear > 0 then -- Only count nodes that spend the tool
dugnodes = dugnodes + 1 dugnodes = dugnodes + 1
@ -75,24 +77,24 @@ function toolranks.new_afteruse(itemstack, user, node, digparams)
end end
if dugnodes > most_digs then if dugnodes > most_digs then
if most_digs_user ~= user:get_player_name() then -- Avoid spam. if most_digs_user ~= pname then -- Avoid spam.
minetest.chat_send_all(S( minetest.chat_send_all(S(
"Most used tool is now a @1@2@3 owned by @4 with @5 uses.", "Most used tool is now a @1@2@3 owned by @4 with @5 uses.",
toolranks.colors.green, toolranks.colors.green,
itemdesc, itemdesc,
toolranks.colors.white, toolranks.colors.white,
user:get_player_name(), pname,
dugnodes dugnodes
)) ))
end end
mod_storage:set_int("most_digs", dugnodes) mod_storage:set_int("most_digs", dugnodes)
mod_storage:set_string("most_digs_user", user:get_player_name()) mod_storage:set_string("most_digs_user", pname)
end end
if itemstack:get_wear() > 60135 then if itemstack:get_wear() > 60135 then
minetest.chat_send_player(user:get_player_name(), S("Your tool is about to break!")) minetest.chat_send_player(user:get_player_name(), S("Your tool is about to break!"))
minetest.sound_play("default_tool_breaks", { minetest.sound_play("default_tool_breaks", {
to_player = user:get_player_name(), to_player = pname,
gain = 2.0, gain = 2.0,
}) })
end end
@ -107,7 +109,7 @@ function toolranks.new_afteruse(itemstack, user, node, digparams)
) )
minetest.chat_send_player(user:get_player_name(), levelup_text) minetest.chat_send_player(user:get_player_name(), levelup_text)
minetest.sound_play("toolranks_levelup", { minetest.sound_play("toolranks_levelup", {
to_player = user:get_player_name(), to_player = pname,
gain = 2.0, gain = 2.0,
}) })
-- Make tool better by modifying tool_capabilities (if defined) -- Make tool better by modifying tool_capabilities (if defined)