From b1b5ffa68fb67f0c00e8cf21d059d89f23a68589 Mon Sep 17 00:00:00 2001
From: An0n3m0us <abdullaahiqbal@hotmail.com>
Date: Fri, 13 Sep 2019 21:32:11 +0100
Subject: [PATCH] Add missing infotext

---
 mods/default/furnace.lua |  1 +
 mods/default/nodes.lua   |  7 ++++++-
 mods/doors/init.lua      |  4 ++--
 mods/vessels/init.lua    | 31 +++++++++++++++++++++----------
 4 files changed, 30 insertions(+), 13 deletions(-)

diff --git a/mods/default/furnace.lua b/mods/default/furnace.lua
index d36a7f66..2335dba3 100644
--- a/mods/default/furnace.lua
+++ b/mods/default/furnace.lua
@@ -292,6 +292,7 @@ minetest.register_node("default:furnace", {
 	on_construct = function(pos)
 		local meta = minetest.get_meta(pos)
 		meta:set_string("formspec", default.get_furnace_inactive_formspec())
+		meta:set_string("infotext", 'Furnace inactive\n(Item: Empty; Fuel: Empty)')
 		local inv = meta:get_inventory()
 		inv:set_size('src', 1)
 		inv:set_size('fuel', 1)
diff --git a/mods/default/nodes.lua b/mods/default/nodes.lua
index a93f2799..f136a542 100644
--- a/mods/default/nodes.lua
+++ b/mods/default/nodes.lua
@@ -2600,7 +2600,12 @@ local function register_sign(material, desc, def)
 				text .. "\" to sign at " .. minetest.pos_to_string(pos))
 			local meta = minetest.get_meta(pos)
 			meta:set_string("text", text)
-			meta:set_string("infotext", '"' .. text .. '"')
+
+			if string.len(text) > 0 then
+				meta:set_string("infotext", '"' .. text .. '"')
+			else
+				meta:set_string("infotext", '')
+			end
 		end,
 	})
 end
diff --git a/mods/doors/init.lua b/mods/doors/init.lua
index eb5dd6ba..9816a7ce 100644
--- a/mods/doors/init.lua
+++ b/mods/doors/init.lua
@@ -333,7 +333,7 @@ function doors.register(name, def)
 
 			if def.protected then
 				meta:set_string("owner", pn)
-				meta:set_string("infotext", S("Owned by @1", pn))
+				meta:set_string("infotext", S("Steel Door\nOwned by @1", pn))
 			end
 
 			if not (creative and creative.is_enabled_for and creative.is_enabled_for(pn)) then
@@ -586,7 +586,7 @@ function doors.register_trapdoor(name, def)
 			local pn = placer:get_player_name()
 			local meta = minetest.get_meta(pos)
 			meta:set_string("owner", pn)
-			meta:set_string("infotext", S("Owned by @1", pn))
+			meta:set_string("infotext", S("Steel Trapdoor\nOwned by @1", pn))
 
 			return (creative and creative.is_enabled_for and creative.is_enabled_for(pn))
 		end
diff --git a/mods/vessels/init.lua b/mods/vessels/init.lua
index 6212ba8e..3f001c19 100644
--- a/mods/vessels/init.lua
+++ b/mods/vessels/init.lua
@@ -16,11 +16,15 @@ local vessels_shelf_formspec =
 	"listring[current_player;main]" ..
 	default.get_hotbar_bg(0, 2.85)
 
-local function get_vessels_shelf_formspec(inv)
+local function update_vessels_shelf(pos)
+	local meta = minetest.get_meta(pos)
+	local inv = meta:get_inventory()
+	local invlist = inv:get_list("vessels")
+
 	local formspec = vessels_shelf_formspec
-	local invlist = inv and inv:get_list("vessels")
 	-- Inventory slots overlay
 	local vx, vy = 0, 0.3
+	local n_items = 0
 	for i = 1, 16 do
 		if i == 9 then
 			vx = 0
@@ -29,10 +33,20 @@ local function get_vessels_shelf_formspec(inv)
 		if not invlist or invlist[i]:is_empty() then
 			formspec = formspec ..
 				"image[" .. vx .. "," .. vy .. ";1,1;vessels_shelf_slot.png]"
+		else
+			local stack = invlist[i]
+			if not stack:is_empty() then
+				n_items = n_items + stack:get_count()
+			end
 		end
 		vx = vx + 1
 	end
-	return formspec
+	meta:set_string("formspec", formspec)
+	if n_items == 0 then
+		meta:set_string("infotext", S("Empty Vessel Shelf"))
+	else
+		meta:set_string("infotext", S("Vessel Shelf (@1 items)", n_items))
+	end
 end
 
 minetest.register_node("vessels:shelf", {
@@ -46,7 +60,7 @@ minetest.register_node("vessels:shelf", {
 
 	on_construct = function(pos)
 		local meta = minetest.get_meta(pos)
-		meta:set_string("formspec", get_vessels_shelf_formspec(nil))
+		update_vessels_shelf(pos)
 		local inv = meta:get_inventory()
 		inv:set_size("vessels", 8 * 2)
 	end,
@@ -63,20 +77,17 @@ minetest.register_node("vessels:shelf", {
 	on_metadata_inventory_move = function(pos, from_list, from_index, to_list, to_index, count, player)
 		minetest.log("action", player:get_player_name() ..
 			   " moves stuff in vessels shelf at ".. minetest.pos_to_string(pos))
-		local meta = minetest.get_meta(pos)
-		meta:set_string("formspec", get_vessels_shelf_formspec(meta:get_inventory()))
+		update_vessels_shelf(pos)
 	end,
 	on_metadata_inventory_put = function(pos, listname, index, stack, player)
 		minetest.log("action", player:get_player_name() ..
 			   " moves stuff to vessels shelf at ".. minetest.pos_to_string(pos))
-		local meta = minetest.get_meta(pos)
-		meta:set_string("formspec", get_vessels_shelf_formspec(meta:get_inventory()))
+		update_vessels_shelf(pos)
 	end,
 	on_metadata_inventory_take = function(pos, listname, index, stack, player)
 		minetest.log("action", player:get_player_name() ..
 			   " takes stuff from vessels shelf at ".. minetest.pos_to_string(pos))
-		local meta = minetest.get_meta(pos)
-		meta:set_string("formspec", get_vessels_shelf_formspec(meta:get_inventory()))
+		update_vessels_shelf(pos)
 	end,
 	on_blast = function(pos)
 		local drops = {}