use message parts instead of concatenation

mods/default/functions.lua

@@ -745,13 +745,13 @@ end
 function default.set_inventory_action_loggers(def, name)
 	def.on_metadata_inventory_move = function(pos, from_list, from_index,
 			to_list, to_index, count, player)
-		default.log_player_action(player, "moves stuff in " .. name, pos)
+		default.log_player_action(player, "moves stuff in", name, pos)
 	end
 	def.on_metadata_inventory_put = function(pos, listname, index, stack, player)
-		default.log_player_action(player, "moves " .. stack:get_name() .. " to " .. name, pos)
+		default.log_player_action(player, "moves", stack:get_name(), "to", name, pos)
 	end
 	def.on_metadata_inventory_take = function(pos, listname, index, stack, player)
-		default.log_player_action(player, "takes " .. stack:get_name() .. " from " .. name, pos)
+		default.log_player_action(player, "takes", stack:get_name(), "from", name, pos)
 	end
 end
 

mods/default/nodes.lua

@@ -2058,9 +2058,8 @@ local function coral_on_place(itemstack, placer, pointed_thing)
 	if minetest.is_protected(pos_under, player_name) or
 			minetest.is_protected(pos_above, player_name) then
 		default.log_player_action(placer,
-			"tried to place " .. itemstack:get_name()
-			.. " at protected position "
-			, pos_under)
+			"tried to place", itemstack:get_name(),
+			"at protected position", pos_under)
 		minetest.record_protection_violation(pos_under, player_name)
 		return itemstack
 	end

mods/default/trees.lua

@@ -572,7 +572,7 @@ function default.sapling_on_place(itemstack, placer, pointed_thing,
 		return itemstack
 	end
 
-	default.log_player_action(placer, "places node " .. sapling_name, pos)
+	default.log_player_action(placer, "places node", sapling_name, pos)
 
 	local take_item = not minetest.is_creative_enabled(player_name)
 	local newnode = {name = sapling_name}

mods/farming/api.lua

@@ -178,7 +178,7 @@ farming.place_seed = function(itemstack, placer, pointed_thing, plantname)
 	end
 
 	-- add the node and remove 1 item from the itemstack
-	default.log_player_action(placer, "places node " .. plantname, pt.above)
+	default.log_player_action(placer, "places node", plantname, pt.above)
 	minetest.add_node(pt.above, {name = plantname, param2 = 1})
 	tick(pt.above)
 	if not minetest.is_creative_enabled(player_name) then

mods/tnt/init.lua

@@ -633,8 +633,7 @@ function tnt.register_tnt(def)
 				if puncher:get_wielded_item():get_name() == "default:torch" then
 					minetest.swap_node(pos, {name = name .. "_burning"})
 					minetest.registered_nodes[name .. "_burning"].on_construct(pos)
-					default.log_player_action(puncher,
-						"ignites " .. node.name, pos)
+					default.log_player_action(puncher, "ignites", node.name, pos)
 				end
 			end,
 			on_blast = function(pos, intensity)