Weblaste error in the ISO code for Slovenian

Software Support
1

Hi @asmecher,

Just noticed there are issues with the locale setting in the Weblate: https://translate.pkp.sfu.ca/new-lang/plugins/hypothesis/ :

  • on the list of available locales to select to translate to there are 2 Slovenian
  • on the correct one the ISO code is sl_DI instead of sl_SI

Can you please fix it?

Best regards, Primož

PS: Hm, just noticed: the same on https://translate.pkp.sfu.ca/new-lang/plugins/backup-plugin/ and https://translate.pkp.sfu.ca/projects/plugins/coins-plugin/
Is that everywhere and I noticed only today?

2

Hi @primozs,

Hmm, sl_DI is confusing to me – I’m not sure why Weblate is suggesting this. Isn’t the full locale code for Slovenian sl_SI (e.g. per http://www.localeplanet.com/icu/sl-SI/index.html)?

Weblate sometimes uses the short form when creating a translation (sl in the case of Slovenian) and when I see that I correct it to sl_SI. OJS currently only supports the long form (though I’ll probably relax that in a future release to harmonize with Weblate).

As far as I’ve seen, the corrected sl_SI contents are indistinguishable in Weblate from the sl form it seems to use when creating translations.

Are you seeing problems with sl_SI?

Thanks,
Alec Smecher
Public Knowledge Project Team

3

Hi @asmecher,

Yes, the sl_SI is the correct code. So far I have always seen the sl_SI in the Weblate. Just the last repos, mentioned above, have wrong locals. Se yourself (only one of them):

51

31

I believe there shoudl be only the sl_SI option available. I don’t know how the Weblaste works. I would assume it coudl be a problem of a certain configuration of the repos.

Best regards, Primož

4

Hi @primozs,

I found a sl_DI typo in this commit: Standardize locale codes · pkp/textEditorExtras@1e5e766 · GitHub

I’ve corrected it, also in coins and backup. It seems that Weblate follows existing patterns for locale codes to some extent, as it pulled the sl_DI locale code from my typo. I’m hoping it’ll revert back now that this has been corrected. Please let me know what you see!

Thanks,
Alec Smecher
Public Knowledge Project Team

5

Hi @asmecher,

I can’t believe, strange. You have fixed coins and backup, where it states only Slovenian. While at hypothesis it is still Slovenian (sl_DI).

Best regards, Primož

6

Hi @primozs,

I’ve fixed that one too! Please let me know if you see any more.

Thanks,
Alec Smecher
Public Knowledge Project Team

7

Hi @asmecher,

Verified, OK now. Thank you.

Best regards, Primož