Opened 9 years ago

Closed 7 years ago

#6955 closed (wontfix)

Unicode XML breaks RelaxNGCompact validator

Reported by: Jeremy Dunck Owned by: Jeremy Dunck
Component: Validators Version: master
Severity: Keywords: unicode
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: UI/UX:

Description

In Django, the RelaxNGCompact validator writes out the field data to a temp file, then shells out to jing to perform RNG validation on that XML.

It's been broken for unicode data since the unicode branch landed. I guess not that many people use the RelaxNGCompact validator. ;-)

The fix is trivial-- just call smart_str on field data on its way into the file.

I'd provide tests, but I'm sort of overworked at the moment. :-/

Attachments (1)

rng-validator-unicode.diff (545 bytes) - added by Jeremy Dunck 9 years ago.
Needs a test; just pointing out the problem for now. :-/

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by Jeremy Dunck

Attachment: rng-validator-unicode.diff added

Needs a test; just pointing out the problem for now. :-/

comment:1 Changed 9 years ago by Jeremy Dunck

Owner: changed from nobody to Jeremy Dunck
Status: newassigned

comment:2 Changed 8 years ago by Marc Fargas

Triage Stage: UnreviewedAccepted

comment:3 Changed 7 years ago by Jeremy Dunck

Resolution: wontfix
Status: assignedclosed

At some point after I logged this, the XML validator was removed from trunk. Closing this as wontfix.

Note: See TracTickets for help on using tickets.
Back to Top