cwrc / DEPRECATED-CWRC-Writer

The Canadian Writing Research Collaboratory (CWRC) is developing an in-browser text markup editor (CWRCWriter) for use by collaborative scholarly editing projects.
http://www.cwrc.ca/projects/infrastructure-projects/technical-projects/cwrc-writer/
GNU General Public License v2.0
24 stars 17 forks source link

Character entity references for angle brackets corrupting file for validation #458

Closed SusanBrown closed 7 years ago

SusanBrown commented 7 years ago

This problem occurs in both the embedded versions, i.e. both with and without DE theming. Seems to be a bug that occurs because of the presence of the entity character references for angle brackets that occur in a revised template.

The line in the template that seems to be causing the problems: <culturalformation><religion><div2><shortprose><p>[prose description of sect or personal beliefs here]</></></></>,</>.

The attempt is to render something like this: <culturalformation><religion><div2><shortprose><p>[prose description of sect or personal beliefs here]</></></></>,</>

The result is that the XML seems to get corrupted so that it is missing some structural elements before being sent to the validation service and the CWRC-Writer says the file is invalid. The file validates without problem in Oxygen.

Here's the file in the unthemed embedded CWRC-Writer: http://cwrc-dev-05.srv.ualberta.ca/islandora/object/cwrc%3A8a44b9cc-a695-40f3-a759-0a78d67c2c6b/edit-cwrc#cwrc:8a44b9cc-a695-40f3-a759-0a78d67c2c6b

Here's a screenshot from @jefferya showing the missing text:

cwrc-writer_validate_cwrc 3abaeefd41-2b88-44d0-81cc-b73a96adcf57

ajmacdonald commented 7 years ago

Please test

SusanBrown commented 7 years ago

It validates in Dev05. Hurray! thanks for the quick fix!

On Nov 18, 2016, at 12:31 PM, Andrew notifications@github.com<mailto:notifications@github.com> wrote:

Please test

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/cwrc/CWRC-Writer/issues/458#issuecomment-261591376, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAhUoDmrEnCVmfS7bXuHu5xUcwX-JDRWks5q_eEAgaJpZM4K2ixI.