1423123 - [az] Verify plural rule for Azerbaijani (switch to rule #1)

Status: VERIFIED FIXED

(Mozilla Localizations :: az / Azerbaijani, enhancement)

Description

[Francesco Lodolo [:flod]]

Azerbaijani has 2 plural forms according to CLDR, like English
http://www.unicode.org/cldr/charts/dev/supplemental/language_plural_rules.html#az

But Firefox and Firefox for Android are set to use the plural form 0, which means no plural forms (like Chinese). In other words, the noun following a cardinal number never changes.

Comment 1

[Francesco Lodolo [:flod]]

@Emin
What's the correct plural form for Azerbaijani?

It's surprising that this didn't surface while using the browser. Considering for example this
https://transvision.mozfr.org/?recherche=tabs.closeWarningMultiple&repo=gecko_strings&sourcelocale=en-US&locale=az&search_type=strings_entities

It's the warning displayed when closing multiple tabs: the first form is empty, the second has a string. I'm pretty sure the dialog on your Firefox should be displaying an empty string.

Comment 2

[Emin Mastizada]

Hi Francesco,
So, there are 2 plural rules in Azerbaijani, and we use it in Firefox. Till know it was giving both versions in English (single version;plural version). We don't use plural rule when there is a number, so that structure helps us to decide when to use.
We haven't noticed that string in close window as when you close main window, mostly Firefox just closes without asking :)
Fixed that sentence by removing the semicolon, do we need to write it twice?
Plural rules for Azerbaijani is exactly same as in Turkish (and other Turkic languages), sometimes Selim writes only 1 version of the sentence, so I would like to learn how Firefox parses it (is it strict to have both versions for plural rules if both are same (when numbers are used) ?)

Comment 3

[Francesco Lodolo [:flod]]

(In reply to Emin Mastizada from comment #2)
> Hi Francesco,
> So, there are 2 plural rules in Azerbaijani, and we use it in Firefox. 

That's the point. You're writing 2 forms in the string, but you're NOT using them.

Azerbaijani uses rule #0 (1 form)
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_and_Plurals

If Azerbaijani has 2 plural forms, like CLDR, we should fix the plural rule and let the existing strings be. We'd only need to revert back the change you just made.

> Fixed that sentence by removing the semicolon, do we need to write it twice?

With the current rule, you only need it once. The point is that we should likely change the rule.

> Plural rules for Azerbaijani is exactly same as in Turkish (and other Turkic
> languages), sometimes Selim writes only 1 version of the sentence, so I
> would like to learn how Firefox parses it (is it strict to have both
> versions for plural rules if both are same (when numbers are used) ?)

There is another bug open for Turkish (bug 1423131), and I really think that we need to change it to use 2 forms as well.
We're going to use ICU/CLDR data in the future, which means you're going to use two forms anyway, since that's the correct form.

If your locale is set to have X forms, you need to have all of them. String is split using the semicolon, so

;Some text

Is 2 plural forms: the first is empty (singular in English), the second has a text.

At this point I strongly advise to change the plural rule to 1 (like English), revert the tabs closing string, and that will fix all the existing plural strings. Thoughts?

Comment 4

[Emin Mastizada]

Ok, reverted back the translation. Is there any other errors?
When the changes to plural rule will be available?

Comment 5

[Francesco Lodolo [:flod]]

I pushed the change to the rule
https://hg.mozilla.org/l10n-central/az/rev/2028313be93e8d56984297fe2878fdfeb5685c93

Tomorrow's Nightly will have the new rule applied. Can you give it a test and confirm if everything behaves as expected? At that point I will sign-off on Beta.

@pike
Can you update the reference in compare-locales too?

Comment 6

[Axel Hecht [:Pike]]

Updated compare-locales, https://hg.mozilla.org/l10n/compare-locales/rev/711c09ec362db54fc7e21b638cbb743203d6473e

Comment 7

[Francesco Lodolo [:flod]]

@Emin
Were you able to test this? Is everything looking as expected?

Comment 8

[Emin Mastizada]

Tested in latest build, message is now seen as expected. Thanks

Comment 9

[Francesco Lodolo [:flod]]

Thanks Emin, closing then.