From e9d5bd8fce3bbd6034e8457e880689bff1eb039d Mon Sep 17 00:00:00 2001
From: OgelGames <olliverdc28@gmail.com>
Date: Fri, 20 Jun 2025 14:48:44 +1000
Subject: [PATCH] check destination node before injecting item

---
 item_transport.lua | 25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/item_transport.lua b/item_transport.lua
index 2b3025b..74dfd73 100644
--- a/item_transport.lua
+++ b/item_transport.lua
@@ -35,14 +35,25 @@ end
 function pipeworks.tube_inject_item(pos, start_pos, velocity, item, owner, tags)
 	-- Take item in any format
 	local stack = ItemStack(item)
+	local to_pos = vector.add(pos, velocity)
+	local def = minetest.registered_nodes[minetest.get_node(to_pos).name]
+	if not def or not def.groups or not (def.groups.tube
+			or def.groups.tubedevice or def.groups.tubedevice_receiver) then
+		local dropped_item = minetest.add_item(pos, stack)
+		if dropped_item then
+			dropped_item:set_velocity(vector.multiply(velocity, 5))
+		end
+		return
+	end
 	local obj = luaentity.add_entity(pos, "pipeworks:tubed_item")
-	obj:set_item(stack:to_string())
-	obj.start_pos = vector.new(start_pos)
-	obj:set_velocity(velocity)
-	obj.owner = owner
-	obj.tags = tags
-	--obj:set_color("red") -- todo: this is test-only code
-	return obj
+	if obj then
+		obj:set_item(stack:to_string())
+		obj.start_pos = vector.new(start_pos)
+		obj:set_velocity(velocity)
+		obj.owner = owner
+		obj.tags = tags
+		return obj
+	end
 end
 
 -- adding two tube functions