data.xml

Support for DOCTYPE when emitting XML

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code and Test

Description

Some consumers of XML files require an explicit DOCTYPE to accept an XML file. data.xml does not currently support the specification of doctypes when emitting XML. When XML is parsed, I believe DOCTYPEs are silently ignored, so there is no representation in the data model for them. The best design is possibly an :doctype option in clojure.data.xml/emit ?

I've attached a web.xml as example.

  1. web.xml
    14/Nov/12 2:06 AM
    0.4 kB
    Thomas Greve Kristensen
  2. DXML-10-Add-support-for-doctype-declarations.patch
    04/Jul/16 9:15 AM
    9 kB
    Christian Egli

Activity

Hide
Christian Egli added a comment -

Patch that adds support for doctype declarations when emitting XML

Show
Christian Egli added a comment - Patch that adds support for doctype declarations when emitting XML
Hide
Christian Egli added a comment -

Is there anything I can do to move this issue forward? I have a local hack around this issue which prevents me from upgrading org.clojure/data.xml. OTOH I really need the new namespace support, so a new version containing both the namespace and the DOCTYPE support would be fantastic.

Thanks

Show
Christian Egli added a comment - Is there anything I can do to move this issue forward? I have a local hack around this issue which prevents me from upgrading org.clojure/data.xml. OTOH I really need the new namespace support, so a new version containing both the namespace and the DOCTYPE support would be fantastic. Thanks
Hide
Ryan Senior added a comment -

Thanks for the patch! This has been applied and will be included in beta2 (which will be released soon)

Show
Ryan Senior added a comment - Thanks for the patch! This has been applied and will be included in beta2 (which will be released soon)

People

Vote (3)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: