Upayavira wrote:
Fine by me too!
Ok, I (majorly) reworked both LocaleAction and I18nMatcher. I also noticed that original I18nMatcher was not properly handling locales with countries and variants (AFAIU, constructor new Locale(locale, "") as first argument takes language and not locale string).
Also, I removed "language-attribute", "country-attribute", "variant-attribute" configuration elements from LocaleAction - IMHO they are clear FS - what's the point in customizing sitemap variable names. Language sitemap variable renamed to "language", same as in I18nMatcher.
Please take a look, check that everything is ok :-)
I was thinking: may be I18nMatcher should be renamed to LocaleMatcher?
Vadim
