WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 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:

Description

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 7 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 ryan7 years ago

  • Owner changed from anonymous to ryan

comment:2 ryan7 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.