fix filtering

Liquipedia · Jan 10, 2025 · 54b33ce · 54b33ce
1 parent 87cddc9
commit 54b33ce
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 174 deletions.
diff --git a/.editorconfig b/.editorconfig
diff --git a/components/widget/tournaments/widget_tournament_label.lua b/components/widget/tournaments/widget_tournament_label.lua
@@ -28,7 +28,7 @@ function TournamentsTickerWidget:render()
 	if not tournament then
 		return
 	end
-	return HtmlWidgets.Li{children = HtmlWidgets.Div{
+	return HtmlWidgets.Div{
 		css = {
 			display = 'flex',
 			gap = '5px',
@@ -68,7 +68,7 @@ function TournamentsTickerWidget:render()
 				},
 			},
 		},
-	}}
+	}
 end
 
 return TournamentsTickerWidget
diff --git a/components/widget/tournaments/widget_tournaments_ticker_sublist.lua b/components/widget/tournaments/widget_tournaments_ticker_sublist.lua
@@ -31,7 +31,7 @@ function TournamentsTickerWidget:render()
 		return Array.reduce(self.props.filterGroups or {}, function(prev, filter)
 			return HtmlWidgets.Div{
 				attributes = {
-					['data-filter-group'] = 'filterbuttons-' .. filter,
+					['data-filter-group'] = 'filterbuttons-' .. string.lower(filter),
 					['data-filter-category'] = tournament[filter],
 					['data-curated'] = tournament.featured and '' or nil,
 				},
@@ -43,7 +43,7 @@ function TournamentsTickerWidget:render()
 	local list = HtmlWidgets.Ul{
 		classes = {'tournaments-list-type-list'},
 		children = Array.map(self.props.tournaments, function(tournament)
-			return createFilterWrapper(tournament, TournamentLabel{tournament = tournament})
+			return HtmlWidgets.Li{children = createFilterWrapper(tournament, TournamentLabel{tournament = tournament})}
 		end),
 	}
 

diff --git a/standard/i18n.lua b/standard/i18n.lua
@@ -6,21 +6,25 @@
 -- Please see https://github.com/Liquipedia/Lua-Modules to contribute
 --
 
+local FnUtil = require('Module:FnUtil')
 local Lua = require('Module:Lua')
 local String = require('Module:StringUtils')
 
 local I18nData = Lua.import('Module:I18n/Data', {loadData = true})
 local I18n = {}
 
-local LANGUAGE = mw.language.getContentLanguage():getCode()
+local getLanguage = FnUtil.memoize(function ()
+	return mw.language.getContentLanguage():getCode()
+end)
 
 ---Interpolates an i18n string with data
 ---TODO Add pluralization support (https://cldr.unicode.org/index/cldr-spec/plural-rules)
 ---@param key string
 ---@param data table<string, string|number>?
 ---@return string
 function I18n.translate(key, data)
-	local langMessages = I18nData[LANGUAGE] or I18nData.en
+	local language = getLanguage()
+	local langMessages = I18nData[language] or I18nData.en
 	local message = langMessages[key] or I18nData.en[key]
 	if not message then
 		return '⧼' .. key .. '⧽'