adityatelange / hugo-PaperMod

A fast, clean, responsive Hugo theme.
https://adityatelange.github.io/hugo-PaperMod/
MIT License
10.26k stars 2.72k forks source link

:bug: Fix menu localization: site.Menus URLs are already localized #1623

Open npellegrin opened 1 month ago

npellegrin commented 1 month ago

What does this PR change? What problem does it solve?

It appears that a website having a custom base URL generate invalid menu links using this template. The reason is a computation of localization using absLangURL over site.Menus -> .URL items.

In recent versions of Hugo, the site.Menus items are already localized. You can see in the official example here, there is no absLangURL declared after .URL: https://gohugo.io/templates/menu/#example

absLangURL should be used to convert a relative URL to a localized URL in case the URL is not localized yet. Calling a already-localized URL with absLangURL will append the language code twice. The problem may be not visible without declaring a custom baseURL because Hugo seems to automatically strip duplicated localization codes for ease.

Was the change discussed in an issue or in the Discussions before?

I have not seen any issue about that, feel free to point it out if I missed it.

PR Checklist

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud