feat(match2): overall stats tab for leagueoflegends matchpage

lua/wikis/commons/MatchPage/Base.lua

@@ -286,8 +286,8 @@ function BaseMatchPage:renderGames()
 	local hasOverallStats = Logic.isNotEmpty(overallStats)
 
 	if hasOverallStats then
-		tabs['name1'] = 'Overall Statistics'
-		tabs['content1'] = overallStats
+		tabs.name1 = 'Overall Statistics'
+		tabs.content1 = overallStats
 	end
 
 	Array.forEach(games, function(game, idx)

lua/wikis/commons/Operator.lua

@@ -19,6 +19,19 @@ function Operator.add(a, b)
 	return a + b
 end
 
+---A `nil`-safe version of `Operator.add`
+---@param a number?
+---@param b number?
+---@return number?
+function Operator.nilSafeAdd(a, b)
+	if not a then
+		return b
+	elseif not b then
+		return a
+	end
+	return a + b
+end
+
 ---Uses the __sub metamethod (a - b)
 ---@param a number
 ---@param b number

lua/wikis/commons/Widget/Match/Page/StatsList.lua

@@ -36,9 +36,14 @@ function MatchPageStatsList:render()
 	if Logic.isEmpty(self.props.data) then return end
 	return Div{
 		classes = {'match-bm-team-stats-list'},
-		children = Array.map(self.props.data, function (dataElement)
-			return self:_renderStat(dataElement)
-		end)
+		children = Array.map(
+			Array.filter(self.props.data, function (element)
+				return element.team1Value ~= nil or element.team2Value ~= nil
+			end),
+			function (dataElement)
+				return self:_renderStat(dataElement)
+			end
+		)
 	}
 end
 

lua/wikis/leagueoflegends/MatchGroup/Input/Custom.lua

@@ -10,23 +10,25 @@ local Lua = require('Module:Lua')
 local Array = Lua.import('Module:Array')
 local FnUtil = Lua.import('Module:FnUtil')
 local HeroNames = Lua.import('Module:ChampionNames', {loadData = true})
+local Logic = Lua.import('Module:Logic')
+local Operator = Lua.import('Module:Operator')
 local String = Lua.import('Module:StringUtils')
 local Table = Lua.import('Module:Table')
 
 local MatchGroupInputUtil = Lua.import('Module:MatchGroup/Input/Util')
 local MatchGroupUtil = Lua.import('Module:MatchGroup/Util/Custom')
 
 local CustomMatchGroupInput = {}
-local MatchFunctions = {}
-local MapFunctions = {}
-
-MatchFunctions.OPPONENT_CONFIG = {
-	resolveRedirect = true,
-	pagifyTeamNames = false,
-	maxNumPlayers = 15,
+local MatchFunctions = {
+	OPPONENT_CONFIG = {
+		resolveRedirect = true,
+		pagifyTeamNames = false,
+		maxNumPlayers = 15,
+	},
+	DEFAULT_MODE = 'team',
+	getBestOf = MatchGroupInputUtil.getBestOf
 }
-MatchFunctions.DEFAULT_MODE = 'team'
-MatchFunctions.getBestOf = MatchGroupInputUtil.getBestOf
+local MapFunctions = {}
 
 ---@class LeagueOfLegendsMapParserInterface
 ---@field getMap fun(mapInput: table): table
@@ -37,6 +39,7 @@ MatchFunctions.getBestOf = MatchGroupInputUtil.getBestOf
 ---@field getHeroBans fun(map: table, opponentIndex: integer): string[]?
 ---@field getParticipants fun(map: table, opponentIndex: integer): table[]?
 ---@field getVetoPhase fun(map: table): table?
+---@field extendMapOpponent? fun(map: table, opponentIndex: integer): table
 
 ---@param match table
 ---@param options? {isMatchPage: boolean?}
@@ -60,11 +63,74 @@ function CustomMatchGroupInput.processMatch(match, options)
 		MapParser = Lua.import('Module:MatchGroup/Input/Custom/Normal')
 	end
 
-	return MatchGroupInputUtil.standardProcessMatch(match, MatchFunctions, nil, MapParser)
+	local processedMatch = MatchGroupInputUtil.standardProcessMatch(match, MatchFunctions, nil, MapParser)
+
+	if options.isMatchPage then
+		CustomMatchGroupInput.aggregateStats(processedMatch)
+	end
+	return processedMatch
+end
+
+---@param match {opponents: MGIParsedOpponent[], games: table[]}
+function CustomMatchGroupInput.aggregateStats(match)
+	Array.forEach(match.opponents, function (opponent, opponentIndex)
+		---@param name string
+		---@return number?
+		local function aggregateStats(name)
+			return Array.reduce(
+				Array.map(match.games, function (game)
+					return (game.opponents[opponentIndex].stats or {})[name]
+				end),
+				Operator.nilSafeAdd
+			)
+		end
+		opponent.extradata = Table.merge(opponent.extradata, {
+			kills = aggregateStats('kills'),
+			deaths = aggregateStats('deaths'),
+			assists = aggregateStats('assists'),
+			towers = aggregateStats('towers'),
+			inhibitors = aggregateStats('inhibitors'),
+			dragons = aggregateStats('dragons'),
+			atakhans = aggregateStats('atakhans'),
+			heralds = aggregateStats('heralds'),
+			barons = aggregateStats('barons')
+		})
+		Array.forEach(opponent.match2players, function (player, playerIndex)
+			player.extradata = {characters = {}}
+			Array.forEach(
+				Array.filter(match.games, function (game)
+					return game.status ~= MatchGroupInputUtil.MATCH_STATUS.NOT_PLAYED
+				end),
+				function (game)
+					local gamePlayerData = game.opponents[opponentIndex].players[playerIndex]
+					if Logic.isEmpty(gamePlayerData) then
+						return
+					end
+					local parsedGameLength = Array.map(
+						Array.parseCommaSeparatedString(game.length --[[@as string]], ':'), function (element)
+							---Directly using tonumber as arg to Array.map causes base out of range error
+							return tonumber(element)
+						end
+					)
+					local gameLength = (parsedGameLength[1] or 0) * 60 + (parsedGameLength[2] or 0)
+					player.extradata.role = player.extradata.role or gamePlayerData.role
+					player.extradata.characters = Array.extend(player.extradata.characters, gamePlayerData.character)
+					player.extradata.kills = Operator.nilSafeAdd(player.extradata.kills, gamePlayerData.kills)
+					player.extradata.deaths = Operator.nilSafeAdd(player.extradata.deaths, gamePlayerData.deaths)
+					player.extradata.assists = Operator.nilSafeAdd(player.extradata.assists, gamePlayerData.assists)
+					player.extradata.damage = Operator.nilSafeAdd(player.extradata.damage, gamePlayerData.damagedone)
+					player.extradata.creepscore = Operator.nilSafeAdd(player.extradata.creepscore, gamePlayerData.creepscore)
+					player.extradata.gold = Operator.nilSafeAdd(player.extradata.gold, gamePlayerData.gold)
+					player.extradata.gameLength = Operator.nilSafeAdd(player.extradata.gameLength, gameLength)
+				end
+			)
+			player.extradata.characters = Logic.nilIfEmpty(player.extradata.characters)
+		end)
+	end)
 end
 
 ---@param match table
----@param opponents table[]
+---@param opponents MGIParsedOpponent[]
 ---@param MapParser LeagueOfLegendsMapParserInterface
 ---@return table[]
 function MatchFunctions.extractMaps(match, opponents, MapParser)
@@ -75,6 +141,7 @@ function MatchFunctions.extractMaps(match, opponents, MapParser)
 		getMap = MapParser.getMap,
 		getLength = MapParser.getLength,
 		getPlayersOfMapOpponent = FnUtil.curry(MapFunctions.getPlayersOfMapOpponent, MapParser),
+		extendMapOpponent = MapParser.extendMapOpponent
 	}
 	local maps = MatchGroupInputUtil.standardProcessMaps(match, opponents, mapParserWrapper)
 
@@ -96,7 +163,7 @@ end
 
 ---@param match table
 ---@param games table[]
----@param opponents table[]
+---@param opponents MGIParsedOpponent[]
 ---@return table
 function MatchFunctions.getExtraData(match, games, opponents)
 	return {
@@ -107,7 +174,7 @@ end
 ---@param MapParser LeagueOfLegendsMapParserInterface
 ---@param match table
 ---@param map table
----@param opponents table[]
+---@param opponents MGIParsedOpponent[]
 ---@return table
 function MapFunctions.getExtraData(MapParser, match, map, opponents)
 	local extraData = {}
@@ -146,7 +213,7 @@ end
 
 ---@param MapParser LeagueOfLegendsMapParserInterface
 ---@param map table
----@param opponent table
+---@param opponent MGIParsedOpponent
 ---@param opponentIndex integer
 ---@return table[]
 function MapFunctions.getPlayersOfMapOpponent(MapParser, map, opponent, opponentIndex)

lua/wikis/leagueoflegends/MatchGroup/Input/Custom/MatchPage.lua

@@ -121,4 +121,34 @@ function CustomMatchGroupInputMatchPage.getObjectives(map, opponentIndex)
 	}
 end
 
+function CustomMatchGroupInputMatchPage.extendMapOpponent(map, opponentIndex)
+	local participants = CustomMatchGroupInputMatchPage.getParticipants(map, opponentIndex)
+
+	if Logic.isEmpty(participants) then
+		return {picks = {}, stats = {}}
+	end
+	---@cast participants -nil
+
+	---@param arr table[]
+	---@param item string
+	---@return number?
+	local function sumItem(arr, item)
+		return Array.reduce(Array.map(arr, Operator.property(item)), Operator.nilSafeAdd, 0)
+	end
+
+	return {
+		side = CustomMatchGroupInputMatchPage.getSide(map, opponentIndex),
+		picks = Array.map(participants, Operator.property('character')),
+		stats = Table.merge(
+			{
+				gold = sumItem(participants, 'gold'),
+				kills = sumItem(participants, 'kills'),
+				deaths = sumItem(participants, 'deaths'),
+				assists = sumItem(participants, 'assists')
+			},
+			CustomMatchGroupInputMatchPage.getObjectives(map, opponentIndex)
+		)
+	}
+end
+
 return CustomMatchGroupInputMatchPage