@oasys Nice Catch !

What if we use [LIMIT] from https://gohugo.io/functions/replace/ ? Does it have same effect

like

-{{- $lang_url := replace $url ( printf "%s/" .Lang) "/" }}
+{{- $lang_url := replace $url ( printf "%s/" .Lang) "/" 1 }}

I will test later, but I suspect that will only fix when running a multi language config. A single language config will probably still have problems (as in my example, there will be no en/ to strip. )

I will test later, but I suspect that will only fix when running a multi language config. A single language config will probably still have problems (as in my example, there will be no en/ to strip. )

True

How about this? I assume multilingual config always has the lang prefix at the beginning of the URI. Using replaceRE anchors the lang match to the beginning (^) and the (/). I think this should work for most cases -- the only one I can think of that won't work is having a "section" name that is the same as a configured language.

{{- $lang_url := replaceRE ( printf "^%s/" .Lang) "" $url 1 }}

This will do the same, but might be more readable:

 {{- $lang_url := strings.TrimPrefix ( printf "%s/" .Lang) $url }}

If you prefer this change instead of the replaceRE one, let me know and I'll update the PR.

This will do the same, but might be more readable:

 {{- $lang_url := strings.TrimPrefix ( printf "%s/" .Lang) $url }}

If you prefer this change instead of the replaceRE one, let me know and I'll update the PR.

I agree, sure we can do this way

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

I agree, sure we can do this way

Done. Thanks!