Do not convert extent sub-elements to HTML definition list on EAD import
|Assignee:||Dan Gillean||% Done:|
|Target version:||Release 2.2.0|
|Google Code Legacy ID:||Tested version:||2.1|
On roundtrip, the attached EAD file changed the extent tag to:
<dl><dt>Extent</dt><dd>Stuff & stuff & more things like * $ # @ ^ !</dd></dl>
The <dl>,<dt> and <dd> tags were not present on export from AtoM, so it's unclear what created them on import.
#2 Updated by Dan Gillean over 7 years ago
The definition list tags added are a result of a hacky fix introduced with #5212. With XSS escaping coming to AtoM in 2.2 (#7647) this solution will no longer work. We have recently had an internal meeting to discuss this - once I have internal approval, I will post the notes of the meeting here, and we can either use this ticket to implement the solution, or file a new ticket and close this one.
Once we have our 2.2 fix in place, I think we should close any of the above issues still open. We can always open new issues in the future if needed, but much of the discussion in those tickets will no longer apply after we implement a fix for this.
#3 Updated by Dan Gillean over 7 years ago
Attaching notes from a meeting held to outline the issue, brainstorm possible solutions and challenges, and decide on an approach.
#9 Updated by Dan Gillean about 7 years ago
- Status changed from QA/Review to Verified
Amazing! The great part is, if you export DC XML, the physdesc nested tags (which come out in DC in the <format> element) are stripped prior to export (though their contents are not), meaning that DC is not broken by this fix.