<!ENTITY % is not recognized by storage/dtd.py about translate

[okulev]

Created [attachment 24](http://bugs.locamotion.org/attachment.cgi?id=24)

Patch for skipping parameter entities

[davidfraser]

Thanks for pointing this out …
I just want to check that the PO file produced merges back correctly etc. Do you
have any pointers to the documentation on this ENTITY % parameter syntax?

[okulev]

Unfortunately, the attached patch is not solution because when generating .dtd,
the “<!ENTITY %” and “&…;” are not inserted back. The patch just ignores them
when parsing instead of turning them into kind of internal structure so that
they can be reconstructed.

[davidfraser]

Have just committed to CVS a patch that parses these properly and can read them
in and write them out.
This also works from within moz2po / po2moz since although the entities are not
marked any differently in the po file, the dtd in the template is used to
recreate them correctly.

Still need to make the %realBrandDTD; get retained on reading and writing

[davidfraser]

I have now committed a fix for handling %realBrandDTD; as well – this now
reliably handles the files global/brand.dtd and global/netError.dtd through the
whole translation process (as far as I know these are the only files containing
this syntax).
Marking Fixed – can you confirm as Verified?

[okulev]

I’ve tried new storage/dtd.py and the result looks great. And it’s more nice
with proper empty lines :-)