locale_get_plural call in PluralTranslatableString is wrong

Created on 1 February 2016, about 9 years ago
Updated 17 February 2023, about 2 years ago

Follow-up to #2573975: function_exists check in PluralTranslatableString is wrong β†’ and #2571375: Remove TranslationManager dependency from LanguageManager β†’

Problem/Motivation

A core class shouldn't refer to code provided by a module. We should somehow be able to this and reverse the dependencies so core has no knowledge of the locale module.

Steps to reproduce

  1. Enabled Language and Interface Translation
  2. Added Romanian
  3. Found some text that uses plurals: Module %name has been enabled.
  4. Changed the plurals for these for Romanian
  5. Enabled some modules (1 module, 2 modules, and 3 modules)
  6. The messages were as expected (see screenshots below) (though I'm not sure what the 2. plural is for in Romanian)

Taken from #11.

Proposed resolution

tbd

Remaining tasks

User interface changes

tbd. Should be none.

API changes

tbd. Maybe.

Data model changes

tbd. Should be none.

πŸ› Bug report
Status

Needs work

Version

10.1 ✨

Component
LocaleΒ  β†’

Last updated 4 days ago

Created by

πŸ‡¬πŸ‡§United Kingdom alexpott πŸ‡ͺπŸ‡ΊπŸŒ

Live updates comments and jobs are added and updated live.
  • D8MI

    (Drupal 8 Multilingual Initiative) is the tag used by the multilingual initiative to mark core issues (and some contributed module issues). For versions other than Drupal 8, use the i18n (Internationalization) tag on issues which involve or affect multilingual / multinational support. That is preferred over Translation.

  • Needs manual testing

    The change/bugfix cannot be fully demonstrated by automated testing, and thus requires manual testing in a variety of environments.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

  • Needs change record

    A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024