Merge pull request #12 from ChimneySwift/ChimneySwift-patch-1

Possibility to lose items when assigning output from the vendor storage

init.lua

@@ -1339,7 +1339,7 @@ local vendor_template = {
         end,
     },
     allow_metadata_inventory_move = function(pos, from_list, from_index, to_list, to_index, count, player)
-        if not can_access_vendor_inv(player, pos) then
+        if (not can_access_vendor_inv(player, pos)) or to_list == "wanted_item" or to_list == "given_item" then
             return 0
         end
         return count
@@ -1516,6 +1516,10 @@ minetest.register_tool("fancy_vend:copy_tool",{
         local meta = itemstack:get_meta()
         local node_meta = minetest.get_meta(pos)
         local new_settings = minetest.deserialize(meta:get_string("settings"))
+        if not new_settings then
+        	minetest.chat_send_player(user:get_player_name(), "No settings to set with. Right-click first on the vendor you want to copy settings from.")
+        	return
+        end
 
         if can_modify_vendor(pos, user) then
             -- Admin vendor priv check