Fallback to a variant if the language in general is not supported

For an example, here, zh is not supported, but zh_Hans and zh_Hant
are. If the user asks for zh, we should choose a variant for them
instead of fallbacking to default.

Some browsers (e.g. Firefox) does not allow users to customize
their language codes. For example, there is no zh-Hans, but only
zh, zh-CN, zh-TW, zh-HK, etc. This provides a workaround for
those users suffering from bad design decisions.
This commit is contained in:
Tusooa Zhu 2022-03-02 19:59:11 -05:00
parent 1a917cfeec
commit 8de573b047
No known key found for this signature in database
GPG key ID: 7B467EDE43A08224
3 changed files with 34 additions and 0 deletions

View file

@ -49,6 +49,20 @@ def supports_locale?(locale) do
|> Enum.member?(locale) |> Enum.member?(locale)
end end
def variant?(locale), do: String.contains?(locale, "_")
def supported_variants_of_locale(locale) do
cond do
variant?(locale) ->
[locale]
supports_locale?(locale) ->
[locale]
true ->
Gettext.known_locales(Pleroma.Web.Gettext)
|> Enum.filter(fn l -> String.starts_with?(l, locale <> "_") end)
end
end
def locale_or_default(locale) do def locale_or_default(locale) do
if supports_locale?(locale) do if supports_locale?(locale) do
locale locale

View file

@ -20,6 +20,12 @@ defp get_locale_from_header(conn) do
conn conn
|> extract_preferred_language() |> extract_preferred_language()
|> normalize_language_codes() |> normalize_language_codes()
|> first_supported()
end
defp first_supported(locales) do
locales
|> Enum.flat_map(&Pleroma.Web.Gettext.supported_variants_of_locale/1)
|> Enum.find(&supported_locale?/1) |> Enum.find(&supported_locale?/1)
end end

View file

@ -47,6 +47,20 @@ test "use supported locale with specifiers from `accept-language`" do
assert %{locale: "zh_Hans"} == conn.assigns assert %{locale: "zh_Hans"} == conn.assigns
end end
test "fallback to some variant of the language if the unqualified language is not supported" do
conn =
:get
|> conn("/cofe")
|> Conn.put_req_header(
"accept-language",
"zh;q=0.9, en;q=0.8, *;q=0.5"
)
|> SetLocalePlug.call([])
assert "zh_" <> _ = Gettext.get_locale()
assert %{locale: "zh_" <> _} = conn.assigns
end
test "use supported locale from cookie" do test "use supported locale from cookie" do
conn = conn =
:get :get