Don't use minetest.register_on_item_pickup() if it's not available

Author: LoneWolfHT

mods/ctf/ctf_modebase/player.lua

@@ -219,51 +219,55 @@ function ctf_modebase.player.give_initial_stuff(player)
 	inv:set_list("main", new)
 end
 
-minetest.register_on_item_pickup(function(itemstack, picker)
-	if ctf_modebase.current_mode and ctf_teams.get(picker) then
-		local mode = ctf_modebase:get_current_mode()
-		for name, func in pairs(mode.initial_stuff_item_levels) do
-			local priority = func(itemstack)
-
-			if priority then
-				local inv = picker:get_inventory()
-				for i=1, 8 do -- loop through the top row of the player's inv
-					local compare = inv:get_stack("main", i)
-
-					if not mode.is_bound_item or not mode.is_bound_item(picker, compare:get_name()) then
-						local cprio = func(compare)
-
-						if cprio and cprio < priority then
-							local item, typ = simplify_for_saved_stuff(compare:get_name())
-							minetest.log(dump(item)..dump(typ))
-							inv:set_stack("main", i, itemstack)
-
-							if item == "sword" and typ == "stone" and
-							ctf_settings.get(picker, "auto_trash_stone_swords") == "true" then
-								return ItemStack("")
-							end
-
-							if item ~= "sword" and typ == "stone" and
-							ctf_settings.get(picker, "auto_trash_stone_tools") == "true" then
-								return ItemStack("")
-							else
-								local result = inv:add_item("main", compare):get_count()
+if minetest.register_on_item_pickup then
+	minetest.register_on_item_pickup(function(itemstack, picker)
+		if ctf_modebase.current_mode and ctf_teams.get(picker) then
+			local mode = ctf_modebase:get_current_mode()
+			for name, func in pairs(mode.initial_stuff_item_levels) do
+				local priority = func(itemstack)
+
+				if priority then
+					local inv = picker:get_inventory()
+					for i=1, 8 do -- loop through the top row of the player's inv
+						local compare = inv:get_stack("main", i)
+
+						if not mode.is_bound_item or not mode.is_bound_item(picker, compare:get_name()) then
+							local cprio = func(compare)
+
+							if cprio and cprio < priority then
+								local item, typ = simplify_for_saved_stuff(compare:get_name())
+								minetest.log(dump(item)..dump(typ))
+								inv:set_stack("main", i, itemstack)
+
+								if item == "sword" and typ == "stone" and
+								ctf_settings.get(picker, "auto_trash_stone_swords") == "true" then
+									return ItemStack("")
+								end
 
-								if result == 0 then
+								if item ~= "sword" and typ == "stone" and
+								ctf_settings.get(picker, "auto_trash_stone_tools") == "true" then
 									return ItemStack("")
 								else
-									compare:set_count(result)
-									return compare
+									local result = inv:add_item("main", compare):get_count()
+
+									if result == 0 then
+										return ItemStack("")
+									else
+										compare:set_count(result)
+										return compare
+									end
 								end
 							end
 						end
 					end
+					break -- We already found a place for it, don't check for one held by a different item type
 				end
-				break -- We already found a place for it, don't check for one held by a different item type
 			end
 		end
-	end
-end)
+	end)
+else
+	minetest.log("error", "You aren't using the latest version of Minetest, auto-trashing and auto-sort won't work")
+end
 
 function ctf_modebase.player.empty_inv(player)
 	player:get_inventory():set_list("main", {})