From: PilzAdam <PilzAdam@gmx.de>
Date: Wed, 13 Feb 2013 17:06:25 +0000 (+0100)
Subject: Fix minetest.item_place_node() and minetest.item_drop() to always return an ItemStack
X-Git-Url: http://81.2.79.47:8989/gitweb/?a=commitdiff_plain;h=86b906d015ccde4e8f576a4e61d7c4ae8842b45d;p=zefram%2Fminetest%2Fminetest_engine.git

Fix minetest.item_place_node() and minetest.item_drop() to always return an ItemStack
---

diff --git a/builtin/item.lua b/builtin/item.lua
index acbdaa53..2233b6c5 100644
--- a/builtin/item.lua
+++ b/builtin/item.lua
@@ -141,7 +141,7 @@ function minetest.item_place_node(itemstack, placer, pointed_thing)
 		minetest.log("info", placer:get_player_name() .. " tried to place"
 			.. " node in invalid position " .. minetest.pos_to_string(above)
 			.. ", replacing " .. oldnode_above.name)
-		return
+		return itemstack
 	end
 
 	-- Place above pointed node
@@ -186,7 +186,7 @@ function minetest.item_place_node(itemstack, placer, pointed_thing)
 		not check_attached_node(place_to, newnode) then
 		minetest.log("action", "attached node " .. def.name ..
 			" can not be placed at " .. minetest.pos_to_string(place_to))
-		return
+		return itemstack
 	end
 
 	-- Add node and update
@@ -262,7 +262,7 @@ function minetest.item_drop(itemstack, dropper, pos)
 	else
 		minetest.env:add_item(pos, itemstack)
 	end
-	return ""
+	return ItemStack("")
 end
 
 function minetest.item_eat(hp_change, replace_with_item)