-
-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
Description
Derived from the bug #99. If I wanted to patch current I18n.pluralize behavior by overriding pluralize method, it wouldn't work. This is because i18n class has code duplication and it does not uses a single point of entry into pluralizer
Describe the solution
To update lines
Lines 383 to 389 in f49c86d
| translation = pluralize({ | |
| i18n: this, | |
| count: options.count || 0, | |
| scope: translation as unknown as string, | |
| options, | |
| baseScope: getFullScope(this, scope, options), | |
| }); |
to something like
translation = this.pluralize({
count: options.count || 0,
scope: translation as unknown as string,
options,
baseScope: getFullScope(this, scope, options),
});
in this case the overrides pluralize will be respected count based translations.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels