unify the both inventories implementation unified_inventory supports now private skins too

1 files changed, 29 insertions, 71 deletions

diff --git a/unified_inventory_page.lua b/unified_inventory_page.lua
index 5a581d5..332e3cb 100644
--- a/unified_inventory_page.lua
+++ b/unified_inventory_page.lua
@@ -1,14 +1,10 @@
 local S = skins.S
 
-local dropdown_values = {}
-local skins_reftab = {}
-local skins_list = skins.get_skinlist_for_player() --public only
 unified_inventory.register_page("skins", {
 	get_formspec = function(player)
 		local skin = skins.get_player_skin(player)
-		local page = skin:get_meta("inv_page") or 1
 		local formspec = "background[0.06,0.99;7.92,7.52;ui_misc_form.png]"..skins.get_skin_info_formspec(skin)..
-				"button[.75,3;6.5,.5;skins_page$"..page..";"..S("Change").."]"
+				"button[.75,3;6.5,.5;skins_page;"..S("Change").."]"
 		return {formspec=formspec}
 	end,
 })
@@ -18,85 +14,47 @@ unified_inventory.register_button("skins", {
 	image = "skins_button.png",
 })
 
--- Create all of the skin-picker pages.
-local total_pages = 1
-for i, skin in ipairs(skins_list) do
-	local page = math.floor((i-1) / 16)+1
-	skin:set_meta("inv_page", page)
-	skin:set_meta("inv_page_index", (i-1)%16+1)
-	total_pages = page
+local function get_formspec(player)
+	-- unified inventory is stateless, but skins pager needs some context usage to be more flexible
+	local context = minetest.deserialize(player:get_attribute('skinsdb_unified_inventory_context')) or {}
+	context = skins.rebuild_formspec_context(player, context)
+	local formspec = "background[0.06,0.99;7.92,7.52;ui_misc_form.png]"..
+			skins.get_skin_selection_formspec(context, -0.2)
+	player:set_attribute('skinsdb_unified_inventory_context', minetest.serialize(context))
+	return formspec
 end
 
-for page=1, total_pages do
-	local formspec = "background[0.06,0.99;7.92,7.52;ui_misc_form.png]"
-	for i = (page-1)*16+1, page*16 do
-		local skin = skins_list[i]
-		if not skin then
-			break
-		end
-
-		local index_p = skin:get_meta("inv_page_index")
-		local x = (index_p-1) % 8
-		local y
-		if index_p > 8 then
-			y = 1.8
-		else
-			y = -0.1
-		end
-		formspec = (formspec.."image_button["..x..","..y..";1,2;"..
-			skin:get_preview()..";skins_set$"..i..";]"..
-			"tooltip[skins_set$"..i..";"..minetest.formspec_escape(skin:get_meta_string("name")).."]")
-	end
-	if total_pages > 1 then
-		local page_prev = page - 1
-		local page_next = page + 1
-		if page_prev < 1 then
-			page_prev = total_pages
-		end
-		if page_next > total_pages then
-			page_next = 1
-		end
-		local page_list = ""
-		dropdown_values = {}
-		for pg=1, total_pages do
-			local pagename = S("Page").." "..pg.."/"..total_pages
-			dropdown_values[pagename] = pg
-			if pg > 1 then page_list = page_list.."," end
-			page_list = page_list..pagename
-		end
-		formspec = (formspec
-			.."button[0,3.8;1,.5;skins_page$"..page_prev..";<<]"
-			.."dropdown[1,3.68;6.5,.5;skins_selpg;"..page_list..";"..page.."]"
-			.."button[7,3.8;1,.5;skins_page$"..page_next..";>>]")
+unified_inventory.register_page("skins_page", {
+	get_formspec = function(player)
+		return {formspec=get_formspec(player)}
 	end
-	unified_inventory.register_page("skins_page$"..(page), {
-		get_formspec = function(player)
-			return {formspec=formspec}
-		end
-	})
-end
-
+})
 
 -- click button handlers
 minetest.register_on_player_receive_fields(function(player, formname, fields)
 	if fields.skins then
+		player:set_attribute('skinsdb_unified_inventory_context',"") --reset context
 		unified_inventory.set_inventory_formspec(player, "craft")
 		return
 	end
+
+	local context -- read context only if skins related action
 	for field, _ in pairs(fields) do
-		local current = string.split(field, "$", 2)
-		if current[1] == "skins_set" then
-			skins.set_player_skin(player, skins_list[tonumber(current[2])])
-			unified_inventory.set_inventory_formspec(player, "skins")
-			return
-		elseif current[1] == "skins_page" then
-			unified_inventory.set_inventory_formspec(player, "skins_page$"..current[2])
-			return
+		if field:sub(1,5) == "skins" then
+			context = minetest.deserialize(player:get_attribute('skinsdb_unified_inventory_context')) or {}
+			break
 		end
 	end
-	if fields.skins_selpg then
-		local page = dropdown_values[fields.skins_selpg]
-		unified_inventory.set_inventory_formspec(player, "skins_page$"..(page))
+	if not context then
 		return
 	end
+
+	local action = skins.on_skin_selection_receive_fields(player, context, fields)
+	if action == 'set' then
+		player:set_attribute('skinsdb_unified_inventory_context',"") --reset context
+		unified_inventory.set_inventory_formspec(player, "skins")
+	elseif action == 'page' then
+		player:set_attribute('skinsdb_unified_inventory_context', minetest.serialize(context))
+		unified_inventory.set_inventory_formspec(player, "skins_page")
+	end
 end)