mesechest: colorize textures

.luacheckrc

@@ -1,22 +0,0 @@
-read_globals = {
-	-- Defined by Minetest
-	"vector", "PseudoRandom", "VoxelArea", "table",
-
-	-- Mods
-	"digiline", "default", "creative",
-
-	-- Required for the mesechest registration
-	minetest = {
-		fields = {
-			register_lbm = {read_only = false},
-			register_node = {read_only = false},
-			registered_on_player_receive_fields = {
-				read_only = false,
-				other_fields = true,
-			},
-		},
-		other_fields = true
-	}
-}
-globals = {"moremesecons", "mesecon"}
-ignore = {"212", "631", "422", "432"}

README.md

@@ -1,11 +1,11 @@
 # MoreMesecons
 
-Based on Mesecons by Jeija <br/>
-By @paly2 and @HybridDog <br/>
-With the participation of @LeMagnesium (bugfix), @Ataron (textures), @JAPP (texture).
+Based on Mesecons by Jeija  
+By @paly2 and @HybridDog  
+With the participation of @LeMagnesium (bugfix), @Ataron (textures), @JAPP (texture).  
 
-Dependencies: [Mesecons](https://github.com/Jeija/minetest-mod-mesecons/) <br/>
-Optional dependencies: [digilines](https://github.com/minetest-mods/digilines)
+Dependencies: [Mesecons](https://github.com/Jeija/minetest-mod-mesecons/)  
+Optional dependencies: [vector_extras](https://github.com/HybridDog/vector_extras/) [digilines](https://github.com/minetest-mods/digilines)
 
 MoreMesecons is a mod for minetest which adds some mesecons items.
 

moremesecons_adjustable_blinkyplant/depends.txt

@@ -0,0 +1,3 @@
+mesecons
+moremesecons_utils
+craft_guide?

moremesecons_adjustable_blinkyplant/init.lua

@@ -64,5 +64,3 @@ minetest.register_craft({
 	recipe = {	{"mesecons_blinkyplant:blinky_plant_off"},
 			{"default:mese_crystal_fragment"},}
 })
-
-minetest.log("action", "[moremesecons_adjustable_blinky_plant] loaded.")

moremesecons_adjustable_blinkyplant/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_adjustable_blinkyplant
-depends = mesecons,moremesecons_utils
-optional_depends = craft_guide

moremesecons_adjustable_player_detector/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_adjustable_player_detector/init.lua

@@ -37,18 +37,18 @@ local object_detector_scan = function (pos)
 	local scan_all = scanname == ""
 	local scan_names = scanname:split(',')
 	local radius = meta:get_int("radius")
-	if radius <= 0 then
+	if radius == 0 then
 		radius = 6
 	end
 	for _,obj in pairs(minetest.get_objects_inside_radius(pos, radius)) do
 		local isname = obj:get_player_name() -- "" is returned if it is not a player; "" ~= nil!
 		if isname ~= "" then
 			if scan_all then
-				return true, isname
+				return true
 			end
 			for _, name in ipairs(scan_names) do
 				if isname == name then
-					return true, isname
+					return true
 				end
 			end
 		end
@@ -65,32 +65,12 @@ local object_detector_digiline = {
 			if channel ~= active_channel then
 				return
 			end
-			if type(msg) == "string" then
-				meta:set_string("scanname", msg)
+			meta:set_string("scanname", msg)
+			if meta:get_string("formspec") ~= "" then
 				make_formspec(meta)
-			elseif type(msg) == "table" then
-				if msg.radius then
-					local r = tonumber(msg.radius)
-					if r then
-						meta:set_int("radius", tonumber(msg.radius))
-						make_formspec(meta)
-					end
-				end
-				if msg.scanname then
-					meta:set_string("scanname", msg.scanname)
-					make_formspec(meta)
-				end
-				if msg.command and msg.command == "get" then
-					local found, name = object_detector_scan(pos)
-					if not found then
-						name = ""
-					end
-					digiline:receptor_send(pos, digiline.rules.default, channel, name)
-				end
 			end
 		end,
-	},
-	receptor = {}
+	}
 }
 
 minetest.register_node("moremesecons_adjustable_player_detector:player_detector_off", {
@@ -156,5 +136,3 @@ minetest.register_abm({
 		end
 	end,
 })
-
-minetest.log("action", "[moremesecons_adjustable_player_detector] loaded.")

moremesecons_adjustable_player_detector/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_adjustable_player_detector
-depends = mesecons
-optional_depends = craft_guide

moremesecons_commandblock/depends.txt

@@ -0,0 +1,3 @@
+mesecons
+moremesecons_utils
+craft_guide?

moremesecons_commandblock/init.lua

@@ -55,14 +55,14 @@ local function resolve_commands(commands, pos)
 	local nearest = nil
 	local min_distance = math.huge
 	local players = minetest.get_connected_players()
-	for _, player in pairs(players) do
+	for index, player in pairs(players) do
 		local distance = vector.distance(pos, player:getpos())
 		if distance < min_distance then
 			min_distance = distance
 			nearest = player:get_player_name()
 		end
 	end
-	local new_commands = commands:gsub("@nearest", nearest)
+	new_commands = commands:gsub("@nearest", nearest)
 	return new_commands, min_distance, new_commands ~= commands
 end
 
@@ -172,5 +172,3 @@ minetest.register_craft({
 		{"group:mesecon_conductor_craftable","default:mese_crystal","group:mesecon_conductor_craftable"}
 	}
 })
-
-minetest.log("action", "[moremesecons_commandblock] loaded.")

moremesecons_commandblock/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_commandblock
-depends = mesecons,moremesecons_utils
-optional_depends = craft_guide

moremesecons_conductor_signalchanger/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_conductor_signalchanger/init.lua

@@ -5,7 +5,7 @@ local nodebox = {
 
 local function signalchanger_get_output_rules(node)
 	local rules = {{x=-1, y=0, z=0}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -13,7 +13,7 @@ end
 
 local function signalchanger_get_input_rules(node)
 	local rules = {{x=0, y=0, z=-1, name="input_on"}, {x=0, y=0, z=1, name="input_off"}, {x=1, y=0, z=0, name="input_signal"}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -79,5 +79,3 @@ minetest.register_craft({
 	output = "moremesecons_conductor_signalchanger:conductor_signalchanger_off",
 	recipe = {{"group:mesecon_conductor_craftable","moremesecons_signalchanger:signalchanger_off"}}
 })
-
-minetest.log("action", "[moremesecons_conductor_signalchanger] loaded.")

moremesecons_conductor_signalchanger/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_conductor_signalchanger
-depends = mesecons
-optional_depends = craft_guide

moremesecons_dual_delayer/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_dual_delayer/init.lua

@@ -1,6 +1,6 @@
 local function dual_delayer_get_input_rules(node)
 	local rules = {{x=1, y=0, z=0}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -8,7 +8,7 @@ end
 
 local function dual_delayer_get_output_rules(node)
 	local rules = {{x=0, y=0, z=1}, {x=0, y=0, z=-1}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -17,19 +17,19 @@ end
 local dual_delayer_activate = function(pos, node)
 	mesecon.receptor_on(pos, {dual_delayer_get_output_rules(node)[1]}) -- Turn on the port 1
 	minetest.swap_node(pos, {name = "moremesecons_dual_delayer:dual_delayer_10", param2 = node.param2})
-	minetest.after(0.4, function()
+	minetest.after(0.4, function(pos, node)
 		mesecon.receptor_on(pos, {dual_delayer_get_output_rules(node)[2]}) -- Turn on the port 2
 		minetest.swap_node(pos, {name = "moremesecons_dual_delayer:dual_delayer_11", param2 = node.param2})
-	end)
+	end, pos, node)
 end
 
 local dual_delayer_deactivate = function(pos, node, link)
 	mesecon.receptor_off(pos, {dual_delayer_get_output_rules(node)[2]}) -- Turn off the port 2
 	minetest.swap_node(pos, {name = "moremesecons_dual_delayer:dual_delayer_10", param2 = node.param2})
-	minetest.after(0.4, function()
+	minetest.after(0.4, function(pos, node)
 		mesecon.receptor_off(pos, {dual_delayer_get_output_rules(node)[1]}) -- Turn off the port 1
 		minetest.swap_node(pos, {name = "moremesecons_dual_delayer:dual_delayer_00", param2 = node.param2})
-	end)
+	end, pos, node)
 end
 
 
@@ -58,10 +58,6 @@ for n,i in pairs({{0,0},{1,0},{1,1}}) do
 		top_texture = pre..top_texture
 	end
 
-	local use_texture_alpha
-	if minetest.features.use_texture_alpha_string_modes then
-		use_texture_alpha = "opaque"
-	end
 	minetest.register_node("moremesecons_dual_delayer:dual_delayer_"..i1 ..i2, {
 		description = "Dual Delayer",
 		drop = "moremesecons_dual_delayer:dual_delayer_00",
@@ -78,7 +74,6 @@ for n,i in pairs({{0,0},{1,0},{1,1}}) do
 		},
 		groups = groups,
 		tiles = {top_texture, "moremesecons_dual_delayer_bottom.png", "moremesecons_dual_delayer_side_left.png", "moremesecons_dual_delayer_side_right.png", "moremesecons_dual_delayer_ends.png", "moremesecons_dual_delayer_ends.png"},
-		use_texture_alpha = use_texture_alpha,
 		mesecons = {
 			receptor = {
 				state = mesecon.state.off,
@@ -98,5 +93,3 @@ minetest.register_craft({
 	output = "moremesecons_dual_delayer:dual_delayer_00 2",
 	recipe = {"mesecons_delayer:delayer_off_1", "mesecons_delayer:delayer_off_1"}
 })
-
-minetest.log("action", "[moremesecons_dual_delayer] loaded.")

moremesecons_dual_delayer/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_dual_delayer
-depends = mesecons
-optional_depends = craft_guide

moremesecons_entity_detector/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_entity_detector/init.lua

@@ -139,5 +139,3 @@ minetest.register_abm({
 		end
 	end,
 })
-
-minetest.log("action", "[moremesecons_entity_detector] loaded.")

moremesecons_entity_detector/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_entity_detector
-depends = mesecons
-optional_depends = craft_guide

moremesecons_igniter/depends.txt

@@ -0,0 +1,3 @@
+mesecons
+fire
+craft_guide?

moremesecons_igniter/init.lua

@@ -46,5 +46,3 @@ minetest.register_craft({
 	recipe = {	{"default:torch"},
 			{"default:mese_crystal_fragment"},}
 })
-
-minetest.log("action", "[moremesecons_igniter] loaded.")

moremesecons_igniter/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_igniter
-depends = mesecons,fire
-optional_depends = craft_guide

moremesecons_induction_transmitter/init.lua

@@ -1,102 +0,0 @@
-local function induction_transmitter_get_input_rules(node)
-	-- All horizontal rules, except the output
-	local rules = {
-		{x=-1,y=0,z=0},
-		{x=1,y=0,z=0},
-		{x=0,y=0,z=-1},
-		{x=0,y=0,z=1}
-	}
-	for i, r in ipairs(rules) do
-		if vector.equals(r, minetest.facedir_to_dir(node.param2)) then
-			table.remove(rules, i)
-		end
-	end
-	return rules
-end
-
-local function induction_transmitter_get_output_rules(node)
-	return {vector.multiply(minetest.facedir_to_dir(node.param2), 2)}
-end
-
-local function act(pos, node, state)
-	minetest.swap_node(pos, {name = "moremesecons_induction_transmitter:induction_transmitter_"..state, param2 = node.param2})
-
-	local dir = minetest.facedir_to_dir(node.param2)
-	local target_pos = vector.add(pos, vector.multiply(dir, 2))
-	local target_node = minetest.get_node(target_pos)
-	if mesecon.is_effector(target_node.name) then
-		-- Switch on an aside node, so it sends a signal to the target node
-		local aside_rule = mesecon.effector_get_rules(target_node)[1]
-		if not aside_rule then
-			return
-		end
-		mesecon["receptor_"..state](vector.add(target_pos, aside_rule), {vector.multiply(aside_rule, -1)})
-	elseif mesecon.is_conductor(target_node.name) then
-		-- Switch on the conductor itself
-		mesecon["receptor_"..state](target_pos, mesecon.conductor_get_rules(target_node))
-	end
-end
-
-mesecon.register_node("moremesecons_induction_transmitter:induction_transmitter", {
-	description = "Induction Transmitter",
-	drawtype = "nodebox",
-	paramtype = "light",
-	paramtype2 = "facedir",
-	node_box = {
-		type = "fixed",
-		fixed = {
-			{-0.5, -0.5, 0.125, 0.5, 0.5, 0.5},
-			{-0.375, -0.375, -0.1875, 0.375, 0.375, 0.125},
-			{-0.25, -0.25, -0.5, 0.25, 0.25, -0.1875},
-		}
-	},
-	selection_box = {
-		type = "fixed",
-		fixed = {
-			{-0.5, -0.5, 0.125, 0.5, 0.5, 0.5},
-			{-0.375, -0.375, -0.1875, 0.375, 0.375, 0.125},
-			{-0.25, -0.25, -0.5, 0.25, 0.25, -0.1875},
-		},
-	},
-}, {
-	tiles = {"default_mese_block.png"},
-	groups = {cracky = 3},
-	mesecons = {
-		receptor = {
-			state = mesecon.state.off,
-			rules = induction_transmitter_get_output_rules
-		},
-		effector = {
-			rules = induction_transmitter_get_input_rules,
-			action_on = function(pos, node)
-				act(pos, node, "on")
-			end
-		}
-	}
-}, {
-	light_source = 5,
-	tiles = {"default_mese_block.png^[brighten"},
-	groups = {cracky = 3, not_in_creative_inventory = 1},
-	mesecons = {
-		receptor = {
-			state = mesecon.state.on,
-			rules = induction_transmitter_get_output_rules
-		},
-		effector = {
-			rules = induction_transmitter_get_input_rules,
-			action_off = function(pos, node)
-				act(pos, node, "off")
-			end
-		}
-	}
-})
-
-minetest.register_craft({
-	output = "moremesecons_induction_transmitter:induction_transmitter_off",
-	recipe = {
-		{"default:mese_crystal_fragment", "mesecons_torch:mesecon_torch_on", "default:mese_crystal_fragment"},
-		{"", "default:mese_crystal_fragment", ""}
-	}
-})
-
-minetest.log("action", "[moremesecons_induction_transmitter] loaded.")

moremesecons_induction_transmitter/mod.conf

@@ -1,2 +0,0 @@
-name = moremesecons_induction_transmitter
-depends = mesecons,mesecons_torch,default

moremesecons_injector_controller/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_injector_controller/init.lua

@@ -1,6 +1,6 @@
 local injector_controller_get_output_rules = function(node)
 	local rules = {{x = 0, y = 0, z = 1}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -10,7 +10,7 @@ local injector_controller_get_input_rules = function(node)
 	local rules = {{x = 0, y = 0, z = -1},
 			{x = 1, y = 0, z = 0},
 			{x = -1, y = 0, z = 0}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -82,5 +82,3 @@ minetest.register_craft({
 	output = "moremesecons_injector_controller:injector_controller_off",
 	recipe = {{"mesecons_blinkyplant:blinky_plant_off","mesecons_gates:and_off"}}
 })
-
-minetest.log("action", "[moremesecons_injector_controller] loaded.")

moremesecons_injector_controller/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_injector_controller
-depends = mesecons
-optional_depends = craft_guide

moremesecons_jammer/depends.txt

@@ -0,0 +1,3 @@
+mesecons
+moremesecons_utils
+craft_guide?

moremesecons_jammer/init.lua

@@ -1,35 +1,55 @@
 -- see wireless jammer
-local storage = minetest.get_mod_storage()
+local get = vector.get_data_from_pos
+local set = vector.set_data_to_pos
+local remove = vector.remove_data_from_pos
 
-local jammers = moremesecons.load_MapDataStorage_legacy(storage, "jammers_v2",
-	"jammers")
+local jammers
+local enable_lbm = moremesecons.setting("jammer", "enable_lbm", false)
+local storage
+if not minetest.get_mod_storage then
+	enable_lbm = true -- No mod storage (<= 0.4.15-stable): force registration of LBM
+	jammers = {}
+else
+	storage = minetest.get_mod_storage()
+	jammers = minetest.deserialize(storage:get_string("jammers")) or {}
+end
 
 local function update_mod_storage()
-	storage:set_string("jammers_v2", jammers:serialize())
+	if not storage then
+		return
+	end
+	storage:set_string("jammers", minetest.serialize(jammers))
 end
 
 local function add_jammer(pos)
-	if jammers:getAt(pos) then
+	if get(jammers, pos.z,pos.y,pos.x) then
 		return
 	end
-	jammers:setAt(pos, true)
+	set(jammers, pos.z,pos.y,pos.x, true)
 	update_mod_storage()
 end
 
 local function remove_jammer(pos)
-	jammers:removeAt(pos)
+	remove(jammers, pos.z,pos.y,pos.x)
 	update_mod_storage()
 end
 
 local function is_jammed(pos)
 	local JAMMER_MAX_DISTANCE = moremesecons.setting("jammer", "max_distance", 10, 1)
 
-	local minp = vector.subtract(pos, JAMMER_MAX_DISTANCE)
-	local maxp = vector.add(pos, JAMMER_MAX_DISTANCE)
-	for p in jammers:iter(minp, maxp) do
-		local d = vector.subtract(pos, p)
-		if d.x ^ 2 + d.y ^ 2 + d.z ^ 2 <= JAMMER_MAX_DISTANCE^2 then
-			return true
+	local pz,py,px = vector.unpack(pos)
+	for z,yxs in pairs(jammers) do
+		if math.abs(pz-z) <= JAMMER_MAX_DISTANCE then
+			for y,xs in pairs(yxs) do
+				if math.abs(py-y) <= JAMMER_MAX_DISTANCE then
+					for x in pairs(xs) do
+						if math.abs(px-x) <= JAMMER_MAX_DISTANCE
+						and (px-x)^2+(py-y)^2+(pz-z)^2 <= JAMMER_MAX_DISTANCE^2 then
+							return true
+						end
+					end
+				end
+			end
 		end
 	end
 	return false
@@ -123,7 +143,7 @@ minetest.register_craft({
 		{"", "moremesecons_wireless:jammer_off", ""}}
 })
 
-if moremesecons.setting("jammer", "enable_lbm", false) then
+if enable_lbm then
 	minetest.register_lbm({
 		name = "moremesecons_jammer:add_jammer",
 		nodenames = {"moremesecons_jammer:jammer_on"},
@@ -131,5 +151,3 @@ if moremesecons.setting("jammer", "enable_lbm", false) then
 		action = add_jammer
 	})
 end
-
-minetest.log("action", "[moremesecons_jammer] loaded.")

moremesecons_jammer/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_jammer
-depends = mesecons,moremesecons_utils
-optional_depends = craft_guide

moremesecons_luablock/depends.txt

@@ -0,0 +1 @@
+mesecons

moremesecons_luablock/init.lua

@@ -1,33 +1,9 @@
-local storage = minetest.get_mod_storage()
-local pos_data = moremesecons.get_storage_data(storage, "pos_data")
-
-local function set_data(pos, code, owner)
-	local data = {
-		code = code,
-		owner = owner
-	}
-	moremesecons.set_data_to_pos(pos_data, pos, data)
-end
-
-local function check_data(pos, code, owner)
-	local stored_data = moremesecons.get_data_from_pos(pos_data, pos)
-	if not stored_data then
-		return false
-	end
-	if code ~= stored_data.code
-			or owner ~= stored_data.owner then
-		return false
-	end
-	return true
-end
-
-
 local function make_formspec(meta, pos)
-	local code = minetest.formspec_escape(meta:get_string("code"))
+	local code = meta:get_string("code")
 	local errmsg = minetest.formspec_escape(meta:get_string("errmsg"))
 	meta:set_string("formspec",
 		"size[10,8;]" ..
-		"textarea[0.5,0.5;9.5,7;code;Code;"..code.."]" ..
+		"textarea[0.5,0.5;10,7;code;Code;"..code.."]" ..
 		"label[0.1,7;"..errmsg.."]" ..
 		"button_exit[4,7.5;2,1;submit;Submit]")
 end
@@ -98,65 +74,44 @@ minetest.register_node("moremesecons_luablock:luablock", {
 		end
 
 		meta:set_string("code", fields.code)
-		set_data(pos, fields.code, name)
 		make_formspec(meta, pos)
 	end,
 	can_dig = function(pos, player)
 		local meta = minetest.get_meta(pos)
 		return meta:get_string("owner") == player:get_player_name()
 	end,
-	on_destruct = function(pos)
-		moremesecons.remove_data_from_pos(pos_data, pos)
-	end,
 	mesecons = {effector = {
 		action_on = function(npos, node)
 			local meta = minetest.get_meta(npos)
 			local code = meta:get_string("code")
-			local owner = meta:get_string("owner")
 			if code == "" then
 				return
 			end
-			if not check_data(npos, code, owner) then
-				minetest.log("warning", "[moremesecons_luablock] Metadata of LuaBlock at pos "..minetest.pos_to_string(npos).." does not match its mod storage data!")
-				return
-			end
-
-			local env = {}
-			for k, v in pairs(_G) do
-				env[k] = v
-			end
-			env.pos = table.copy(npos)
-			env.mem = minetest.deserialize(meta:get_string("mem")) or {}
-
-			local func, err_syntax
-			if _VERSION == "Lua 5.1" then
-				func, err_syntax = loadstring(code)
-				if func then
-					setfenv(func, env)
-				end
-			else
-				func, err_syntax = load(code, nil, "t", env)
-			end
+			-- We do absolutely no check there.
+			-- There is no limitation in the number of instruction the LuaBlock can execute
+			-- or the usage it can make of loops.
+			-- It is executed in the global namespace.
+			-- Remember: *The LuaBlock is highly dangerous and should be manipulated cautiously!*
+			local func, err = loadstring(code)
 			if not func then
-				meta:set_string("errmsg", err_syntax)
-				make_formspec(meta, npos)
+				meta:set_string("errmsg", err)
+				make_formspec(meta, pos)
 				return
 			end
+			-- Set the "pos" global
+			local old_pos = pos -- In case there's already an existing "pos" global
+			pos = table.copy(npos)
+			local good, err = pcall(func)
+			pos = old_pos
 
-			local good, err_runtime = pcall(func)
-
-			if not good then
-				meta:set_string("errmsg", err_runtime)
-				make_formspec(meta, npos)
+			if not good then -- Runtime error
+				meta:set_string("errmsg", err)
+				make_formspec(meta, pos)
 				return
 			end
 
-			meta:set_string("mem", minetest.serialize(env.mem))
-
 			meta:set_string("errmsg", "")
-			make_formspec(meta, npos)
+			make_formspec(meta, pos)
 		end
 	}}
 })
-
-minetest.log("action", "[moremesecons_luablock] loaded.")

moremesecons_luablock/mod.conf

@@ -1,2 +0,0 @@
-name = moremesecons_luablock
-depends = mesecons,moremesecons_utils

moremesecons_luacontroller_tool/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+moremesecons_utils

moremesecons_luacontroller_tool/init.lua

@@ -1,3 +1,7 @@
+--[[
+vector_extras there: https://github.com/HybridDog/vector_extras
+]]
+
 local templates = {MoreMesecons = {
 	logic = [[-- AND
 port.a = pin.b and pin.c
@@ -13,80 +17,25 @@ port.a = not (pin.b or pin.c)
 port.a = pin.b ~= pin.c
 -- XNOR / NXOR
 port.a = pin.b == pin.c]],
-
 	digilinesth = [[digiline_send(channel, msg)
 if event.type == "digiline" then
 	print(event.channel)
 	print(event.msg)
 end]],
-
-	clock = [[number_of_oscillations = 0 -- 0 for infinity
-interval = 1
-input_port = "A"
-output_port = "C"
-
-if event.type == "on" and event.pin.name == input_port and not mem.running then
-  if not mem.counter then
-    mem.counter = 0
-  end
-  mem.running = true
-  port[string.lower(output_port)] = true
-  interrupt(interval)
-  mem.counter = mem.counter + 1
-elseif event.type == "off" and event.pin.name == input_port and mem.running and number_of_oscillations == 0 then
-  mem.running = false
-  mem.counter = 0
-elseif event.type == "interrupt" then
-  if not port[string.lower(output_port)] and mem.running then
-    port[string.lower(output_port)] = true
-    interrupt(interval)
-    mem.counter = mem.counter + 1
-  else
-    port[string.lower(output_port)] = false
-    if mem.counter < number_of_oscillations or number_of_oscillations == 0 and mem.running then
-      interrupt(interval)
-    else
-      mem.running = false
-      mem.counter = 0
-    end
-  end
-end]],
-
-	counter = [[counter_limit = 5
-output_time = 0.5
-input_port = "A"
-output_port = "C"
-
-if event.type == "on" and event.pin.name == input_port then
-  if not mem.counter then
-    mem.counter = 0
-  end
-  mem.counter = mem.counter + 1
-  if mem.counter >= counter_limit then
-     port[string.lower(output_port)] = true
-     interrupt(output_time)
-     mem.counter = 0
-  end
-elseif event.type == "interrupt" then
-  port[string.lower(output_port)] = false
-end]]
 }}
 
 
 local file_path = minetest.get_worldpath().."/MoreMesecons_lctt"
 
 -- load templates from a compressed file
-do
-	local templates_file = io.open(file_path, "rb")
-	if templates_file then
-		local templates_raw = templates_file:read("*all")
-		io.close(templates_file)
-		if templates_raw
-		and templates_raw ~= "" then
-			local data = minetest.deserialize(minetest.decompress(templates_raw))
-			for name,t in pairs(data) do
-				templates[name] = t
-			end
+local templates_file = io.open(file_path, "rb")
+if templates_file then
+	local templates_raw = templates_file:read("*all")
+	io.close(templates_file)
+	if templates_raw
+	and templates_raw ~= "" then
+		for name,t in pairs(minetest.deserialize(minetest.decompress(templates_raw))) do
+			templates[name] = t
 		end
 	end
 end
@@ -177,7 +126,7 @@ local function get_selection_formspec(pname, selected_template)
 		fill_formspec_dropdown_list(pl_templates, selected_template)..
 
 	-- show selected template
-		"textarea[0,1;10.5,8.5;template_code;template code:;"..minetest.formspec_escape(template_code).."]"..
+		"textarea[0,1;10.5,8.5;template_code;template code:;"..template_code.."]"..
 
 	-- save name
 		"field[5,9.5;5,0;save_name;savename;"..selected_template.."]"..
@@ -201,8 +150,8 @@ end
 
 -- do not localize the function directly here to support possible overwritten luacontrollers
 local luac_def = minetest.registered_nodes["mesecons_luacontroller:luacontroller0000"]
-local function set_luacontroller_code(pos, code, sender)
-	luac_def.on_receive_fields(pos, nil, {code=code, program=""}, sender)
+local function set_luacontroller_code(pos, code)
+	luac_def.on_receive_fields(pos, nil, {code=code, program=""})
 end
 
 minetest.register_tool("moremesecons_luacontroller_tool:lctt", {
@@ -222,10 +171,10 @@ minetest.register_tool("moremesecons_luacontroller_tool:lctt", {
 			return
 		end
 
-		pdata[pname] = {
+		pdata[pname] = pdata[pname] or {
 			pos = pos,
 			player_name = pname,
-			template_name = pdata[pname] and pdata[pname].template_name or next(templates[pname] or templates[next(templates)]),
+			template_name = next(templates[pname] or templates[next(templates)]),
 		}
 		minetest.show_formspec(pname, "moremesecons:luacontroller_tool", get_selection_formspec(pdata[pname].player_name, pdata[pname].template_name))
 	end,
@@ -305,7 +254,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
 		-- replace the code of the luacontroller with the template
 		local code = get_code_or_nil(pname, fields.player_name, fields.template_name)
 		if code then
-			set_luacontroller_code(pos, code, player)
+			set_luacontroller_code(pos, code)
 			minetest.chat_send_player(pname, "code set to template at "..minetest.pos_to_string(pos))
 		end
 		return
@@ -315,7 +264,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
 		-- add the template to the end of the code of the luacontroller
 		local code = get_code_or_nil(pname, fields.player_name, fields.template_name)
 		if code then
-			set_luacontroller_code(pos, meta:get_string("code").."\r"..code, player)
+			set_luacontroller_code(pos, meta:get_string("code").."\r"..code)
 			minetest.chat_send_player(pname, "code added to luacontroller at "..minetest.pos_to_string(pos))
 		end
 		return
@@ -355,5 +304,3 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
 		return
 	end
 end)
-
-minetest.log("action", "[moremesecons_luacontroller_tool] loaded.")

moremesecons_luacontroller_tool/mod.conf

@@ -1,2 +0,0 @@
-name = moremesecons_luacontroller_tool
-depends = mesecons,mesecons_luacontroller,moremesecons_utils

moremesecons_mesechest/depends.txt

@@ -0,0 +1 @@
+default

moremesecons_mesechest/init.lua

@@ -2,7 +2,7 @@ local function mesechest_get_output_rules(node)
 	local rules = {{x=-1, y=0, z=0},
 			{x=0, y=0, z=-1},
 			{x=0, y=0, z=1}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -61,7 +61,7 @@ for i, f in ipairs(minetest.registered_on_player_receive_fields) do
 	end
 end
 
-default.chest.register_chest("moremesecons_mesechest:mesechest", {
+default.register_chest("mesechest", {
 	description = "Mese Chest",
 	tiles = {
 		"default_chest_top.png^[colorize:#d8e002:70",
@@ -82,7 +82,7 @@ default.chest.register_chest("moremesecons_mesechest:mesechest", {
 	}
 })
 
-default.chest.register_chest("moremesecons_mesechest:mesechest_locked", {
+default.register_chest("mesechest_locked", {
 	description = "Locked Mese Chest",
 	tiles = {
 		"default_chest_top.png^[colorize:#d8e002:70",
@@ -106,21 +106,3 @@ default.chest.register_chest("moremesecons_mesechest:mesechest_locked", {
 
 minetest.register_node = old_minetest_register_node
 minetest.register_lbm = old_minetest_register_lbm
-
-minetest.register_craft({
-	output = "moremesecons_mesechest:mesechest",
-	recipe = {{"group:mesecon_conductor_craftable", "default:chest", "group:mesecon_conductor_craftable"}}
-})
-
-minetest.register_craft({
-	output = "moremesecons_mesechest:mesechest_locked",
-	recipe = {{"group:mesecon_conductor_craftable", "default:chest_locked", "group:mesecon_conductor_craftable"}}
-})
-
--- Legacy
-minetest.register_alias("default:mesechest", "moremesecons_mesechest:mesechest")
-minetest.register_alias("mesechest", "moremesecons_mesechest:mesechest")
-minetest.register_alias("default:mesechest_locked", "moremesecons_mesechest:mesechest")
-minetest.register_alias("mesechest_locked", "moremesecons_mesechest:mesechest_locked")
-
-minetest.log("action", "[moremesecons_mesechest] loaded.")

moremesecons_mesechest/mod.conf

@@ -1,2 +0,0 @@
-name = moremesecons_mesechest
-depends = default,mesecons

moremesecons_playerkiller/depends.txt

@@ -0,0 +1,4 @@
+mesecons
+mesecons_materials
+moremesecons_utils
+craft_guide?

moremesecons_playerkiller/init.lua

@@ -4,7 +4,7 @@ local kill_nearest_player = function(pos)
 	-- Search the nearest player
 	local nearest
 	local min_distance = MAX_DISTANCE
-	for _, player in pairs(minetest.get_connected_players()) do
+	for index, player in pairs(minetest.get_connected_players()) do
 		local distance = vector.distance(pos, player:getpos())
 		if distance < min_distance then
 			min_distance = distance
@@ -30,7 +30,7 @@ local kill_nearest_player = function(pos)
 
 	-- And kill him
 	nearest:set_hp(0)
-	minetest.log("action", "Player "..owner.." kills player "..nearest:get_player_name().." using a MoreMesecons Player Killer.")
+	minetest.log("action", "Player "..owner.." kills player "..nearest.." using a MoreMesecons Player Killer.")
 end
 
 minetest.register_craft({
@@ -60,5 +60,3 @@ minetest.register_node("moremesecons_playerkiller:playerkiller", {
 	end,
 	sounds = default.node_sound_stone_defaults(),
 })
-
-minetest.log("action", "[moremesecons_playerkiller] loaded.")

moremesecons_playerkiller/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_playerkiller
-depends = mesecons,mesecons_materials,moremesecons_utils
-optional_depends = craft_guide

moremesecons_sayer/depends.txt

@@ -0,0 +1,5 @@
+mesecons
+mesecons_noteblock
+moremesecons_utils
+default
+craft_guide?

moremesecons_sayer/init.lua

@@ -35,7 +35,7 @@ if use_speech_dispatcher then
 		nil,
 		""
 	}
-	local language = minetest.settings:get("language") or "en"
+	local language = minetest.setting_get("language") or "en"
 	if language ~= "en" then
 		tab[3] = "-l "..language
 	end
@@ -121,5 +121,3 @@ minetest.register_craft({
 	recipe = {{"mesecons_luacontroller:luacontroller0000", "mesecons_noteblock:noteblock"},
 		{"group:wood", "group:wood"}}
 })
-
-minetest.log("action", "[moremesecons_sayer] loaded.")

moremesecons_sayer/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_sayer
-depends = mesecons,mesecons_noteblock,moremesecons_utils,default
-optional_depends = craft_guide

moremesecons_signalchanger/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_signalchanger/init.lua

@@ -6,7 +6,7 @@ local nodebox = {
 local function signalchanger_get_output_rules(node)
 	local rules = {{x=-1, y=0, z=0},
 			{x=1, y=0, z=0}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -14,7 +14,7 @@ end
 
 local function signalchanger_get_input_rules(node)
 	local rules = {{x=0, y=0, z=-1, name="input_on"}, {x=0, y=0, z=1, name="input_off"}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -79,5 +79,3 @@ minetest.register_craft({
 	output = "moremesecons_signalchanger:signalchanger_off",
 	recipe = {{"group:mesecon_conductor_craftable","moremesecons_switchtorch:switchtorch_off","group:mesecon_conductor_craftable"}}
 })
-
-minetest.log("action", "[moremesecons_signalchanger] loaded.")

moremesecons_signalchanger/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_signalchanger
-depends = mesecons
-optional_depends = craft_guide

moremesecons_switchtorch/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_switchtorch/init.lua

@@ -80,7 +80,7 @@ minetest.register_node("moremesecons_switchtorch:switchtorch_on", {
 	selection_box = torch_selectionbox,
 	groups = {dig_immediate=3, not_in_creative_inventory = 1},
 	drop = "moremesecons_switchtorch:switchtorch_off",
-	light_source = 9,
+	light_source = LIGHT_MAX-5,
 	mesecons = {receptor = {
 		state = mesecon.state.on,
 		rules = torch_get_output_rules
@@ -127,5 +127,3 @@ minetest.register_abm({
 -- 2 = x+1
 -- 0 = y+1
 -- 1 = y-1
-
-minetest.log("action", "[moremesecons_switchtorch] loaded.")

moremesecons_switchtorch/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_switchtorch
-depends = mesecons
-optional_depends = craft_guide

moremesecons_teleporter/depends.txt

@@ -0,0 +1,3 @@
+mesecons
+moremesecons_utils
+craft_guide?

moremesecons_teleporter/init.lua

@@ -1,19 +1,34 @@
-local storage = minetest.get_mod_storage()
+local teleporters
+local teleporters_rids
 
-local teleporters = minetest.deserialize(storage:get_string("teleporters")) or {}
-local teleporters_rids = moremesecons.load_MapDataStorage_legacy(storage,
-	"teleporters_rids_v2", "teleporters_rids")
+local enable_lbm = moremesecons.setting("teleporter", "enable_lbm", false)
+local storage
+if not minetest.get_mod_storage then
+	enable_lbm = true -- No mod storage (<= 0.4.15-stable): force registration of LBM
+	teleporters = {}
+	teleporters_rids = {}
+	jammers = {}
+else
+	storage = minetest.get_mod_storage()
+
+	teleporters = minetest.deserialize(storage:get_string("teleporters")) or {}
+	teleporters_rids = minetest.deserialize(storage:get_string("teleporters_rids")) or {}
+	jammers = minetest.deserialize(storage:get_string("jammers")) or {}
+end
 
 local function update_mod_storage()
+	if not storage then
+		return
+	end
 	storage:set_string("teleporters", minetest.serialize(teleporters))
-	storage:set_string("teleporters_rids_v2", teleporters_rids:serialize())
+	storage:set_string("teleporters_rids", minetest.serialize(teleporters_rids))
 end
 
 
 local function register(pos)
-	if not teleporters_rids:getAt(pos) then
+	if not vector.get_data_from_pos(teleporters_rids, pos.z,pos.y,pos.x) then
 		table.insert(teleporters, pos)
-		teleporters_rids:setAt(pos, #teleporters)
+		vector.set_data_to_pos(teleporters_rids, pos.z,pos.y,pos.x, #teleporters)
 		update_mod_storage()
 	end
 end
@@ -24,12 +39,12 @@ local function teleport_nearest(pos)
 
 	-- Search for the nearest player
 	local nearest = nil
-	local min_distance_player = MAX_PLAYER_DISTANCE
+	local min_distance = MAX_PLAYER_DISTANCE
 	local players = minetest.get_connected_players()
-	for _, player in pairs(players) do
+	for index, player in pairs(players) do
 		local distance = vector.distance(pos, player:getpos())
-		if distance <= min_distance_player then
-			min_distance_player = distance
+		if distance <= min_distance then
+			min_distance = distance
 			nearest = player
 		end
 	end
@@ -89,16 +104,16 @@ minetest.register_node("moremesecons_teleporter:teleporter", {
 	sounds = default.node_sound_stone_defaults(),
 	on_construct = register,
 	on_destruct = function(pos)
-		local RID = teleporters_rids:getAt(pos)
+		local RID = vector.get_data_from_pos(teleporters_rids, pos.z,pos.y,pos.x)
 		if RID then
 			table.remove(teleporters, RID)
-			teleporters_rids:removeAt(pos)
+			vector.remove_data_from_pos(teleporters_rids, pos.z,pos.y,pos.x)
 			update_mod_storage()
 		end
 	end,
 })
 
-if moremesecons.setting("teleporter", "enable_lbm", false) then
+if enable_lbm then
 	minetest.register_lbm({
 		name = "moremesecons_teleporter:add_teleporter",
 		nodenames = {"moremesecons_teleporter:teleporter"},
@@ -106,5 +121,3 @@ if moremesecons.setting("teleporter", "enable_lbm", false) then
 		action = register
 	})
 end
-
-minetest.log("action", "[moremesecons_teleporter] loaded.")

moremesecons_teleporter/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_teleporter
-depends = mesecons,moremesecons_utils
-optional_depends = craft_guide

moremesecons_timegate/depends.txt

@@ -0,0 +1,2 @@
+mesecons
+craft_guide?

moremesecons_timegate/init.lua

@@ -1,6 +1,6 @@
 local timegate_get_output_rules = function(node)
 	local rules = {{x = 0, y = 0, z = 1}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -8,7 +8,7 @@ end
 
 local timegate_get_input_rules = function(node)
 	local rules = {{x = 0, y = 0, z = -1}}
-	for _ = 0, node.param2 do
+	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
 	end
 	return rules
@@ -25,17 +25,14 @@ local function timegate_activate(pos, node)
 	node.name = "moremesecons_timegate:timegate_on"
 	minetest.swap_node(pos, node)
 	mesecon.receptor_on(pos)
-	minetest.after(time, function()
-		local node = minetest.get_node(pos)
-		if node.name == "moremesecons_timegate:timegate_on" then
-			mesecon.receptor_off(pos)
-			node.name = "moremesecons_timegate:timegate_off"
-			minetest.swap_node(pos, node)
-		end
-	end)
+	minetest.after(time, function(pos, node)
+		mesecon.receptor_off(pos)
+		node.name = "moremesecons_timegate:timegate_off"
+		minetest.swap_node(pos, node)
+	end, pos, node)
 end
 
-local boxes = {{ -6/16, -8/16, -6/16, 6/16, -7/16, 6/16 },		-- the main slab
+boxes = {{ -6/16, -8/16, -6/16, 6/16, -7/16, 6/16 },		-- the main slab
 
 	 { -2/16, -7/16, -4/16, 2/16, -26/64, -3/16 },		-- the jeweled "on" indicator
 	 { -3/16, -7/16, -3/16, 3/16, -26/64, -2/16 },
@@ -47,10 +44,6 @@ local boxes = {{ -6/16, -8/16, -6/16, 6/16, -7/16, 6/16 },		-- the main slab
 	 { -8/16, -8/16, -1/16, -6/16, -7/16, 1/16 },		-- the two wire stubs
 	 { 6/16, -8/16, -1/16, 8/16, -7/16, 1/16 }}
 
-local use_texture_alpha
-if minetest.features.use_texture_alpha_string_modes then
-	use_texture_alpha = "opaque"
-end
 mesecon.register_node("moremesecons_timegate:timegate", {
 	description = "Time Gate",
 	drawtype = "nodebox",
@@ -88,7 +81,6 @@ mesecon.register_node("moremesecons_timegate:timegate", {
 			"moremesecons_timegate_sides_off.png",
 			"moremesecons_timegate_sides_off.png"
 		},
-		use_texture_alpha = use_texture_alpha,
 		groups = {bendy=2,snappy=1,dig_immediate=2},
 		mesecons = {
 			receptor =
@@ -111,7 +103,6 @@ mesecon.register_node("moremesecons_timegate:timegate", {
 			"moremesecons_timegate_sides_on.png",
 			"moremesecons_timegate_sides_on.png"
 		},
-		use_texture_alpha = use_texture_alpha,
 		groups = {bendy=2,snappy=1,dig_immediate=2, not_in_creative_inventory=1},
 		mesecons = {
 			receptor = {
@@ -134,5 +125,3 @@ minetest.register_craft({
 
 minetest.register_alias("moremesecons_temporarygate:temporarygate_off", "moremesecons_timegate:timegate_off")
 minetest.register_alias("moremesecons_temporarygate:temporarygate_on", "moremesecons_timegate:timegate_on")
-
-minetest.log("action", "[moremesecons_timegate] loaded.")

moremesecons_timegate/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_timegate
-depends = mesecons
-optional_depends = craft_guide

moremesecons_utils/depends.txt

@@ -0,0 +1 @@
+vector_extras?

moremesecons_utils/init.lua

@@ -4,15 +4,15 @@ function moremesecons.setting(modname, settingname, default, min)
 	local setting = "moremesecons_" .. modname .. "." .. settingname
 
 	if type(default) == "boolean" then
-		local ret = minetest.settings:get_bool(setting)
+		local ret = minetest.setting_getbool(setting)
 		if ret == nil then
 			ret = default
 		end
 		return ret
 	elseif type(default) == "string" then
-		return minetest.settings:get(setting) or default
+		return minetest.setting_get(setting) or default
 	elseif type(default) == "number" then
-		local ret = tonumber(minetest.settings:get(setting)) or default
+		local ret = tonumber(minetest.setting_get(setting)) or default
 		if not ret then
 			minetest.log("warning", "[moremesecons_"..modname.."]: setting '"..setting.."' must be a number. Set to default value ("..tostring(default)..").")
 			ret = default
@@ -28,355 +28,53 @@ function moremesecons.setting(modname, settingname, default, min)
 	end
 end
 
--- Storage helpers
+-- Vector helpers
+-- All the following functions are from the vector_extras mod (https://github.com/HybridDog/vector_extras).
+-- If you enable that mod, its functions will be used instead of the ones defined below
 
-function moremesecons.get_storage_data(storage, name)
-	return {
-		tab = minetest.deserialize(storage:get_string(name)) or {},
-		name = name,
-		storage = storage
-	}
-end
-
-function moremesecons.set_data_to_pos(sto, pos, data)
-	sto.tab[minetest.hash_node_position(pos)] = data
-	sto.storage:set_string(sto.name, minetest.serialize(sto.tab))
-end
-
-function moremesecons.get_data_from_pos(sto, pos)
-	return sto.tab[minetest.hash_node_position(pos)]
-end
-
-function moremesecons.remove_data_from_pos(sto, pos)
-	sto.tab[minetest.hash_node_position(pos)] = nil
-	sto.storage:set_string(sto.name, minetest.serialize(sto.tab))
-end
-
--- Some additional vector helpers
-
--- The same as minetest.hash_node_position; I copied it to ensure backwards
--- compatibility and used hexadecimal number notation
-local function node_position_key(pos)
-	return (pos.z + 0x8000) * 0x10000 * 0x10000
-		 + (pos.y + 0x8000) * 0x10000
-		 +  pos.x + 0x8000
-end
-
-local MapDataStorage = {}
-setmetatable(MapDataStorage, {__call = function()
-	local obj = {}
-	setmetatable(obj, MapDataStorage)
-	return obj
-end})
-MapDataStorage.__index = {
-	getAt = function(self, pos)
-		return self[node_position_key(pos)]
-	end,
-	setAt = function(self, pos, data)
-		-- If x, y or z is omitted, the key corresponds to a position outside
-		-- of the map (hopefully), so it can be used to skip lines and planes
-		local vi_z = (pos.z + 0x8000) * 0x10000 * 0x10000
-		local vi_zy = vi_z + (pos.y + 0x8000) * 0x10000
-		local vi = vi_zy + pos.x + 0x8000
-		local is_new = self[vi] == nil
-		self[vi] = data
-		if is_new then
-			self[vi_z] = (self[vi_z] or 0) + 1
-			self[vi_zy] = (self[vi_zy] or 0) + 1
-		end
-	end,
-	setAtI = function(self, vi, data)
-		local vi_zy = vi - vi % 0x10000
-		local vi_z = vi - vi % (0x10000 * 0x10000)
-		local is_new = self[vi] == nil
-		self[vi] = data
-		if is_new then
-			self[vi_z] = (self[vi_z] or 0) + 1
-			self[vi_zy] = (self[vi_zy] or 0) + 1
-		end
-	end,
-	removeAt = function(self, pos)
-		local vi_z = (pos.z + 0x8000) * 0x10000 * 0x10000
-		local vi_zy = vi_z + (pos.y + 0x8000) * 0x10000
-		local vi = vi_zy + pos.x + 0x8000
-		if self[vi] == nil then
-			-- Nothing to remove
-			return
-		end
-		self[vi] = nil
-		-- Update existence information for the xy plane and x line
-		self[vi_z] = self[vi_z] - 1
-		if self[vi_z] == 0 then
-			self[vi_z] = nil
-			self[vi_zy] = nil
-			return
-		end
-		self[vi_zy] = self[vi_zy] - 1
-		if self[vi_zy] == 0 then
-			self[vi_zy] = nil
-		end
-	end,
-	iter = function(self, pos1, pos2)
-		local ystride = 0x10000
-		local zstride = 0x10000 * 0x10000
-
-		-- Skip z values where no data can be found
-		pos1 = vector.new(pos1)
-		local vi_z = (pos1.z + 0x8000) * 0x10000 * 0x10000
-		while not self[vi_z] do
-			pos1.z = pos1.z + 1
-			vi_z = vi_z + zstride
-			if pos1.z > pos2.z then
-				-- There are no values to iterate through
-				return function() return end
+if not vector.get_data_from_pos then
+	function vector.get_data_from_pos(tab, z,y,x)
+		local data = tab[z]
+		if data then
+			data = data[y]
+			if data then
+				return data[x]
 			end
 		end
-		-- Skipping y values is not yet implemented and may require much code
+	end
+end
 
-		local xrange = pos2.x - pos1.x + 1
-		local yrange = pos2.y - pos1.y + 1
-		local zrange = pos2.z - pos1.z + 1
-
-		-- x-only and y-only parts of the vector index of pos1
-		local vi_y = (pos1.y + 0x8000) * 0x10000
-		local vi_x =  pos1.x + 0x8000
-
-		local y = 0
-		local z = 0
-
-		local vi = node_position_key(pos1)
-		local pos = vector.new(pos1)
-		local nextaction = vi + xrange
-		pos.x = pos.x - 1
-		vi = vi - 1
-		local function iterfunc()
-			-- continue along x until it needs to jump
-			vi = vi + 1
-			pos.x = pos.x + 1
-			if vi ~= nextaction then
-				local v = self[vi]
-				if v == nil then
-					-- No data here
-					return iterfunc()
-				end
-				-- The returned position must not be changed
-				return pos, v
-			end
-
-			-- Reset x position
-			vi = vi - xrange
-			-- Go along y until pos2.y is exceeded
-			while true do
-				y = y + 1
-				pos.y = pos.y + 1
-				-- Set vi to index(pos1.x, pos1.y + y, pos1.z + z)
-				vi = vi + ystride
-				if y == yrange then
-					break
-				end
-				if self[vi - vi_x] then
-					nextaction = vi + xrange
-
-					vi = vi - 1
-					pos.x = pos1.x - 1
-					return iterfunc()
-				end
-				-- Nothing along this x line, so increase y again
-			end
-
-			-- Go back along y
-			vi = vi - yrange * ystride
-			y = 0
-			pos.y = pos1.y
-			-- Go along z until pos2.z is exceeded
-			while true do
-				z = z + 1
-				pos.z = pos.z + 1
-				vi = vi + zstride
-				if z == zrange then
-					-- Cuboid finished, return nil
-					return
-				end
-				if self[vi - vi_x - vi_y] then
-					y = 0
-					nextaction = vi + xrange
-
-					vi = vi - 1
-					pos.x = pos1.x - 1
-					return iterfunc()
-				end
-				-- Nothing in this xy plane, so increase z again
-			end
-		end
-		return iterfunc
-	end,
-	iterAll = function(self)
-		local previous_vi = nil
-		local function iterfunc()
-			local vi, v = next(self, previous_vi)
-			previous_vi = vi
-			if not vi then
+if not vector.set_data_to_pos then
+	function vector.set_data_to_pos(tab, z,y,x, data)
+		if tab[z] then
+			if tab[z][y] then
+				tab[z][y][x] = data
 				return
 			end
-			local z = math.floor(vi / (0x10000 * 0x10000))
-			vi = vi - z * 0x10000 * 0x10000
-			local y = math.floor(vi / 0x10000)
-			if y == 0 or z == 0 then
-				-- The index does not refer to a position inside the map
-				return iterfunc()
-			end
-			local x = vi - y * 0x10000 - 0x8000
-			y = y - 0x8000
-			z = z - 0x8000
-			return {x=x, y=y, z=z}, v
+			tab[z][y] = {[x] = data}
+			return
 		end
-		return iterfunc
-	end,
-	serialize = function(self)
-		local indices = {}
-		local values = {}
-		local i = 1
-		for pos, v in self:iterAll() do
-			local vi = node_position_key(pos)
-			-- Convert the double reversible to a string;
-			-- minetest.serialize does not (yet) do this
-			indices[i] = ("%.17g"):format(vi)
-			values[i] = v
-		end
-		return minetest.serialize({
-			version = "MapDataStorage_v1",
-			indices = "return {" .. table.concat(indices, ",") .. "}",
-			values = minetest.serialize(values),
-		})
-	end,
-}
-MapDataStorage.deserialize = function(txtdata)
-	local data = minetest.deserialize(txtdata)
-	if data.version ~= "MapDataStorage_v1" then
-		minetest.log("error", "Unknown MapDataStorage version: " ..
-			data.version)
+		tab[z] = {[y] = {[x] = data}}
 	end
-	-- I assume that minetest.deserialize correctly deserializes the indices,
-	-- which are in the %a format
-	local indices = minetest.deserialize(data.indices)
-	local values = minetest.deserialize(data.values)
-	if not indices or not values then
-		return MapDataStorage()
-	end
-	data = MapDataStorage()
-	for i = 1,#indices do
-		local vi = indices[i]
-		local v = values[i]
-		data:setAtI(vi, v)
-	end
-	return data
-end
-moremesecons.MapDataStorage = MapDataStorage
-
-
--- Legacy
-
--- vector_extras there: https://github.com/HybridDog/vector_extras
--- Creates a MapDataStorage object from old vector_extras generated table
-function moremesecons.load_old_data_from_pos(t)
-	local data = MapDataStorage()
-	for z, yxv in pairs(t) do
-		for y, xv in pairs(yxv) do
-			for x, v in pairs(xv) do
-				data:setAt({x=x, y=y, z=z}, v)
-			end
-		end
-	end
-	return data
 end
 
-function moremesecons.load_old_dfp_storage(modstorage, name)
-	local data = minetest.deserialize(modstorage:get_string(name))
-	if not data then
-		return
-	end
-	return moremesecons.load_old_data_from_pos(data)
-end
-
-function moremesecons.load_MapDataStorage_legacy(modstorage, name, oldname)
-	local t_old = moremesecons.load_old_dfp_storage(modstorage, oldname)
-	local t
-	if t_old and t_old ~= "" then
-		t = t_old
-		modstorage:set_string(name, t:serialize())
-		modstorage:set_string(oldname, nil)
-		return t
-	end
-	t = modstorage:get_string(name)
-	if t and t ~= "" then
-		return MapDataStorage.deserialize(t)
-	end
-	return MapDataStorage()
-end
-
-
-
---[[
--- This testing code shows an example usage of the MapDataStorage code
-local function do_test()
-	print("Test if iter returns correct positions when a lot is set")
-	local data = MapDataStorage()
-	local k = 0
-	for x = -5, 3 do
-		for y = -5, 3 do
-			for z = -5, 3 do
-				k = k + 1
-				data:setAt({x=x, y=y, z=z}, k)
-			end
+if not vector.remove_data_from_pos then
+	function vector.remove_data_from_pos(tab, z,y,x)
+		if vector.get_data_from_pos(tab, z,y,x) == nil then
+			return
+		end
+		tab[z][y][x] = nil
+		if not next(tab[z][y]) then
+			tab[z][y] = nil
+		end
+		if not next(tab[z]) then
+			tab[z] = nil
 		end
 	end
-	local expected_positions = {}
-	for z = -4, 2 do
-		for y = -4, 2 do
-			for x = -4, 2 do
-				expected_positions[#expected_positions+1] = {x=x, y=y, z=z}
-			end
-		end
-	end
-	local i = 0
-	for pos in data:iter({x=-4, y=-4, z=-4}, {x=2, y=2, z=2}) do
-		i = i + 1
-		assert(vector.equals(pos, expected_positions[i]))
-	end
-
-	print("Test if iter works correctly on a corner")
-	local found = false
-	for pos in data:iter({x=-8, y=-7, z=-80}, {x=-5, y=-5, z=-5}) do
-		assert(not found)
-		found = true
-		assert(vector.equals(pos, {x=-5, y=-5, z=-5}))
-	end
-	assert(found)
-
-	print("Test if iter finds all corners")
-	local expected_positions = {}
-	local k = 1
-	for _, z in ipairs({-9, -6}) do
-		for _, y in ipairs({-9, -6}) do
-			for _, x in ipairs({-8, -6}) do
-				local pos = {x=x, y=y, z=z}
-				expected_positions[#expected_positions+1] = pos
-				data:setAt(pos, k)
-				k = k + 1
-			end
-		end
-	end
-	local i = 1
-	for pos, v in data:iter({x=-8, y=-9, z=-9}, {x=-6, y=-6, z=-6}) do
-		assert(v == i)
-		assert(vector.equals(pos, expected_positions[i]))
-		i = i + 1
-		--~ print("found " .. minetest.pos_to_string(pos))
-	end
-	assert(i == 8 + 1, "Not enough or too many corners found")
-
-	--~ data:iterAll()
 end
-do_test()
---]]
-minetest.log("action", "[moremesecons_utils] loaded.")
+
+if not vector.unpack then
+	function vector.unpack(pos)
+		return pos.z, pos.y, pos.x
+	end
+end

moremesecons_utils/mod.conf

@@ -1,2 +0,0 @@
-name = moremesecons_utils
-description = Various helping functions for moremesecons

moremesecons_wireless/depends.txt

@@ -0,0 +1,4 @@
+mesecons
+moremesecons_utils
+digilines?
+craft_guide?

moremesecons_wireless/init.lua

@@ -1,145 +1,123 @@
-local storage = minetest.get_mod_storage()
+local wireless
+local wireless_meta -- This table contains wireless metadatas, it is a lot faster to access
+local jammers
 
--- Names wireless_meta, and jammers were used in old versions of this mod.
--- There is legacy code at the end of this file to migrate the mod storage.
-local wireless = minetest.deserialize(storage:get_string("networks")) or {}
-local wireless_meta = moremesecons.get_storage_data(storage, "wireless_meta_2")
-local jammers = moremesecons.get_storage_data(storage, "jammers_2")
+local enable_lbm = moremesecons.setting("wireless", "enable_lbm", false)
+local storage
+if not minetest.get_mod_storage then
+	enable_lbm = true -- No mod storage (<= 0.4.15-stable): force registration of LBM
+	wireless = {}
+	wireless_meta = {owners = {}, channels = {}, ids = {}}
+	jammers = {}
+else
+	storage = minetest.get_mod_storage()
+
+	wireless = minetest.deserialize(storage:get_string("wireless")) or {}
+	wireless_meta = minetest.deserialize(storage:get_string("wireless_meta")) or {owners = {}, channels = {}, ids = {}}
+	jammers = minetest.deserialize(storage:get_string("jammers")) or {}
+end
 
 local function update_mod_storage()
-	storage:set_string("networks", minetest.serialize(wireless))
+	if not storage then
+		return
+	end
+	storage:set_string("wireless", minetest.serialize(wireless))
+	storage:set_string("wireless_meta", minetest.serialize(wireless_meta))
+	storage:set_string("jammers", minetest.serialize(jammers))
 end
 
-local wireless_effector_off
+-- localize these functions with small names because they work fairly fast
+local get = vector.get_data_from_pos
+local set = vector.set_data_to_pos
+local remove = vector.remove_data_from_pos
+
 local function remove_wireless(pos)
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos)
-	if not wls then
+	local owner = get(wireless_meta.owners, pos.z,pos.y,pos.x)
+	if not owner or owner == "" then
+		return
+	end
+	remove(wireless_meta.owners, pos.z,pos.y,pos.x)
+	if not wireless[owner] or not next(wireless[owner]) then
+		wireless[owner] = nil
 		return
 	end
 
-	if not wls.owner or wls.owner == "" then
-		moremesecons.remove_data_from_pos(wireless_meta, pos)
+	local channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	if not channel or channel == "" then
 		return
 	end
 
-	if not wireless[wls.owner] or not next(wireless[wls.owner]) then
-		wireless[wls.owner] = nil
-		moremesecons.remove_data_from_pos(wireless_meta, pos)
-		return
-	end
-
-	if not wls.channel or wls.channel == "" then
-		moremesecons.remove_data_from_pos(wireless_meta, pos)
-		return
-	end
-
-	local network = wireless[wls.owner][wls.channel]
-
-	if network.sources[wls.id] then
-		wireless_effector_off(pos)
-	end
-
-	moremesecons.remove_data_from_pos(wireless_meta, pos)
-
-	network.members[wls.id] = nil
-	if not next(network.members) then
-		wireless[wls.owner][wls.channel] = nil
-		if not next(wireless[wls.owner]) then
-			wireless[wls.owner] = nil
+	table.remove(wireless[owner][channel], get(wireless_meta.ids, pos.z,pos.y,pos.x))
+	if #wireless[owner][channel] == 0 then
+		wireless[owner][channel] = nil
+		if not next(wireless[owner]) then
+			wireless[owner] = nil
 		end
 	end
-	update_mod_storage()
+
+	remove(wireless_meta.channels, pos.z,pos.y,pos.x)
+	remove(wireless_meta.ids, pos.z,pos.y,pos.x)
 end
 
+local set_channel
 local function set_owner(pos, owner)
 	if not owner or owner == "" then
 		return
 	end
-
-	remove_wireless(pos)
-
 	local meta = minetest.get_meta(pos)
-	if meta then
-		meta:set_string("owner", owner)
-	end
-
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos) or {}
-	wls.owner = owner
-	moremesecons.set_data_to_pos(wireless_meta, pos, wls)
-
+	meta:set_string("owner", owner)
+	set(wireless_meta.owners, pos.z,pos.y,pos.x, owner)
 	if not wireless[owner] then
 		wireless[owner] = {}
 	end
 
-	if meta then
-		meta:set_string("infotext", "Wireless owned by " .. owner .. " on " .. ((wls.channel and wls.channel ~= "") and "channel " .. wls.channel or "undefined channel"))
+	local channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	if channel and channel ~= "" then
+		if not wireless[owner][channel] then
+			wireless[owner][channel] = {}
+		end
+		set_channel(pos, channel)
 	end
+
+	meta:set_string("infotext", "Wireless owned by " .. owner .. " on " .. ((channel and channel ~= "") and "channel " .. channel or "undefined channel"))
 end
 
-local wireless_receptor_on
-local wireless_receptor_off
-local wireless_effector_on
-local function set_channel(pos, channel)
+function set_channel(pos, channel)
 	if not channel or channel == "" then
 		return
 	end
 
 	local meta = minetest.get_meta(pos)
-
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos)
-	if not wls or wls.owner == "" then
+	local owner = get(wireless_meta.owners, pos.z,pos.y,pos.x)
+	if not owner or owner == "" then
 		return
 	end
 
-	if wls.id then
+	local old_channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	if old_channel and old_channel ~= "" and old_channel ~= channel then
 		remove_wireless(pos)
+		set_owner(pos, owner)
 	end
 
-	if meta then
-		meta:set_string("channel", channel)
+	meta:set_string("channel", channel)
+	set(wireless_meta.channels, pos.z,pos.y,pos.x, channel)
+	if not wireless[owner] then
+		wireless[owner] = {}
 	end
-	wls.channel = channel
-	moremesecons.set_data_to_pos(wireless_meta, pos, wls)
-
-	if not wireless[wls.owner] then
-		wireless[wls.owner] = {}
-	end
-	if not wireless[wls.owner][channel] then
-		wireless[wls.owner][channel] = {
-			members = {},
-			sources = {}
-		}
+	if not wireless[owner][channel] then
+		wireless[owner][channel] = {}
 	end
 
-	-- Find the first free ID
-	local id = 1
-	while wireless[wls.owner][channel].members[id] do
-		id = id + 1
-	end
-	wls.id = id
-	moremesecons.set_data_to_pos(wireless_meta, pos, wls)
-
-	local network = wireless[wls.owner][channel]
-
-	network.members[id] = pos
-
-	if meta then
-		meta:set_int("id", id)
-	end
-
-	update_mod_storage()
-
-	if meta then
-		meta:set_string("infotext", "Wireless owned by " .. wls.owner .. " on channel " .. channel)
-	end
-
-	if wls.effector then
-		wireless_effector_on(pos)
-	elseif next(network.sources) then
-		wireless_receptor_on(pos, id, network, false)
+	local id = get(wireless_meta.ids, pos.z,pos.y,pos.x)
+	if id then
+		wireless[owner][channel][id] = pos
 	else
-		wireless_receptor_off(pos, id, network, false)
+		table.insert(wireless[owner][channel], pos)
+		meta:set_int("id", #wireless[owner][channel])
+		set(wireless_meta.ids, pos.z,pos.y,pos.x, #wireless[owner][channel])
 	end
+
+	meta:set_string("infotext", "Wireless owned by " .. owner .. " on channel " .. channel)
 end
 
 local function register_wireless(pos)
@@ -148,140 +126,81 @@ local function register_wireless(pos)
 	if owner == "" then
 		return
 	end
-	remove_wireless(pos)
-
 	set_owner(pos, owner)
 
 	local channel = meta:get_string("channel")
 	if channel ~= "" then
 		set_channel(pos, channel)
 	end
-end
 
-local function check_wireless_exists(pos)
-	local nn = minetest.get_node(pos).name
-	if nn:sub(1, 30) == "moremesecons_wireless:wireless" then
-		return true
-	elseif nn ~= "ignore" then
-		-- Defer the remove_wireless() call so it doesn't interfere
-		-- with pairs().
-		minetest.after(0, remove_wireless, pos)
-		return false
-	end
-end
-
-function wireless_receptor_on(pos, id, network, check)
-	if check == false or check_wireless_exists(pos) then
-		minetest.swap_node(pos, {name = "moremesecons_wireless:wireless_on"})
-		if not network.sources[id] then
-			mesecon.receptor_on(pos)
-		end
-	end
-end
-
-function wireless_receptor_off(pos, id, network, check)
-	if check == false or check_wireless_exists(pos) then
-		minetest.swap_node(pos, {name = "moremesecons_wireless:wireless_off"})
-		mesecon.receptor_off(pos)
-	end
-end
-
-local function activate_network(owner, channel)
-	local network = wireless[owner][channel]
-	for i, wl_pos in pairs(network.members) do
-		wireless_receptor_on(wl_pos, i, network)
-	end
-end
-
-local function deactivate_network(owner, channel)
-	local network = wireless[owner][channel]
-	for i, wl_pos in pairs(network.members) do
-		wireless_receptor_off(wl_pos, i, network)
-	end
+	update_mod_storage()
 end
 
 local is_jammed
-function wireless_effector_on(pos)
+local function wireless_activate(pos)
 	if is_jammed(pos) then
 		-- jamming doesn't disallow receiving signals, only sending them
 		return
 	end
 
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos)
-	if not wls then
+	local channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	local owner = get(wireless_meta.owners, pos.z,pos.y,pos.x)
+	local id = get(wireless_meta.ids, pos.z,pos.y,pos.x)
+
+	if owner == "" or not wireless[owner] or channel == "" or not wireless[owner][channel] then
 		return
 	end
 
-	wls.effector = true
-
-	moremesecons.set_data_to_pos(wireless_meta, pos, wls)
-
-	if wls.owner == "" or not wireless[wls.owner] or wls.channel == "" or not wireless[wls.owner][wls.channel] then
-		return
+	minetest.swap_node(pos, {name = "moremesecons_wireless:wireless_on"})
+	for i, wl_pos in ipairs(wireless[owner][channel]) do
+		if i ~= id then
+			minetest.swap_node(wl_pos, {name = "moremesecons_wireless:wireless_on"})
+			mesecon.receptor_on(wl_pos)
+		end
 	end
-
-	local network = wireless[wls.owner][wls.channel]
-	network.sources[wls.id] = true
-	activate_network(wls.owner, wls.channel)
-
-	update_mod_storage()
 end
 
-function wireless_effector_off(pos)
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos)
-	if not wls then
-		return
-	end
-
-	wls.effector = nil
-	moremesecons.set_data_to_pos(wireless_meta, pos, wls)
-
-	if wls.owner == "" or not wireless[wls.owner] or wls.channel == "" or not wireless[wls.owner][wls.channel] then
-		return
-	end
-
-	local network = wireless[wls.owner][wls.channel]
-	network.sources[wls.id] = nil
-	if not next(network.sources) then
-		deactivate_network(wls.owner, wls.channel)
-	else
-		-- There is another source in the network. Turn this wireless into
-		-- a receptor.
-		mesecon.receptor_on(pos)
-	end
-
-	update_mod_storage()
-end
-
--- This table is required to prevent a message from being sent in loop between wireless nodes
-local sending_digilines = {}
-
-local function on_digiline_receive(pos, node, channel, msg)
+local function wireless_deactivate(pos)
 	if is_jammed(pos) then
 		return
 	end
 
-	local wls = moremesecons.get_data_from_pos(wireless_meta, pos)
-	if not wls then
+	local channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	local owner = get(wireless_meta.owners, pos.z,pos.y,pos.x)
+	local id = get(wireless_meta.ids, pos.z,pos.y,pos.x)
+
+	if owner == "" or not wireless[owner] or channel == "" or not wireless[owner][channel] then
 		return
 	end
 
-	if wls.owner == "" or not wireless[wls.owner] or channel == "" or not wireless[wls.owner][wls.channel] then
+	minetest.swap_node(pos, {name = "moremesecons_wireless:wireless_off"})
+	for i, wl_pos in ipairs(wireless[owner][channel]) do
+		if i ~= id then
+			minetest.swap_node(wl_pos, {name = "moremesecons_wireless:wireless_off"})
+			mesecon.receptor_off(wl_pos)
+		end
+	end
+end
+
+local function on_digiline_receive(pos, node, channel, msg)
+	local setchan = minetest.get_meta(pos):get_string("channel") -- Note : the digiline channel is the same as the wireless channel. TODO: Making two different channels and a more complex formspec ?
+	if channel ~= setchan or is_jammed(pos) or setchan == "" then
 		return
 	end
 
-	local pos_hash = minetest.hash_node_position(pos)
-	if sending_digilines[pos_hash] then
+	local channel = get(wireless_meta.channels, pos.z,pos.y,pos.x)
+	local owner = get(wireless_meta.owners, pos.z,pos.y,pos.x)
+	local id = get(wireless_meta.ids, pos.z,pos.y,pos.x)
+
+	if owner == "" or not wireless[owner] or channel == "" or not wireless[owner][channel] then
 		return
 	end
 
-	sending_digilines[pos_hash] = true
-	for i, wl_pos in pairs(wireless[wls.owner][wls.channel].members) do
-		if i ~= wls.id then
+	for i, wl_pos in ipairs(wireless[owner][channel]) do
+		if i ~= id then
 			digiline:receptor_send(wl_pos, digiline.rules.default, channel, msg)
 		end
 	end
-	sending_digilines[pos_hash] = nil
 end
 
 mesecon.register_node("moremesecons_wireless:wireless", {
@@ -300,10 +219,13 @@ mesecon.register_node("moremesecons_wireless:wireless", {
 	end,
 	on_destruct = function(pos)
 		remove_wireless(pos)
+		update_mod_storage()
 		mesecon.receptor_off(pos)
 	end,
 	after_place_node = function(pos, placer)
+		local placername = placer:get_player_name()
 		set_owner(pos, placer:get_player_name())
+		update_mod_storage()
 	end,
 	on_receive_fields = function(pos, _, fields, player)
 		local meta = minetest.get_meta(pos)
@@ -314,6 +236,7 @@ mesecon.register_node("moremesecons_wireless:wireless", {
 			-- Old wireless
 			if not minetest.is_protected(pos, playername) then
 				set_owner(pos, playername)
+				update_mod_storage()
 			else
 				return
 			end
@@ -321,24 +244,130 @@ mesecon.register_node("moremesecons_wireless:wireless", {
 
 		if playername == owner then
 			set_channel(pos, fields.channel)
+			update_mod_storage()
 		end
 	end,
 }, {
 	tiles = {"moremesecons_wireless_off.png"},
 	groups = {cracky=3},
 	mesecons = {effector = {
-		action_on = wireless_effector_on
+		action_on = wireless_activate,
 	}},
 }, {
 	tiles = {"moremesecons_wireless_on.png"},
 	groups = {cracky=3, not_in_creative_inventory=1},
 	mesecons = {effector = {
-		action_off = wireless_effector_off
+		action_off = wireless_deactivate
 	}},
 })
 
 minetest.register_alias("moremesecons_wireless:wireless", "moremesecons_wireless:wireless_off")
 
+local jammers = {}
+local function add_jammer(pos)
+	if get(jammers, pos.z,pos.y,pos.x) then
+		return
+	end
+	set(jammers, pos.z,pos.y,pos.x, true)
+	update_mod_storage()
+end
+
+local function remove_jammer(pos)
+	remove(jammers, pos.z,pos.y,pos.x)
+	update_mod_storage()
+end
+
+-- looks big, but should work fast
+function is_jammed(pos)
+	local JAMMER_MAX_DISTANCE = moremesecons.setting("wireless", "jammer_max_distance", 15, 1)
+
+	local pz,py,px = vector.unpack(pos)
+	for z,yxs in pairs(jammers) do
+		if math.abs(pz-z) <= JAMMER_MAX_DISTANCE then
+			for y,xs in pairs(yxs) do
+				if math.abs(py-y) <= JAMMER_MAX_DISTANCE then
+					for x in pairs(xs) do
+						if math.abs(px-x) <= JAMMER_MAX_DISTANCE
+						and (px-x)^2+(py-y)^2+(pz-z)^2 <= JAMMER_MAX_DISTANCE^2 then
+							return true
+						end
+					end
+				end
+			end
+		end
+	end
+	return false
+end
+
+mesecon.register_node("moremesecons_wireless:jammer", {
+	description = "Wireless Jammer",
+	paramtype = "light",
+	drawtype = "nodebox",
+},{
+	tiles = {"mesecons_wire_off.png^moremesecons_jammer_top.png", "moremesecons_jammer_bottom.png", "mesecons_wire_off.png^moremesecons_jammer_side_off.png"},
+	node_box = {
+		type = "fixed",
+		fixed = {
+			-- connection
+			{-1/16, -0.5, -0.5, 1/16, -7/16, 0.5},
+			{-0.5, -0.5, -1/16, 0.5, -7/16, 1/16},
+
+			--stabilization
+			{-1/16, -7/16, -1/16, 1/16, -6/16, 1/16},
+
+			-- fields
+			{-7/16, -6/16, -7/16, 7/16, -4/16, 7/16},
+			{-5/16, -4/16, -5/16, 5/16, -3/16, 5/16},
+			{-3/16, -3/16, -3/16, 3/16, -2/16, 3/16},
+			{-1/16, -2/16, -1/16, 1/16, -1/16, 1/16},
+		},
+	},
+	groups = {dig_immediate=2},
+	mesecons = {effector = {
+		rules = mesecon.rules.flat,
+		action_on = function(pos)
+			add_jammer(pos)
+			minetest.swap_node(pos, {name="moremesecons_wireless:jammer_on"})
+		end
+	}}
+},{
+	tiles = {"mesecons_wire_on.png^moremesecons_jammer_top.png", "moremesecons_jammer_bottom.png", "mesecons_wire_on.png^moremesecons_jammer_side_on.png"},
+	node_box = {
+		type = "fixed",
+		fixed = {
+			-- connection
+			{-1/16, -0.5, -0.5, 1/16, -7/16, 0.5},
+			{-0.5, -0.5, -1/16, 0.5, -7/16, 1/16},
+
+			--stabilization
+			{-1/16, -7/16, -1/16, 1/16, 5/16, 1/16},
+
+			-- fields
+			{-7/16, -6/16, -7/16, 7/16, -4/16, 7/16},
+			{-5/16, -3/16, -5/16, 5/16, -1/16, 5/16},
+			{-3/16, 0, -3/16, 3/16, 2/16, 3/16},
+			{-1/16, 3/16, -1/16, 1/16, 5/16, 1/16},
+		},
+	},
+	groups = {dig_immediate=2, not_in_creative_inventory=1},
+	mesecons = {effector = {
+		rules = mesecon.rules.flat,
+		action_off = function(pos)
+			remove_jammer(pos)
+			minetest.swap_node(pos, {name="moremesecons_wireless:jammer_off"})
+		end
+	}},
+	on_destruct = remove_jammer,
+	on_construct = add_jammer,
+})
+
+minetest.register_craft({
+	output = "moremesecons_wireless:jammer_off",
+	recipe = {
+		{"moremesecons_wireless:wireless", "mesecons_torch:mesecon_torch_on", "moremesecons_wireless:wireless"}
+	}
+})
+
 minetest.register_craft({
 	output = "moremesecons_wireless:wireless_off 2",
 	recipe = {
@@ -348,105 +377,7 @@ minetest.register_craft({
 	}
 })
 
-local function remove_jammer(pos)
-	moremesecons.remove_data_from_pos(jammers, pos)
-end
-
-local function add_jammer(pos)
-	remove_jammer(pos)
-	moremesecons.set_data_to_pos(jammers, pos, true)
-end
-
-function is_jammed(pos)
-	local JAMMER_MAX_DISTANCE = moremesecons.setting("wireless", "jammer_max_distance", 15, 1)
-	local JAMMER_MAX_DISTANCE_SQUARE = JAMMER_MAX_DISTANCE^2 -- Cache this result
-
-	for pos_hash, _ in pairs(jammers.tab) do
-		local j_pos = minetest.get_position_from_hash(pos_hash)
-		-- Fast comparisons first
-		if math.abs(pos.x - j_pos.x) <= JAMMER_MAX_DISTANCE and
-				math.abs(pos.y - j_pos.y) <= JAMMER_MAX_DISTANCE and
-				math.abs(pos.z - j_pos.z) <= JAMMER_MAX_DISTANCE and
-				(pos.x - j_pos.x)^2 + (pos.y - j_pos.y)^2 + (pos.z - j_pos.z)^2 <= JAMMER_MAX_DISTANCE_SQUARE then
-			return true
-		end
-	end
-
-	return false
-end
-
-if moremesecons.setting("wireless", "enable_jammer", true) then
-	mesecon.register_node("moremesecons_wireless:jammer", {
-		description = "Wireless Jammer",
-		paramtype = "light",
-		drawtype = "nodebox",
-	},{
-		tiles = {"mesecons_wire_off.png^moremesecons_jammer_top.png", "moremesecons_jammer_bottom.png", "mesecons_wire_off.png^moremesecons_jammer_side_off.png"},
-		node_box = {
-			type = "fixed",
-			fixed = {
-				-- connection
-				{-1/16, -0.5, -0.5, 1/16, -7/16, 0.5},
-				{-0.5, -0.5, -1/16, 0.5, -7/16, 1/16},
-
-				--stabilization
-				{-1/16, -7/16, -1/16, 1/16, -6/16, 1/16},
-
-				-- fields
-				{-7/16, -6/16, -7/16, 7/16, -4/16, 7/16},
-				{-5/16, -4/16, -5/16, 5/16, -3/16, 5/16},
-				{-3/16, -3/16, -3/16, 3/16, -2/16, 3/16},
-				{-1/16, -2/16, -1/16, 1/16, -1/16, 1/16},
-			},
-		},
-		groups = {dig_immediate=2},
-		mesecons = {effector = {
-			rules = mesecon.rules.flat,
-			action_on = function(pos)
-				add_jammer(pos)
-				minetest.swap_node(pos, {name="moremesecons_wireless:jammer_on"})
-			end
-		}}
-	},{
-		tiles = {"mesecons_wire_on.png^moremesecons_jammer_top.png", "moremesecons_jammer_bottom.png", "mesecons_wire_on.png^moremesecons_jammer_side_on.png"},
-		node_box = {
-			type = "fixed",
-			fixed = {
-				-- connection
-				{-1/16, -0.5, -0.5, 1/16, -7/16, 0.5},
-				{-0.5, -0.5, -1/16, 0.5, -7/16, 1/16},
-
-				--stabilization
-				{-1/16, -7/16, -1/16, 1/16, 5/16, 1/16},
-
-				-- fields
-				{-7/16, -6/16, -7/16, 7/16, -4/16, 7/16},
-				{-5/16, -3/16, -5/16, 5/16, -1/16, 5/16},
-				{-3/16, 0, -3/16, 3/16, 2/16, 3/16},
-				{-1/16, 3/16, -1/16, 1/16, 5/16, 1/16},
-			},
-		},
-		groups = {dig_immediate=2, not_in_creative_inventory=1},
-		mesecons = {effector = {
-			rules = mesecon.rules.flat,
-			action_off = function(pos)
-				remove_jammer(pos)
-				minetest.swap_node(pos, {name="moremesecons_wireless:jammer_off"})
-			end
-		}},
-		on_destruct = remove_jammer,
-		on_construct = add_jammer,
-	})
-
-	minetest.register_craft({
-		output = "moremesecons_wireless:jammer_off",
-		recipe = {
-			{"moremesecons_wireless:wireless", "mesecons_torch:mesecon_torch_on", "moremesecons_wireless:wireless"}
-		}
-	})
-end
-
-if moremesecons.setting("wireless", "enable_lbm", false) then
+if enable_lbm then
 	minetest.register_lbm({
 		name = "moremesecons_wireless:add_jammer",
 		nodenames = {"moremesecons_wireless:jammer_on"},
@@ -462,41 +393,43 @@ if moremesecons.setting("wireless", "enable_lbm", false) then
 	})
 end
 
+
 -- Legacy
-if storage:get_string("wireless_meta_2") == "" then
-	local wireless_meta_1 = minetest.deserialize(storage:get_string("wireless_meta"))
-	if not wireless_meta_1 then
-		return
-	end
+if storage and storage:get_string("wireless_rids") and storage:get_string("wireless_rids") ~= "" then
+	-- Upgrade mod storage!
+	local wireless_rids = minetest.deserialize(storage:get_string("wireless_rids"))
+	local old_wireless = table.copy(wireless)
+	wireless = {}
 
-	minetest.log("action", "[moremesecons_wireless] Migrating mod storage data...")
-	local jammers_1 = minetest.deserialize(storage:get_string("jammers"))
+	minetest.after(0, function(old_wireless)
+		-- After loading all mods, try to guess owners based on the areas mod database.
+		-- That won't work for all wireless. Owners of remaining wireless will be set
+		-- to the first player using their formspec.
+		if not areas then
+			return
+		end
+		for RID, pos in ipairs(old_wireless) do
+			local numerous_owners = false
+			local owner
+			for _, area in pairs(areas:getAreasAtPos(pos)) do
+				if owner and area.owner ~= owner then
+					numerous_owners = true
+					break
+				end
+				owner = area.owner
+			end
 
-	local get = function(t, pos)
-		-- FIXME: this does not test explicitly for false,
-		-- but channel is never false
-		return t[pos.z] and t[pos.z][pos.y] and t[pos.z][pos.y][pos.x]
-	end
+			if not numerous_owners and owner then
+				set_owner(pos, owner)
+				set_channel(pos, minetest.get_meta(pos):get_string("channel"))
+			end
+		end
+	end, old_wireless)
 
-	for z, data_z in pairs(wireless_meta_1.owners) do
-	for y, data_y in pairs(data_z) do
-	for x, owner in pairs(data_y) do
-		local pos = {x = x, y = y, z = z}
-		set_owner(pos, owner)
-		set_channel(pos, get(wireless_meta_1.channels, pos))
-	end
-	end
-	end
-
-	for z, data_z in pairs(jammers_1) do
-	for y, data_y in pairs(data_z) do
-	for x, jammer in pairs(data_y) do
-		local pos = {x = x, y = y, z = z}
-		moremesecons.set_data_to_pos(jammers, pos, jammer)
-	end
-	end
-	end
-	minetest.log("action", "[moremesecons_wireless] Done!")
+	-- Remove wireless_rids from storage
+	storage:from_table({
+		jammers = jammers,
+		wireless_meta = wireless_meta,
+		wireless = wireless
+	})
 end
-
-minetest.log("action", "[moremesecons_wireless] loaded.")

moremesecons_wireless/mod.conf

@@ -1,3 +0,0 @@
-name = moremesecons_wireless
-depends = mesecons,moremesecons_utils
-optional_depends = digilines,craft_guide

settingtypes.txt

@@ -63,9 +63,6 @@ moremesecons_teleporter.enable_lbm (Enable Registration LBM) bool false
 
 [Wireless]
 
-# Whether to enable the wireless jammer node
-moremesecons_wireless.enable_jammer (Enable wireless jammer) bool true
-
 # Wireless Jammer action range
 # Any value less than or equal to 0 will be changed to 1 and a NaN value will be changed to the default value
 moremesecons_wireless.jammer_max_distance (Wireless Jammer action range) float 15