Code

#20311 closed Bug (fixed)

Avoid double Plural-Forms entry in translation files

Reported by: naktinis@… Owned by: nobody
Component: Internationalization Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Since xgettext already generates plural form entry, we have to make sure we remove the template before adding the correct plural form.

Without this fix makemessages command would generate:
...
"Plural-Forms: nplurals=INTEGER; plural=EXPRESSION;\n"
"Plural-Forms: nplurals=1; plural=0\n"
...

Attachments (1)

patch (1003 bytes) - added by naktinis@… 15 months ago.
Patch

Download all attachments as: .zip

Change History (4)

Changed 15 months ago by naktinis@…

Patch

comment:1 Changed 14 months ago by claudep

  • Needs documentation unset
  • Needs tests set
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted
  • Type changed from Uncategorized to Bug

comment:2 Changed 14 months ago by claudep

I guess this was unnoticed because it happens only if the first processed file contains plural strings.

comment:3 Changed 13 months ago by Ramiro Morales <cramm0@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 1559f84d8b5c2d4d088a27fc0edf02f01a1d65f0:

Fixed #20311 -- Make sure makemessages doesn't create duplicate Plural-Forms .po file headers.

Thanks naktinis for the report and initial patch.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.