search

PerseusDL / canonical

This will be the base repo for all text and annotation data published in the PDL
16 stars 17 forks source link

tlg0023.tlg001.perseus-grc1.xml use unregistered entities #79

Closed PonteIneptique closed 9 years ago

PonteIneptique commented 9 years ago

In /CTS_XML_TEI/perseus/greekLit/tlg0023/tlg001/tlg0023.tlg001.perseus-grc1.xml, when adding the document to latest BaseX, I get the following error

"perseus-canonical/CTS_XML_TEI/perseus/greekLit/tlg0023/tlg001/tlg0023.tlg001.perseus-grc1.xml" (Line 161): The entity "lpar" was referenced, but not declared.

lcerrato commented 9 years ago

( ( ) ) probably unnecessary in Unicode version? required for correct beta code rendering

PonteIneptique commented 9 years ago

Well I was actually starting to look into what was lpar... Should I add to my actual PR a replacement of lpar and rpar by ( and ) or should I leave it to you to add thos entities if Betacode has to be supported ?

lcerrato commented 9 years ago

Is beta code intended to be supported? I don't think that is the case.

I'm not aware of the limitations of ( and ) with the new Unicode texts but I would not think they needed to be entities in UTF-8.

PonteIneptique commented 9 years ago

Well as it is not my call, I won't add it to my general PR of formatting issue and see if someone wants do it.

balmas commented 9 years ago

I would like to get rid of the old DTD entities as much as possible in the unicode versions.

lcerrato commented 9 years ago

@srdee we can probably close this