Log every inventory actions

technic/machines/HV/nuclear_reactor.lua

@@ -437,6 +437,10 @@ local digiline_def = function(pos, _, channel, msg)
 	end
 end
 
+local on_metadata_inventory_move = technic.machine_on_inventory_move("technic:hv_nuclear_reactor_core")
+local on_metadata_inventory_put  = technic.machine_on_inventory_put("technic:hv_nuclear_reactor_core")
+local on_metadata_inventory_take = technic.machine_on_inventory_take("technic:hv_nuclear_reactor_core")
+
 minetest.register_node("technic:hv_nuclear_reactor_core", {
 	description = reactor_desc,
 	tiles = {
@@ -480,6 +484,9 @@ minetest.register_node("technic:hv_nuclear_reactor_core", {
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
 	allow_metadata_inventory_move = technic.machine_inventory_move,
+	on_metadata_inventory_move = on_metadata_inventory_move,
+	on_metadata_inventory_put = on_metadata_inventory_put,
+	on_metadata_inventory_take = on_metadata_inventory_take,
 	technic_run = run,
 })
 
@@ -521,6 +528,9 @@ minetest.register_node("technic:hv_nuclear_reactor_core_active", {
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
 	allow_metadata_inventory_move = technic.machine_inventory_move,
+	on_metadata_inventory_move = on_metadata_inventory_move,
+	on_metadata_inventory_put = on_metadata_inventory_put,
+	on_metadata_inventory_take = on_metadata_inventory_take,
 	technic_run = run,
 	technic_on_disable = function(pos, node)
 		local timer = minetest.get_node_timer(pos)

technic/machines/HV/quarry.lua

@@ -554,6 +554,9 @@ minetest.register_node("technic:quarry", {
 	allow_metadata_inventory_take = function(pos, listname, index, stack, player)
 		return player_allowed(pos, player:get_player_name()) and stack:get_count() or 0
 	end,
+	on_metadata_inventory_move = technic.machine_on_inventory_move("technic:quarry"),
+	on_metadata_inventory_put = technic.machine_on_inventory_put("technic:quarry"),
+	on_metadata_inventory_take = technic.machine_on_inventory_take("technic:quarry"),
 	mesecons = {
 		effector = {
 			action_on = function(pos)

technic/machines/MV/tool_workshop.lua

@@ -131,6 +131,9 @@ minetest.register_node("technic:tool_workshop", {
 	can_dig = technic.machine_can_dig,
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
+	on_metadata_inventory_move = technic.machine_on_inventory_move("technic:tool_workshop"),
+	on_metadata_inventory_put = technic.machine_on_inventory_put("technic:tool_workshop"),
+	on_metadata_inventory_take = technic.machine_on_inventory_take("technic:tool_workshop"),
 	tube = {
 		can_insert = function (pos, node, stack, direction)
 			return minetest.get_meta(pos):get_inventory():room_for_item("src", stack)

technic/machines/other/coal_alloy_furnace.lua

@@ -47,6 +47,10 @@ formspec = formspec..
 	"listring[context;fuel]"..
 	"listring[current_player;main]"
 
+local on_metadata_inventory_move = technic.machine_on_inventory_move("technic:coal_alloy_furnace")
+local on_metadata_inventory_put  = technic.machine_on_inventory_put("technic:coal_alloy_furnace")
+local on_metadata_inventory_take = technic.machine_on_inventory_take("technic:coal_alloy_furnace")
+
 minetest.register_node("technic:coal_alloy_furnace", {
 	description = machine_name,
 	tiles = {"technic_coal_alloy_furnace_top.png",  "technic_coal_alloy_furnace_bottom.png",
@@ -72,6 +76,9 @@ minetest.register_node("technic:coal_alloy_furnace", {
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
 	allow_metadata_inventory_move = technic.machine_inventory_move,
+	on_metadata_inventory_move = on_metadata_inventory_move,
+	on_metadata_inventory_put = on_metadata_inventory_put,
+	on_metadata_inventory_take = on_metadata_inventory_take,
 })
 
 minetest.register_node("technic:coal_alloy_furnace_active", {
@@ -92,6 +99,9 @@ minetest.register_node("technic:coal_alloy_furnace_active", {
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
 	allow_metadata_inventory_move = technic.machine_inventory_move,
+	on_metadata_inventory_move = on_metadata_inventory_move,
+	on_metadata_inventory_put = on_metadata_inventory_put,
+	on_metadata_inventory_take = on_metadata_inventory_take,
 })
 
 minetest.register_abm({

technic/machines/other/constructor.lua

@@ -205,6 +205,9 @@ local function make_constructor(mark, length)
 		allow_metadata_inventory_put = allow_inventory_put,
 		allow_metadata_inventory_take = technic.machine_inventory_take,
 		allow_metadata_inventory_move = technic.machine_inventory_move,
+		on_metadata_inventory_move = technic.machine_on_inventory_move("technic:constructor_mk"..mark),
+		on_metadata_inventory_put = technic.machine_on_inventory_put("technic:constructor_mk"..mark),
+		on_metadata_inventory_take = technic.machine_on_inventory_take("technic:constructor_mk"..mark),
 		on_rotate = function(pos, node, user, mode, new_param2)
 			if mode ~= 1 then
 				return false

technic/machines/other/injector.lua

@@ -152,6 +152,9 @@ minetest.register_node("technic:injector", {
 	allow_metadata_inventory_put = technic.machine_inventory_put,
 	allow_metadata_inventory_take = technic.machine_inventory_take,
 	allow_metadata_inventory_move = technic.machine_inventory_move,
+	on_metadata_inventory_move = technic.machine_on_inventory_move(nodename),
+	on_metadata_inventory_put = technic.machine_on_inventory_put(nodename),
+	on_metadata_inventory_take = technic.machine_on_inventory_take(nodename),
 	after_place_node = pipeworks.after_place,
 	after_dig_node = pipeworks.after_dig
 })

technic/machines/register/battery_box.lua

@@ -331,6 +331,11 @@ function technic.register_battery_box(nodename, data)
 		return true
 	end
 
+	-- This nodename is not an actual nodename - it is the base nodename.
+	local on_metadata_inventory_move = technic.machine_on_inventory_move(nodename)
+	local on_metadata_inventory_put  = technic.machine_on_inventory_put(nodename)
+	local on_metadata_inventory_take = technic.machine_on_inventory_take(nodename)
+
 	for i = 0, 8 do
 		local groups = {snappy=2, choppy=2, oddly_breakable_by_hand=2,
 				technic_machine=1, ["technic_"..ltier]=1, axey=2, handy=1}
@@ -389,6 +394,9 @@ function technic.register_battery_box(nodename, data)
 			allow_metadata_inventory_put = technic.machine_inventory_put,
 			allow_metadata_inventory_take = technic.machine_inventory_take,
 			allow_metadata_inventory_move = technic.machine_inventory_move,
+			on_metadata_inventory_move = on_metadata_inventory_move,
+			on_metadata_inventory_put = on_metadata_inventory_put,
+			on_metadata_inventory_take = on_metadata_inventory_take,
 			technic_run = run,
 			on_timer = on_timer,
 			on_rightclick = function(pos) update_node(pos, true) end,

technic/machines/register/common.lua

@@ -190,3 +190,42 @@ function technic.machine_inventory_move(pos, from_list, from_index,
 	local stack = minetest.get_meta(pos):get_inventory():get_stack(from_list, from_index)
 	return inv_change(pos, player, count, from_list, to_list, stack)
 end
+
+function technic.machine_on_inventory_put(nodename)
+	return function(pos, listname, index, stack, player)
+		minetest.log("action", string.format(
+			"%s put %s into %s[%u] of %s at %s",
+			player:is_player() and player:get_player_name() or "A mod",
+			stack:to_string(),
+			listname, index,
+			nodename,
+			minetest.pos_to_string(pos)
+		))
+	end
+end
+
+function technic.machine_on_inventory_take(nodename)
+	return function(pos, listname, index, stack, player)
+		minetest.log("action", string.format(
+			"%s took %s from %s[%u] of %s at %s",
+			player:is_player() and player:get_player_name() or "A mod",
+			stack:to_string(),
+			listname, index,
+			nodename,
+			minetest.pos_to_string(pos)
+		))
+	end
+end
+
+function technic.machine_on_inventory_move(nodename)
+	return function(pos, from_list, from_index, to_list, to_index, count, player)
+		minetest.log("action", string.format(
+			"%s moved item from %s[%u] to %s[%u] of %s at %s",
+			player:is_player() and player:get_player_name() or "A mod",
+			from_list, from_index,
+			to_list, to_index,
+			nodename,
+			minetest.pos_to_string(pos)
+		))
+	end
+end

technic/machines/register/machine_base.lua

@@ -167,6 +167,10 @@ function technic.register_base_machine(nodename, data)
 		tentry = ""
 	end
 
+	local on_metadata_inventory_move = technic.machine_on_inventory_move(nodename)
+	local on_metadata_inventory_put  = technic.machine_on_inventory_put(nodename)
+	local on_metadata_inventory_take = technic.machine_on_inventory_take(nodename)
+
 	minetest.register_node(colon..nodename, {
 		description = def.description,
 		tiles = {
@@ -216,6 +220,9 @@ function technic.register_base_machine(nodename, data)
 		allow_metadata_inventory_put = technic.machine_inventory_put,
 		allow_metadata_inventory_take = technic.machine_inventory_take,
 		allow_metadata_inventory_move = technic.machine_inventory_move,
+		on_metadata_inventory_move = on_metadata_inventory_move,
+		on_metadata_inventory_put = on_metadata_inventory_put,
+		on_metadata_inventory_take = on_metadata_inventory_take,
 		technic_run = run,
 		after_place_node = def.tube and pipeworks.after_place,
 		after_dig_node = technic.machine_after_dig_node,
@@ -265,6 +272,9 @@ function technic.register_base_machine(nodename, data)
 		allow_metadata_inventory_put = technic.machine_inventory_put,
 		allow_metadata_inventory_take = technic.machine_inventory_take,
 		allow_metadata_inventory_move = technic.machine_inventory_move,
+		on_metadata_inventory_move = on_metadata_inventory_move,
+		on_metadata_inventory_put = on_metadata_inventory_put,
+		on_metadata_inventory_take = on_metadata_inventory_take,
 		technic_run = run,
 		technic_disabled_machine_name = nodename,
 		on_receive_fields = function(pos, formname, fields, sender)

technic_chests/inventory.lua

@@ -166,6 +166,11 @@ function technic.chests.move_inv(from_inv, to_inv, filter)
 	return moved_items
 end
 
+-- DEPRECATED: use technic.machine_on_inventory_* instead.
+---@deprecated
+---@see technic.machine_on_inventory_put
+---@see technic.machine_on_inventory_take
+---@see technic.machine_on_inventory_move
 function technic.chests.log_inv_change(pos, name, change, items)
 	local spos = minetest.pos_to_string(pos)
 	if change == "move" then
@@ -176,3 +181,23 @@ function technic.chests.log_inv_change(pos, name, change, items)
 		minetest.log("action", name.." takes "..items.." from chest at "..spos)
 	end
 end
+
+function technic.chests.log_fast_move(pos, name, nodename, change, items)
+	local spos = minetest.pos_to_string(pos)
+	local itemlist = ""
+	for _, stack in ipairs(items) do
+		itemlist = itemlist .. " " .. stack.name .. " " .. stack.count
+	end
+
+	if change == "put" then
+		minetest.log("action", string.format(
+			"%s fast-moved the following item from their inventory into %s at %s:",
+			name, nodename, spos
+		) .. itemlist)
+	elseif change == "take" then
+		minetest.log("action", string.format(
+			"%s fast-moved the following item from %s at %s into their inventory:",
+			name, nodename, spos
+		) .. itemlist)
+	end
+end

technic_chests/register.lua

@@ -57,6 +57,10 @@ function technic.chests.register_chest(nodename, data)
 		data.digilines = nil
 	end
 
+	local on_metadata_inventory_move = technic.machine_on_inventory_move(nodename)
+	local on_metadata_inventory_put  = technic.machine_on_inventory_put(nodename)
+	local on_metadata_inventory_take = technic.machine_on_inventory_take(nodename)
+
 	data.formspec = technic.chests.get_formspec(data)
 	local def = {
 		description = data.description,
@@ -144,7 +148,7 @@ function technic.chests.register_chest(nodename, data)
 				if data.digilines and meta:get_int("send_take") == 1 then
 					technic.chests.send_digiline_message(pos, "take", player, moved_items)
 				end
-				technic.chests.log_inv_change(pos, player:get_player_name(), "take", "stuff")
+				technic.chests.log_fast_move(pos, player:get_player_name(), nodename, "take", moved_items)
 				return 0
 			end
 			return count
@@ -160,7 +164,7 @@ function technic.chests.register_chest(nodename, data)
 				if data.digilines and meta:get_int("send_put") == 1 then
 					technic.chests.send_digiline_message(pos, "put", player, moved_items)
 				end
-				technic.chests.log_inv_change(pos, player:get_player_name(), "put", "stuff")
+				technic.chests.log_fast_move(pos, player:get_player_name(), nodename, "put", moved_items)
 				return 0
 			end
 			return stack:get_count()
@@ -171,21 +175,9 @@ function technic.chests.register_chest(nodename, data)
 			end
 			return stack:get_count()
 		end,
-		on_metadata_inventory_move = function(pos, from_list, from_index, to_list, to_index, count, player)
-			technic.chests.log_inv_change(pos, player:get_player_name(), "move", "stuff")
-		end,
-		on_metadata_inventory_put = function(pos, listname, index, stack, player)
-			if data.digilines and minetest.get_meta(pos):get_int("send_put") == 1 then
-				technic.chests.send_digiline_message(pos, "put", player, {stack:to_table()})
-			end
-			technic.chests.log_inv_change(pos, player:get_player_name(), "put", stack:get_name())
-		end,
-		on_metadata_inventory_take = function(pos, listname, index, stack, player)
-			if data.digilines and minetest.get_meta(pos):get_int("send_take") == 1 then
-				technic.chests.send_digiline_message(pos, "take", player, {stack:to_table()})
-			end
-			technic.chests.log_inv_change(pos, player:get_player_name(), "take", stack:get_name())
-		end,
+		on_metadata_inventory_move = on_metadata_inventory_move,
+		on_metadata_inventory_put = on_metadata_inventory_put,
+		on_metadata_inventory_take = on_metadata_inventory_take,
 		on_blast = function(pos)
 			if data.locked or data.protected then
 				return

technic_cnc/api.lua

@@ -383,6 +383,10 @@ function technic_cnc.register_cnc_machine(nodename, def)
 		groups.tubedevice_receiver = 1
 	end
 
+	local on_metadata_inventory_move = technic.machine_on_inventory_move(nodename)
+	local on_metadata_inventory_put  = technic.machine_on_inventory_put(nodename)
+	local on_metadata_inventory_take = technic.machine_on_inventory_take(nodename)
+
 	-- Inactive state CNC machine
 	minetest.register_node(":" .. nodename, {
 		description = def.description,
@@ -412,6 +416,9 @@ function technic_cnc.register_cnc_machine(nodename, def)
 		allow_metadata_inventory_put = def.allow_metadata_inventory_put or allow_metadata_inventory_put,
 		allow_metadata_inventory_take = def.allow_metadata_inventory_take or allow_metadata_inventory_take,
 		allow_metadata_inventory_move = def.allow_metadata_inventory_move or allow_metadata_inventory_move,
+		on_metadata_inventory_move = on_metadata_inventory_move,
+		on_metadata_inventory_put = on_metadata_inventory_put,
+		on_metadata_inventory_take = on_metadata_inventory_take,
 		on_receive_fields = on_receive_fields,
 		technic_run = def.technic_run or technic_run,
 	})
@@ -436,6 +443,9 @@ function technic_cnc.register_cnc_machine(nodename, def)
 			allow_metadata_inventory_put = def.allow_metadata_inventory_put or allow_metadata_inventory_put,
 			allow_metadata_inventory_take = def.allow_metadata_inventory_take or allow_metadata_inventory_take,
 			allow_metadata_inventory_move = def.allow_metadata_inventory_move or allow_metadata_inventory_move,
+			on_metadata_inventory_move = on_metadata_inventory_move,
+			on_metadata_inventory_put = on_metadata_inventory_put,
+			on_metadata_inventory_take = on_metadata_inventory_take,
 			on_receive_fields = on_receive_fields,
 			technic_run = def.technic_run or technic_run,
 			technic_disabled_machine_name = nodename,