From 16146231b8f8c8562c1d1fd4a2c84bcf4134c3c5 Mon Sep 17 00:00:00 2001
From: DS-Minetest <vorunbekannt75@web.de>
Date: Mon, 03 Jul 2017 23:35:19 +0200
Subject: [PATCH] don't place eg. chests without data

---
 wrench/init.lua |   13 +++++++++----
 1 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/wrench/init.lua b/wrench/init.lua
index f99fd14..bae77aa 100644
--- a/wrench/init.lua
+++ b/wrench/init.lua
@@ -24,10 +24,7 @@
 
 local function get_meta_type(name, metaname)
 	local def = wrench.registered_nodes[name]
-	if not def or not def.metas or not def.metas[metaname] then
-		return nil
-	end
-	return def.metas[metaname]
+	return def and def.metas and def.metas[metaname] or nil
 end
 
 local function get_pickup_name(name)
@@ -42,6 +39,14 @@
 	local data = itemstack:get_meta():get_string("data")
 	data = (data ~= "" and data) or	itemstack:get_metadata()
 	data = minetest.deserialize(data)
+	if not data then
+		minetest.remove_node(pos)
+		minetest.log("error", placer:get_player_name().." wanted to place "..
+				name.." at "..minetest.pos_to_string(pos)..
+				", but it had no data.")
+		minetest.log("verbose", "itemstack: "..itemstack:to_string())
+		return true
+	end
 	minetest.set_node(pos, {name = data.name, param2 = node.param2})
 	for name, value in pairs(data.metas) do
 		local meta_type = get_meta_type(data.name, name)

--
Gitblit v1.8.0