.stylua.toml

@@ -0,0 +1,10 @@
+column_width = 80
+line_endings = "Unix"
+indent_type = "Spaces"
+indent_width = 2
+quote_style = "AutoPreferDouble"
+call_parentheses = "None"
+collapse_simple_statement = "Always"
+
+[sort_requires]
+enabled = true

api.lua

@@ -13,10 +13,10 @@ format instead. For example, multiple pages could be shown in one formspec.
 ---@field contexts table Table of `contexts[playername] = player_context`.
 ---@field enabled boolean Set to false to disable. Good for inventory rehaul mods like unified inventory.
 sfinv = {
-	pages = {},
-	pages_unordered = {},
-	contexts = {},
-	enabled = true,
+  pages = {},
+  pages_unordered = {},
+  contexts = {},
+  enabled = true,
 }
 
 ---@alias SFInvFields string[] A list of page names.
@@ -41,27 +41,33 @@ sfinv = {
 ---@param name string
 ---@param def SFInvDef
 function sfinv.register_page(name, def)
-	assert(name, "Invalid sfinv page. Requires a name")
-	assert(def, "Invalid sfinv page. Requires a def[inition] table")
-	assert(def.get, "Invalid sfinv page. Def requires a get function.")
-	assert(not sfinv.pages[name], "Attempt to register already registered sfinv page " .. dump(name))
-
-	sfinv.pages[name] = def
-	def.name = name
-	table.insert(sfinv.pages_unordered, def)
+  assert(name, "Invalid sfinv page. Requires a name")
+  assert(def, "Invalid sfinv page. Requires a def[inition] table")
+  assert(def.get, "Invalid sfinv page. Def requires a get function.")
+  assert(
+    not sfinv.pages[name],
+    "Attempt to register already registered sfinv page " .. dump(name)
+  )
+
+  sfinv.pages[name] = def
+  def.name = name
+  table.insert(sfinv.pages_unordered, def)
 end
 
 -- Overrides fields of an page registered with register_page.
 ---@param name string
 ---@param def SFInvDef
 function sfinv.override_page(name, def)
-	assert(name, "Invalid sfinv page override. Requires a name")
-	assert(def, "Invalid sfinv page override. Requires a def[inition] table")
-	local page = sfinv.pages[name]
-	assert(page, "Attempt to override sfinv page " .. dump(name) .. " which does not exist.")
-	for key, value in pairs(def) do
-		page[key] = value
-	end
+  assert(name, "Invalid sfinv page override. Requires a name")
+  assert(def, "Invalid sfinv page override. Requires a def[inition] table")
+  local page = sfinv.pages[name]
+  assert(
+    page,
+    "Attempt to override sfinv page " .. dump(name) .. " which does not exist."
+  )
+  for key, value in pairs(def) do
+    page[key] = value
+  end
 end
 
 -- Creates tabheader or "".
@@ -72,12 +78,16 @@ end
 ---@return string formspec
 ---@diagnostic disable-next-line: duplicate-set-field
 function sfinv.get_nav_fs(player, context, nav, current_idx)
-	-- Only show tabs if there is more than one page.
-	if #nav > 1 then
-		return "tabheader[0,0;sfinv_nav_tabs;" .. table.concat(nav, ",") .. ";" .. current_idx .. ";true;false]"
-	else
-		return ""
-	end
+  -- Only show tabs if there is more than one page.
+  if #nav > 1 then
+    return "tabheader[0,0;sfinv_nav_tabs;"
+      .. table.concat(nav, ",")
+      .. ";"
+      .. current_idx
+      .. ";true;false]"
+  else
+    return ""
+  end
 end
 
 local theme_inv = [[
@@ -94,157 +104,152 @@ local theme_inv = [[
 ---@return string formspec
 ---@diagnostic disable-next-line: duplicate-set-field
 function sfinv.make_formspec(player, context, content, show_inv, size)
-	local tmp = {
-		size or "size[8,9.1]",
-		sfinv.get_nav_fs(player, context, context.nav_titles, context.nav_idx),
-		show_inv and theme_inv or "",
-		content,
-	}
-	return table.concat(tmp, "")
+  local tmp = {
+    size or "size[8,9.1]",
+    sfinv.get_nav_fs(player, context, context.nav_titles, context.nav_idx),
+    show_inv and theme_inv or "",
+    content,
+  }
+  return table.concat(tmp, "")
 end
 
 -- Get the page name of the first page to show to a player.
 ---@param player mt.PlayerObjectRef
 ---@return string
-function sfinv.get_homepage_name(player)
-	return "sfinv:crafting"
-end
+function sfinv.get_homepage_name(player) return "sfinv:crafting" end
 
 -- Builds current page's formspec.
 ---@param player mt.PlayerObjectRef
 ---@param context SFInvContext
 ---@return string
 function sfinv.get_formspec(player, context)
-	-- Generate navigation tabs
-	local nav = {}
-	local nav_ids = {}
-	local current_idx = 1
-	for i, pdef in pairs(sfinv.pages_unordered) do
-		if not pdef.is_in_nav or pdef:is_in_nav(player, context) then
-			nav[#nav + 1] = pdef.title
-			nav_ids[#nav_ids + 1] = pdef.name
-			if pdef.name == context.page then
-				current_idx = #nav_ids
-			end
-		end
-	end
-	context.nav = nav_ids
-	context.nav_titles = nav
-	context.nav_idx = current_idx
-
-	-- Generate formspec
-	local page = sfinv.pages[context.page] or sfinv.pages["404"]
-	if page then
-		return page:get(player, context)
-	else
-		local old_page = context.page
-		local home_page = sfinv.get_homepage_name(player)
-
-		if old_page == home_page then
-			minetest.log("error", "[sfinv] Couldn't find " .. dump(old_page) .. ", which is also the old page")
-
-			return ""
-		end
-
-		context.page = home_page
-		assert(sfinv.pages[context.page], "[sfinv] Invalid homepage")
-		minetest.log("warning", "[sfinv] Couldn't find " .. dump(old_page) .. " so switching to homepage")
-
-		return sfinv.get_formspec(player, context)
-	end
+  -- Generate navigation tabs
+  local nav = {}
+  local nav_ids = {}
+  local current_idx = 1
+  for i, pdef in pairs(sfinv.pages_unordered) do
+    if not pdef.is_in_nav or pdef:is_in_nav(player, context) then
+      nav[#nav + 1] = pdef.title
+      nav_ids[#nav_ids + 1] = pdef.name
+      if pdef.name == context.page then current_idx = #nav_ids end
+    end
+  end
+  context.nav = nav_ids
+  context.nav_titles = nav
+  context.nav_idx = current_idx
+
+  -- Generate formspec
+  local page = sfinv.pages[context.page] or sfinv.pages["404"]
+  if page then
+    return page:get(player, context)
+  else
+    local old_page = context.page
+    local home_page = sfinv.get_homepage_name(player)
+
+    if old_page == home_page then
+      minetest.log(
+        "error",
+        "[sfinv] Couldn't find "
+          .. dump(old_page)
+          .. ", which is also the old page"
+      )
+
+      return ""
+    end
+
+    context.page = home_page
+    assert(sfinv.pages[context.page], "[sfinv] Invalid homepage")
+    minetest.log(
+      "warning",
+      "[sfinv] Couldn't find " .. dump(old_page) .. " so switching to homepage"
+    )
+
+    return sfinv.get_formspec(player, context)
+  end
 end
 
 -- Gets the player's context.
 ---@param player mt.PlayerObjectRef
 ---@return SFInvContext
 function sfinv.get_or_create_context(player)
-	local name = player:get_player_name()
-	local context = sfinv.contexts[name]
-	if not context then
-		context = {
-			page = sfinv.get_homepage_name(player),
-		}
-		sfinv.contexts[name] = context
-	end
-	return context
+  local name = player:get_player_name()
+  local context = sfinv.contexts[name]
+  if not context then
+    context = {
+      page = sfinv.get_homepage_name(player),
+    }
+    sfinv.contexts[name] = context
+  end
+  return context
 end
 
 ---@param player mt.PlayerObjectRef
 ---@param context SFInvContext
 function sfinv.set_context(player, context)
-	sfinv.contexts[player:get_player_name()] = context
+  sfinv.contexts[player:get_player_name()] = context
 end
 
 -- (Re)builds page formspec and calls `set_inventory_formspec()`.
 ---@param player mt.PlayerObjectRef
 ---@param context SFInvContext|nil
 function sfinv.set_player_inventory_formspec(player, context)
-	local fs = sfinv.get_formspec(player, context or sfinv.get_or_create_context(player))
-	player:set_inventory_formspec(fs)
+  local fs =
+    sfinv.get_formspec(player, context or sfinv.get_or_create_context(player))
+  player:set_inventory_formspec(fs)
 end
 
 -- Changes the page.
 ---@param player mt.PlayerObjectRef
 ---@param pagename string
 function sfinv.set_page(player, pagename)
-	local context = sfinv.get_or_create_context(player)
-	local oldpage = sfinv.pages[context.page]
-	if oldpage and oldpage.on_leave then
-		oldpage:on_leave(player, context)
-	end
-	context.page = pagename
-	local page = sfinv.pages[pagename]
-	if page.on_enter then
-		page:on_enter(player, context)
-	end
-	sfinv.set_player_inventory_formspec(player, context)
+  local context = sfinv.get_or_create_context(player)
+  local oldpage = sfinv.pages[context.page]
+  if oldpage and oldpage.on_leave then oldpage:on_leave(player, context) end
+  context.page = pagename
+  local page = sfinv.pages[pagename]
+  if page.on_enter then page:on_enter(player, context) end
+  sfinv.set_player_inventory_formspec(player, context)
 end
 
 ---@param player mt.PlayerObjectRef
 ---@return string
 function sfinv.get_page(player)
-	local context = sfinv.contexts[player:get_player_name()] ---@type SFInvContext
-	return context and context.page or sfinv.get_homepage_name(player)
+  local context = sfinv.contexts[player:get_player_name()] ---@type SFInvContext
+  return context and context.page or sfinv.get_homepage_name(player)
 end
 
 minetest.register_on_joinplayer(function(player)
-	if sfinv.enabled then
-		sfinv.set_player_inventory_formspec(player)
-	end
+  if sfinv.enabled then sfinv.set_player_inventory_formspec(player) end
 end)
 
-minetest.register_on_leaveplayer(function(player)
-	sfinv.contexts[player:get_player_name()] = nil
-end)
+minetest.register_on_leaveplayer(
+  function(player) sfinv.contexts[player:get_player_name()] = nil end
+)
 
 minetest.register_on_player_receive_fields(function(player, formname, fields)
-	if formname ~= "" or not sfinv.enabled then
-		return false
-	end
-
-	-- Get Context
-	local name = player:get_player_name()
-	local context = sfinv.contexts[name]
-	if not context then
-		sfinv.set_player_inventory_formspec(player)
-		return false
-	end
-
-	-- Was a tab selected?
-	if fields.sfinv_nav_tabs and context.nav then
-		local tid = tonumber(fields.sfinv_nav_tabs)
-		if tid and tid > 0 then
-			local id = context.nav[tid]
-			local page = sfinv.pages[id]
-			if id and page then
-				sfinv.set_page(player, id)
-			end
-		end
-	else
-		-- Pass event to page
-		local page = sfinv.pages[context.page]
-		if page and page.on_player_receive_fields then
-			return page:on_player_receive_fields(player, context, fields)
-		end
-	end
+  if formname ~= "" or not sfinv.enabled then return false end
+
+  -- Get Context
+  local name = player:get_player_name()
+  local context = sfinv.contexts[name]
+  if not context then
+    sfinv.set_player_inventory_formspec(player)
+    return false
+  end
+
+  -- Was a tab selected?
+  if fields.sfinv_nav_tabs and context.nav then
+    local tid = tonumber(fields.sfinv_nav_tabs)
+    if tid and tid > 0 then
+      local id = context.nav[tid]
+      local page = sfinv.pages[id]
+      if id and page then sfinv.set_page(player, id) end
+    end
+  else
+    -- Pass event to page
+    local page = sfinv.pages[context.page]
+    if page and page.on_player_receive_fields then
+      return page:on_player_receive_fields(player, context, fields)
+    end
+  end
 end)

init.lua

@@ -1,24 +1,24 @@
 -- sfinv/init.lua
 
-dofile(minetest.get_modpath("sfinv") .. "/api.lua")
+dofile(minetest.get_modpath "sfinv" .. "/api.lua")
 
 -- Load support for MT game translation.
-local S = minetest.get_translator("sfinv")
+local S = minetest.get_translator "sfinv"
 
 sfinv.register_page("sfinv:crafting", {
-	title = S("Crafting"),
-	get = function(self, player, context)
-		return sfinv.make_formspec(
-			player,
-			context,
-			[[
+  title = S "Crafting",
+  get = function(self, player, context)
+    return sfinv.make_formspec(
+      player,
+      context,
+      [[
 				list[current_player;craft;1.75,0.5;3,3;]
 				list[current_player;craftpreview;5.75,1.5;1,1;]
 				image[4.75,1.5;1,1;sfinv_crafting_arrow.png]
 				listring[current_player;main]
 				listring[current_player;craft]
 			]],
-			true
-		)
-	end,
+      true
+    )
+  end,
 })