From 890028864ebe1754b3afbe7299e3fde3510450a6 Mon Sep 17 00:00:00 2001
From: Thomas--S <Thomas--S@users.noreply.github.com>
Date: Sat, 26 Sep 2020 18:29:54 +0200
Subject: [PATCH] Signs: Fix potential after_dig error

Fixes a coroutine error with TA quarry.
See https://github.com/minetest/minetest/blob/34e3ede8eeb05e193e64ba3d055fc67959d87d86/doc/lua_api.txt#L6222

A better long-term solution might be to use preserve_metadata.
---
 lib.lua | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/lib.lua b/lib.lua
index 459db8f..c311fb7 100644
--- a/lib.lua
+++ b/lib.lua
@@ -229,10 +229,16 @@ function signs_bot.lib.after_dig_sign_node(pos, oldnode, oldmetadata, digger)
 		smeta:set_int("err_code", tonumber(oldmetadata.fields.err_code))
 		smeta:set_string("err_msg", oldmetadata.fields.err_msg or "")
 	end
-	local inv = minetest.get_inventory({type="player", name=digger:get_player_name()})
-	local left_over = inv:add_item("main", sign)
-	if left_over:get_count() > 0 then
+	local player_name = digger:get_player_name()
+	-- See https://github.com/minetest/minetest/blob/34e3ede8eeb05e193e64ba3d055fc67959d87d86/doc/lua_api.txt#L6222
+	if player_name == "" then
 		minetest.add_item(pos, sign)
+	else
+		local inv = minetest.get_inventory({type="player", name=digger:get_player_name()})
+		local left_over = inv:add_item("main", sign)
+		if left_over:get_count() > 0 then
+			minetest.add_item(pos, sign)
+		end
 	end
 end