Open benwbrum opened 6 years ago
In addition to these:
revisionList
The list of edits we produce in the revisionList is too long to be meaningful -- remove it entirely.
Changes to add non-people/place subjects to the teiHeader
moved those subjects out of personList
or placeList
and into taxonomy
. This should be reversed.
Regarding the revision list: I do like it, but it is indeed very long – mostly because even when importing a work from IIIF or PDF, each page 'import' is a change.
Another small issue I encountered is that FromThePage.com
is hardcoded in the subelements of <editionStmt>
: <resp>Initial upload of this work's facsimile images and metadata to FromThePage.com for editing</resp>
is exported from our local FromThePage.
The facs
issue is fixed by #3388
Corresp is covered by #4208
Several problems exist in the TEI-XML export format; most of which are too small to warrant their own issue.
No URL in
facs
The
pb
element at the beginning of each page contains afacs
attribute that is supposed to point to a location of the page facsimile. Currently it is a relative URL instead of something usable by systems we export to:All paragraphs have
corresp
Bilingual texts are supposed to link paragraphs using a
corresp
attribute. We do this correctly for works which support translation, but also generate acorresp
attribute onp
elements when we don't support translation, leaving a dangling reference:TEI is not correctly formatted.
Simply reading the rendered TEI into Nokogiri and outputting it again as the rendering process would fix the problems with indentation resulting from sections of XML generated by views and helper functions.
Subjects within section headers are not linked
If a subject is marked up within a section header, the generated TEI seems to omit the whole
rs
tag replacement we use elsewhere: