[#28941] - Pluralization fix
- Closed
- 5 May 2020
- Medium
- Build: staging
- # 28941
- Diff
- saharin88:pluralization
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Not taken into account 21, 31, etc.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Language & Strings |
No, it's not related to Crowdin and it's also not needed in the english strings.
The variable can be used in ukranian (or any language) regardless if it is present or not in the english string.
No, it's not related to Crowdin and it's also not needed in the english strings.
The variable can be used in ukranian (or any language) regardless if it is present or not in the english string.
Yes, I was wrong, everything is right there (method return MORE suffix if 21, 31, etc).
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-05 15:46:34 |
| Closed_By | ⇒ | saharin88 | |
| Labels |
Added:
?
?
|
||
Actually, you have to adjust the getPluralSuffixes function in your language pack to something like this:
if ($count % 10 == 1)
{
return array('1');
}
.... more cases here
Ukranian has some quite complex plural rules as it looks (see https://unicode-org.github.io/cldr-staging/charts/37/supplemental/language_plural_rules.html#uk)
As of 3.9.18 this is the localise.php for the Ukrainian pack
public static function getPluralSuffixes($count)
{
if($count == 0)
{
$return = [ '0' ];
}
else
{
$return = [ $count % 10 == 1 && $count % 100 != 11 ? '1' : ($count % 10 >= 2 && $count % 10 <= 4 && ($count % 100 < 10 || $count % 100 >= 20) ? '2' : 'MORE') ];
}
return $return;
}
@Bakual is this a crowdin thing?
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28941.