From e3c2616328b70f7a54ae9aa1a5256a2e79afbe9e Mon Sep 17 00:00:00 2001 From: stujones11 Date: Sun, 4 Jan 2015 21:37:12 +0000 Subject: [PATCH] Improve error logging a little --- 3d_armor/armor.lua | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/3d_armor/armor.lua b/3d_armor/armor.lua index 2189739..7a2bf4f 100644 --- a/3d_armor/armor.lua +++ b/3d_armor/armor.lua @@ -94,10 +94,10 @@ armor.set_player_armor = function(self, player) local name = player:get_player_name() local player_inv = player:get_inventory() if not name then - minetest.log("error", "Failed to read player name") + minetest.log("error", "set_player_armor: Failed to read player name") return elseif not player_inv then - minetest.log("error", "Failed to read player inventory") + minetest.log("error", "set_player_armor: Failed to read player inventory") return end local armor_texture = "3d_armor_trans.png" @@ -182,9 +182,14 @@ end armor.update_armor = function(self, player) if not player then + minetest.log("error", "update_armor: Invalid player object reference") return end local name = player:get_player_name() + if not name then + minetest.log("error", "update_armor: Invalid player name") + return + end local hp = player:get_hp() or 0 if hp == 0 or hp == self.player_hp[name] then return @@ -193,10 +198,10 @@ armor.update_armor = function(self, player) local player_inv = player:get_inventory() local armor_inv = minetest.get_inventory({type="detached", name=name.."_armor"}) if not player_inv then - minetest.log("error", "Failed to read player inventory") + minetest.log("error", "update_armor: Failed to read player inventory") return elseif not armor_inv then - minetest.log("error", "Failed to read detached inventory") + minetest.log("error", "update_armor: Failed to read detached inventory") return end local heal_max = 0 @@ -253,7 +258,15 @@ armor.get_armor_formspec = function(self, name) end armor.update_inventory = function(self, player) + if not player then + minetest.log("error", "update_inventory: Invalid player object reference") + return + end local name = player:get_player_name() + if not name then + minetest.log("error", "update_inventory: Invalid player name") + return + end if inv_mod == "unified_inventory" then if unified_inventory.current_page[name] == "armor" then unified_inventory.set_inventory_formspec(player, "armor")