Opened 6 years ago

Last modified 5 years ago

#29825 closed Bug

ngettext returns invalid result if msgstr is also a valid msgid in the same catalog — at Version 4

Reported by: Jeremy Moffitt Owned by: nobody
Component: Internationalization Version: 2.1
Severity: Normal Keywords: ngettext localization
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Tim Graham)

When ngettext is called with a msgid and that msgid has a msgstr that is also a valid msgid in the same catalog, the return value is not the msgstr, but instead is a single character from the msgstr. The problem seems to be that the msgstr is passed back into ngettext and since it has a valid value in the catalog, the code breaks into the else block and returns the 0 index character...

    django.ngettext = function(singular, plural, count) {
      var value = django.catalog[singular];
      if (typeof(value) == 'undefined') {
        return (count == 1) ? singular : plural;
      } else {
        return value[django.pluralidx(count)];
      }
    };

The example in OpenStack Horizon (see link below) is that the French bundle contains the following:

msgid "Image"
msgstr "Image"

This should return "Image" ... instead it returns "I" ... the result of django.catalog["Image"] being "Image" , which then breaks into the else block of the following if statement, resulting in a return value of "Image"[0] ... or the capital letter "I". For languages where the msgstr does not match a valid key in the file, this problem does not occur.

see also openstack bug: https://bugs.launchpad.net/horizon/+bug/1778189

Change History (4)

comment:1 by Jeremy Moffitt, 6 years ago

Description: modified (diff)

comment:2 by Claude Paroz, 6 years ago

Component: UncategorizedInternationalization
Type: UncategorizedBug

Jeremy, would you be able to write a failing test? (probably in i18n.tests.TranslationTests)?

in reply to:  2 comment:3 by Jeremy Moffitt, 6 years ago

Replying to Claude Paroz:

Jeremy, would you be able to write a failing test? (probably in i18n.tests.TranslationTests)?

I can try, I'm not particularly familiar with the Django test suite. I ran into this debugging the Horizon bug in OpenStack and have no experience using Django outside of Horizon.

comment:4 by Tim Graham, 6 years ago

Description: modified (diff)
Triage Stage: UnreviewedAccepted

Tentatively accepting for investigation.

Note: See TracTickets for help on using tickets.
Back to Top