Fix armor destroy bugs on death

mods/ITEMS/minetest-3d_armor/3d_armor/armor.conf

@@ -16,7 +16,8 @@ ARMOR_FIRE_NODES = {
 	{"mcl_fire:eternal_fire",   3, 4},
 }
 
-ARMOR_DROP = true
+-- Armor drop/destroy disabled. Armor dropping is handled in mcl_death
+ARMOR_DROP = false
 ARMOR_DESTROY = false
 ARMOR_HEAL_MULTIPLIER = 0
 ARMOR_RADIATION_MULTIPLIER = 0

mods/PLAYER/mcl_death/init.lua

@@ -1,22 +1,37 @@
 minetest.register_on_dieplayer(function(player)
 	local keep = minetest.setting_getbool("mcl_keepInventory") or false
 	if keep == false then
+		-- Drop inventory, crafting grid and armor
 		local inv = player:get_inventory()
 		local pos = player:getpos()
-		local lists = { "main", "craft", "armor" }
+		local name, player_armor_inv, armor_armor_inv, pos = armor:get_valid_player(player, "[on_dieplayer]")
+		local lists = {
+			{ inv = inv, listname = "main", drop = true },
+			{ inv = inv, listname = "craft", drop = true },
+			{ inv = player_armor_inv, listname = "armor", drop = true },
+			{ inv = armor_armor_inv, listname = "armor", drop = false },
+		}
 		for l=1,#lists do
-			for i,stack in ipairs(inv:get_list(lists[l])) do
+			local inv = lists[l].inv
-				local x = math.random(0, 9)/3
+			local listname = lists[l].listname
-				local z = math.random(0, 9)/3
+			local drop = lists[l].drop
-				pos.x = pos.x + x
+			if inv ~= nil then
-				pos.z = pos.z + z
+				for i, stack in ipairs(inv:get_list(listname)) do
-				minetest.add_item(pos, stack)
+					local x = math.random(0, 9)/3
-				stack:clear()
+					local z = math.random(0, 9)/3
-				inv:set_stack(lists[l], i, stack)
+					pos.x = pos.x + x
-				pos.x = pos.x - x
+					pos.z = pos.z + z
-				pos.z = pos.z - z
+					if drop then
+						minetest.add_item(pos, stack)
+					end
+					stack:clear()
+					inv:set_stack(listname, i, stack)
+					pos.x = pos.x - x
+					pos.z = pos.z - z
+				end
 			end
 		end
+		armor:set_player_armor(player)
 		armor:update_inventory(player)
 	end