Closed
Bug 1423144
Opened 7 years ago
Closed 7 years ago
[fa] Verify plural rule for Persian (switch to rule #2)
Categories
(Mozilla Localizations :: fa / Persian, enhancement)
Mozilla Localizations
fa / Persian
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: flod, Assigned: Arash-M)
References
Details
Persian has 2 plural forms according to CLDR
http://www.unicode.org/cldr/charts/dev/supplemental/language_plural_rules.html#fa
But Firefox and Firefox for Android are set to use the plural form 0, which means no plural forms (like Chinese). Basically, the noun never changes depending when it follows a cardinal number.
Translate House has 2 forms like CLDR
http://docs.translatehouse.org/projects/localization-guide/en/latest/l10n/pluralforms.html
In case, several strings need to be reviewed and updated after correcting the plural form.
Assignee | ||
Comment 2•7 years ago
|
||
Persian plural forms are depending on context. The noun does not change when the sentence is plural but the verb does. So when the sentence does not have a verb, we don't need two forms, but when it does we need them.
For instance, there is no difference in translating "2 hours ago" and "1 hour ago" in Persian, but "1 notification is available" and "2 notifications are available" are different when translating.
Generally I set the plural forms to 2, and then if the sentence does not need two forms, I just copy the translation over to both.
Flags: needinfo?(mousavi.arash)
Reporter | ||
Comment 3•7 years ago
|
||
The problem is that Firefox is set to only use one of those two (the first one).
https://transvision.mozfr.org/?recherche=addonConfirmInstall.message&repo=gecko_strings&sourcelocale=en-US&locale=fa&search_type=strings_entities
This will be used
این سایت تمایل به نصب افزونه بر روی #1 دارد:
This will never be used
این سایت تمایل به نصب #2 افزونه بر روی #1 دارد:
At this point I think we should change the rule in Firefox. If we follow CLDR, we need the same rule as French:
0 is singular
1 is singular
any other number is plural
Does that make sense?
Assignee | ||
Comment 4•7 years ago
|
||
There's still no difference in singular and plural translation of the string you mentioned in Persian, but this one needs different translation for plural form: https://transvision.mozfr.org/?sourcelocale=en-US&locale=fa&repo=gecko_strings&search_type=entities&recherche=browser/chrome/browser/browser.properties:addonsInstalled&entire_string=entire_string
It makes sense to change the number of plural forms though.
Reporter | ||
Comment 5•7 years ago
|
||
OK, can you confirm that the "French" rule is correct? Following your example above:
In English:
0 notifications are available
1 notification is available
2 notifications are available
etc.
In Persian, the first two would be identical (0 identical to 1), then 2 and all other numbers with a different form?
Assignee | ||
Comment 6•7 years ago
|
||
Yes, that's correct.
Reporter | ||
Comment 7•7 years ago
|
||
I switched to rule #2. I'll check in about 10 minutes if there are still errors related to the number of plural forms.
Assignee: nobody → francesco.lodolo
Summary: [fa] Verify plural rule for Persian → [fa] Verify plural rule for Persian (switch to rule #2)
Reporter | ||
Comment 8•7 years ago
|
||
Landing
https://hg.mozilla.org/l10n-central/fa/rev/e1429c702b99
Looks like there are a few strings to fix now:
- details.experiment.time.daysPassed has 1 plural forms (requested: 2)
- details.experiment.time.daysRemaining has 1 plural forms (requested: 2)
- experiment.time.daysPassed has 1 plural forms (requested: 2)
- experiment.time.daysRemaining has 1 plural forms (requested: 2)
- newtabpopup.open has 1 plural forms (requested: 2)
- showAllSearchResults has 1 plural forms (requested: 2)
- tabs.closeWarningMultiple has 1 plural forms (requested: 2)
- unnamedTabsArrivingNotificationNoDevice.body has 1 plural forms (requested: 2)
- webextPerms.hostDescription.tooManySites has 1 plural forms (requested: 2)
- webextPerms.hostDescription.tooManyWildcards has 1 plural forms (requested: 2)
In case you want to double check all the strings with plurals
https://pontoon.mozilla.org/fa/firefox/all-resources/?status=translated&search=localization_and_plurals&string=159945
https://pontoon.mozilla.org/fa/firefox-for-android/all-resources/?status=translated&search=localization_and_plurals&string=159945
Reporter | ||
Updated•7 years ago
|
Assignee: francesco.lodolo → mousavi.arash
Comment 9•7 years ago
|
||
Fixed in compare-locales, https://hg.mozilla.org/l10n/compare-locales/rev/23e1103e2dd52c40f9267f8e5dada515916e7656.
Assignee | ||
Comment 10•7 years ago
|
||
Reviewed and fixed all plural forms. Thank you very much.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•