fix(i18n): fix regression in current locale

[ematipico]

Changes

Closes #9970

I reverted the change to use pathname instead for RouteData::route..route contains the actual name of the file e.g. /blog/[...locale, which is not what I needed.

So, I reverted the code to use RouteData::pathname first and then URL::pathname in the former isn't present. RouteData::pathname covers SSR cases.

This PR also fixes another issue where Astro.currentLocale wasn't returning the correct value in case a locale is configured using the path property

Testing

I added new test cases.

Docs

N/A

[changeset-bot]

🦋 Changeset detected

Latest commit: 3f0c6f1

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[bluwy]

Fix looks good to me. A bit worried about the perf implication though since it has 3 nested loops, but we could revisit them next time. Also, is the two changesets intentional or should there only be one?

[ematipico]

Also, is the two changesets intentional or should there only be one?

It's intentional, because the two bugs are unrelated (two different use cases), even though the function is the same.