Ticket #3577 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

gettext.php breaks on poedit plural po files

Reported by: nbachiyski Assigned to: ryan
Priority: high Milestone: 2.1
Component: General Version: 2.1
Severity: normal Keywords: patch bg|has-patch i18n
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

gettext-semicolon.diff (0.5 kB) - added by nbachiyski on 01/14/07 22:43:12.

Change History

01/14/07 22:43:12 changed by nbachiyski

  • attachment gettext-semicolon.diff added.

01/15/07 22:59:40 changed by ryan

  • owner changed from anonymous to ryan.

01/16/07 01:37:50 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

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