[pull] master from MT-CTF:master

.github/pull_request_template.md

@@ -0,0 +1,3 @@
+
+
+- [ ] This PR has been tested locally

.github/workflows/luacheck.yml

@@ -0,0 +1,10 @@
+name: luacheck
+on: [push, pull_request]
+jobs:
+  luacheck:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Luacheck
+        uses: lunarmodules/luacheck@master

.gitignore

@@ -13,4 +13,6 @@ debug.txt
 /mods/ctf/ctf_map/textures/*Back.*
 /mods/ctf/ctf_map/textures/*Left.*
 /mods/ctf/ctf_map/textures/*Right.*
-/mods/ctf/ctf_map/textures/*_screenshot.png
+/mods/ctf/ctf_map/textures/*_screenshot.png
+
+*.blend1

README.md

@@ -45,22 +45,22 @@ Note that this version may be slightly behind the git version, but it will be a
 * If you use Visual Studio Code we recommend these extensions:
   * https://marketplace.visualstudio.com/items?itemName=sumneko.lua
   * https://marketplace.visualstudio.com/items?itemName=dwenegar.vscode-luacheck
+  * https://marketplace.visualstudio.com/items?itemName=GreenXenith.minetest-tools
 
 ## License
 
 Created by [rubenwardy](https://rubenwardy.com/).
 Developed by [LandarVargan](https://github.com/LoneWolfHT).
 Previous Developers: [savilli](https://github.com/savilli).
 
-Licenses where not specified:
-Code: LGPLv2.1+
-Textures: CC-BY-SA 3.0
-
-### Textures
+Check out [mods/](mods/) to see all the installed mods and their respective licenses.
 
-* [Header](menu/header.png): CC BY-SA 4.0 by xenonca
-* [Background Image](menu/background.png): CC0 (where applicable) by Apelta (Uses [Minetest Game](https://github.com/minetest/minetest_game) textures, the majority of which are licensed CC-BY-SA 3.0). The player skin used is licensed CC-BY-SA 3.0
+Licenses where not specified:\
+Code: [GNU LGPLv2.1+](https://www.gnu.org/licenses/old-licenses/lgpl-2.1.html)\
+Media: [CC BY-SA 3.0 Unported](https://creativecommons.org/licenses/by-sa/3.0/)
 
-### Mods
+### Textures
 
-Check out [mods/](mods/) to see all the installed mods and their respective licenses.
+* [Header](menu/header.png): [CC BY 3.0 Unported](https://creativecommons.org/licenses/by/3.0/) by [SuddenSFD](https://github.com/SuddenSFD)
+* [Background Image](menu/background.png): [CC BY-SA 4.0 International](https://creativecommons.org/licenses/by-sa/4.0/) (where applicable) by [GreenBlob](https://github.com/a-blob) (Uses [Minetest Game](https://github.com/minetest/minetest_game) textures, the majority of which are licensed [CC BY-SA 3.0 Unported](https://creativecommons.org/licenses/by-sa/3.0/)). The player skins used are licensed under [CC BY-SA 3.0 Unported](https://creativecommons.org/licenses/by-sa/3.0/)
+* [Icon](menu/icon.png): [CC BY 3.0](https://creativecommons.org/licenses/by/3.0/) by [SuddenSFD](https://github.com/SuddenSFD)

docs/accurate_statbar.patch

@@ -6,13 +6,16 @@ index 46c947b6..f4372843 100644
  local hud_ids = {}
 
  local function scaleToDefault(player, field)
-+       return player["get_" .. field](player)
--        -- Scale "hp" or "breath" to the default dimensions
--       local current = player["get_" .. field](player)
--       local nominal = core["PLAYER_MAX_".. field:upper() .. "_DEFAULT"]
--       local max_display = math.max(nominal,
--               math.max(player:get_properties()[field .. "_max"], current))
--       return current / max_display * nominal
++       if field == "health" then
++           return player["get_" .. field](player)
++       else
+            -- Scale "hp" or "breath" to the default dimensions
+            local current = player["get_" .. field](player)
+            local nominal = core["PLAYER_MAX_".. field:upper() .. "_DEFAULT"]
+            local max_display = math.max(nominal,
+                math.max(player:get_properties()[field .. "_max"], current))
+            return current / max_display * nominal
++       end
  end
 
  local function update_builtin_statbars(player)

docs/ctf-api.md

@@ -24,7 +24,7 @@ ctf_settings.register("my_setting", {
 	on_change = function(player, new_value)
 		<...>
 	end
-}
+})
 ```
 
 ### `ctf_settings.set(player, setting, value)`
@@ -39,7 +39,10 @@ ctf_settings.register("my_setting", {
 
 ---
 # mods/ctf/
-TODO
+TODO, below is a collection of quick notes for later
+
+## ctf_teams
+* https://modern.ircdocs.horse/formatting.html#colors-16-98
 
 ---
 # mods/mtg/

mods/apis/ctf_gui/dev.lua

@@ -1,31 +1,48 @@
+local unset_function = "return "
+
 function ctf_gui.show_formspec_dev(player, formname, formspec, formcontext)
 	local filepath = minetest.get_worldpath().."/ctf_gui/"
 	local filename = filepath.."file_edit.txt"
+	local slower_loop = false
+
+	minetest.chat_send_all("Started formspec editing file at "..filename)
 
 	minetest.mkdir(filepath)
 
 	local file = assert(io.open(filename, "w"))
-
-	file:write(formspec)
-
+		if type(formspec) ~= "function" then
+			file:write(formspec)
+		else
+			file:write(unset_function)
+		end
 	file:close()
 
 	local function interval()
-		if formspec:sub(1, 3) == "[f]" then
-			local result, form = pcall(loadstring(formspec:sub(4)), formcontext)
-			ctf_gui.show_formspec(player, formname, result and form or "")
+		if type(formspec) == "function" then
+			ctf_gui.show_formspec(player, formname, formspec(formcontext))
+		elseif formspec:match("^%s*return") then
+			local result, form = pcall((loadstring(formspec) or function() return function() end end)(), formcontext)
+
+			ctf_gui.show_formspec(player, formname,
+				result and form or "size[10,10]hypertext[0,0;10,10;err;"..minetest.formspec_escape(form or "").."]"
+			)
+
+			slower_loop = not result
 		else
 			ctf_gui.show_formspec(player, formname, formspec)
 		end
 
-		minetest.after(1, function()
+		minetest.after(slower_loop and 3 or 1, function()
 			local f = assert(io.open(filename, "r"))
+			local new_form = f:read("*a")
 
-			formspec = f:read("*a")
+			if new_form ~= unset_function then
+				formspec = new_form
+			end
 
 			f:close()
 
-			if formspec:match("^exit") then
+			if type(formspec) == "function" or not formspec:match("^exit") then
 				interval()
 			else
 				minetest.request_shutdown("Formspec dev requested shutdown", true)

mods/apis/ctf_gui/init.lua

@@ -39,6 +39,7 @@ function ctf_gui.init()
 				local action = ctx._on_formspec_input(pname, ctx, fields, ...)
 
 				if action == "refresh" then
+					minetest.log("action", "Refreshing formspec "..dump(ctx._formname).." to "..dump(pname))
 					minetest.show_formspec(pname, ctx._formname, ctx._formspec(ctx))
 				end
 			end
@@ -115,6 +116,7 @@ function ctf_gui.show_formspec(player, formname, formspec, formcontext)
 	context[player]._formname = formname
 	context[player]._formspec = formspec
 
+	minetest.log("action", "Showing new_formspec "..dump(formname).." to "..dump(player))
 	if type(formspec) == "function" then
 		minetest.show_formspec(player, formname, formspec(formcontext))
 	else
@@ -144,6 +146,7 @@ do
 			end
 		end
 
+		-- minetest.log("action", "[ctf_gui] unpacking: "..dump(l))
 		return format(base, unpck(l))
 	end
 
@@ -457,9 +460,16 @@ function ctf_gui.old_show_formspec(player, formname, formdef)
 		formdef._info = formdef
 		context[player] = formdef
 
-		minetest.show_formspec(player, formname, formspec)
+		if minetest.get_player_by_name(player) then
+			minetest.log("action", "Showing formspec "..dump(formname).." to "..dump(player))
+			minetest.show_formspec(player, formname, formspec)
+		end
 	end,
 	formdef, ctf_gui.ELEM_SIZE, ctf_gui.SCROLLBAR_WIDTH)
 end
 
+minetest.register_on_leaveplayer(function(player)
+	context[player:get_player_name()] = nil
+end)
+
 dofile(minetest.get_modpath("ctf_gui").."/dev.lua")