Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#3577 closed defect (bug) (fixed)

gettext.php breaks on poedit plural po files

Reported by: nbachiyski Owned by: ryan
Milestone: 2.1 Priority: high
Severity: normal Version: 2.1
Component: General Keywords: patch bg|has-patch i18n
Focuses: Cc:


poEdit doesn't put a semi-colon at the end of plural headers (even if explicitly told so). However the standard requires them and out gettext implementation blows with parse error (it uses eval) if a semi-colon is missing.

So I added one ;)

P.S. There is no problem to have more than one semi-colon at the end of a statement, so I skipped the sanity check if there actually *is* a semi-colon at the end of the header.

Attachments (1)

gettext-semicolon.diff (521 bytes) - added by nbachiyski 15 years ago.

Download all attachments as: .zip

Change History (3)

#1 @ryan
15 years ago

  • Owner changed from anonymous to ryan

#2 @ryan
15 years ago

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

(In [4736]) Fix plural form parsing. Props nbachiyski. fixes #3577

Note: See TracTickets for help on using tickets.