Merge branch 'bed-compat-ng' of github.com:mt-mods/jumpdrive into bed-compat-ng

Author: SwissalpS

.github/workflows/integration-test.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v1
+    - uses: actions/checkout@master
     - name: apt
       run: sudo apt-get install -y luarocks
     - name: luacheck install

.github/workflows/luacheck.yml

@@ -0,0 +1,15 @@
+name: test
+
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: buckaroobanzay/mtt@main
+      with:
+        modname: jumpdrive
+        git_dependencies: |
+          https://github.com/minetest-mods/areas.git

.github/workflows/test.yml

@@ -1,7 +1,3 @@
-unused_args = false
-allow_defined_top = true
-
-ignore = {"512"}
 
 globals = {
 	"jumpdrive",
@@ -35,5 +31,9 @@ read_globals = {
 	"ropes",
 	"sethome",
 	"drawers",
-	"player_monoids"
+	"player_monoids",
+	"vizlib",
+	"mcl_sounds",
+	"mcl_formspec",
+	"mtt"
 }

.luacheckrc

@@ -4,7 +4,10 @@ minetest.register_node("jumpdrive:backbone", {
 	description = "Jumpdrive Backbone",
 
 	tiles = {"jumpdrive_backbone.png"},
-	groups = {cracky=3,oddly_breakable_by_hand=3},
-	sounds = default.node_sound_glass_defaults(),
+	groups = {cracky=3,oddly_breakable_by_hand=3,handy=1,pickaxey=1},
+	_mcl_blast_resistance = 2,
+	_mcl_hardness = 0.9,
+	sounds = jumpdrive.sounds.node_sound_glass_defaults(),
+	is_ground_content = false,
 	light_source = 13
 })

backbone.lua

@@ -1,13 +1,24 @@
 
+local book_item, book_written = ""
+
+if minetest.get_modpath("default") then
+   book_item = "default:book"
+   book_written = "default:book_written"
+end
+
+if minetest.get_modpath("mcl_books") then
+   book_item = "mcl_books:book"
+   book_written = "mcl_books:written_book"
+end
 
 jumpdrive.write_to_book = function(pos, sender)
 	local meta = minetest.get_meta(pos)
 	local inv = meta:get_inventory()
 
-	if inv:contains_item("main", {name="default:book", count=1}) then
-		local stack = inv:remove_item("main", {name="default:book", count=1})
+	if inv:contains_item("main", {name=book_item, count=1}) then
+		local stack = inv:remove_item("main", {name=book_item, count=1})
 
-		local new_stack = ItemStack("default:book_written")
+		local new_stack = ItemStack(book_written)
 
 		local data = {}
 
@@ -63,7 +74,7 @@ jumpdrive.read_from_book = function(pos)
 	for i = inv_size, 1, -1 do
 		stack = inv:get_stack("main", i)
 		stack_name = stack:get_name()
-		if "default:book_written" == stack_name then
+		if book_written == stack_name then
 			-- remove item from inventory
 			inv:set_stack("main", i, ItemStack())
 			stack_meta = stack:get_meta()

bookmark.lua

@@ -1,40 +1,72 @@
 
-jumpdrive.sanitize_coord = function(coord)
+local c_air = minetest.get_content_id("air")
+
+function jumpdrive.clear_area(pos1, pos2)
+	local manip = minetest.get_voxel_manip()
+	local e1, e2 = manip:read_from_map(pos1, pos2)
+	local source_area = VoxelArea:new({MinEdge=e1, MaxEdge=e2})
+	local source_data = manip:get_data()
+
+
+	for z=pos1.z, pos2.z do
+	for y=pos1.y, pos2.y do
+	for x=pos1.x, pos2.x do
+
+		local source_index = source_area:index(x, y, z)
+		source_data[source_index] = c_air
+	end
+	end
+	end
+
+	manip:set_data(source_data)
+	manip:write_to_map()
+
+	-- remove metadata
+	local target_meta_pos_list = minetest.find_nodes_with_meta(pos1, pos2)
+	for _,target_pos in pairs(target_meta_pos_list) do
+		local target_meta = minetest.get_meta(target_pos)
+		target_meta:from_table(nil)
+	end
+end
+
+function jumpdrive.sanitize_coord(coord)
 	return math.max( math.min( coord, 31000 ), -31000 )
 end
 
 -- get pos object from pos
-jumpdrive.get_meta_pos = function(pos)
+function jumpdrive.get_meta_pos(pos)
 	local meta = minetest.get_meta(pos);
 	return {x=meta:get_int("x"), y=meta:get_int("y"), z=meta:get_int("z")}
 end
 
 -- set pos object from pos
-jumpdrive.set_meta_pos = function(pos, target)
+function jumpdrive.set_meta_pos(pos, target)
 	local meta = minetest.get_meta(pos);
 	meta:set_int("x", target.x)
 	meta:set_int("y", target.y)
 	meta:set_int("z", target.z)
 end
 
 -- get offset from meta
-jumpdrive.get_radius = function(pos)
+function jumpdrive.get_radius(pos)
 	local meta = minetest.get_meta(pos);
 	return math.max(math.min(meta:get_int("radius"), jumpdrive.config.max_radius), 1)
 end
 
 -- calculates the power requirements for a jump
-jumpdrive.calculate_power = function(radius, distance, sourcePos, targetPos)
+-- params: radius, distance, sourcePos, targetPos
+function jumpdrive.calculate_power(radius, distance)
 	return 10 * distance * radius
 end
 
 
 -- preflight check, for overriding
-jumpdrive.preflight_check = function(source, destination, radius, playername)
+-- params: source, destination, radius, playername
+function jumpdrive.preflight_check()
 	return { success=true }
 end
 
-jumpdrive.reset_coordinates = function(pos)
+function jumpdrive.reset_coordinates(pos)
 	local meta = minetest.get_meta(pos)
 
 	meta:set_int("x", pos.x)

common.lua

@@ -21,6 +21,7 @@ end
 
 if has_technic_mod then
 	dofile(MP.."/compat/anchor.lua")
+	dofile(MP.."/compat/technic_networks.lua")
 end
 
 if has_locator_mod then
@@ -75,7 +76,5 @@ jumpdrive.node_compat = function(name, source_pos, target_pos, source_pos1, sour
 end
 
 jumpdrive.commit_node_compat = function()
-	if has_pipeworks_mod then
-		jumpdrive.teleporttube_compat_commit()
-	end
+	-- Nothing to do here
 end

compat/compat.lua

@@ -2,7 +2,7 @@
 local nodedef = minetest.registered_nodes["elevator:motor"]
 
 minetest.override_item("elevator:motor", {
-	on_movenode = function(from_pos, to_pos)
+	on_movenode = function(_, to_pos)
 		minetest.log("action", "[jumpdrive] Restoring elevator @ " .. to_pos.x .. "/" .. to_pos.y .. "/" .. to_pos.z)
 		nodedef.after_place_node(to_pos, nil, nil)
 	end

compat/elevator.lua

@@ -0,0 +1,57 @@
+--
+-- Compatibility hacks for technic plus new network system
+--
+-- More information:
+-- https://github.com/mt-mods/technic/issues/100
+--
+-- See also proposal draft to actually move networks instead of rebuilding:
+-- https://github.com/mt-mods/jumpdrive/pull/79
+--
+
+-- Check for technic mod version compatibility
+if technic.remove_network and technic.pos2network and technic.machines then
+
+	local function on_movenode(from_pos, to_pos, info)
+		-- Destroy network caches at source location, inside jump area
+		local src_net_id = technic.pos2network(from_pos)
+		if src_net_id then
+			technic.remove_network(src_net_id)
+		end
+
+		-- Destroy network caches at target location, outside jump area
+		local edge = info.edge
+		for axis, value in pairs(edge) do
+			if value ~= 0 then
+				local axis_dir = {x=0,y=0,z=0}
+				axis_dir[axis] = value
+				local edge_pos = vector.add(to_pos, axis_dir)
+				local dst_net_id = technic.pos2network(edge_pos)
+				if dst_net_id then
+					technic.remove_network(dst_net_id)
+				end
+			end
+		end
+	end
+
+	-- Collect groups for registered technic cables
+	local cable_groups = {}
+	for tier,_ in pairs(technic.machines) do
+		cable_groups[("technic_%s_cable"):format(tier:lower())] = 1
+	end
+
+	local function is_network_node(_, def)
+		if not def.groups then return end
+		for group,_ in pairs(cable_groups) do
+			if def.groups[group] then return true end
+		end
+		return def.groups["technic_machine"]
+	end
+
+	-- Inject on_movenode functionality but only if node does not already implement it
+	for name, def in pairs(minetest.registered_nodes) do
+		if not def.on_movenode and is_network_node(name, def) then
+			minetest.override_item(name, { on_movenode = on_movenode })
+		end
+	end
+
+end