Loading an XML document with QDomDocument and namespace processing enabled (*) works,
but saving it back gives a bloated and redundant XML file with "xmlns:prefix=namespaceuri" added
to each and every element.

<?xml version='1.0' encoding='UTF-8'?>
<office:document-content xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0">
 <office:font-face-decls xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0">
  <style:font-face xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" svg:font-family="Bitstream Vera Sans" xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" style:name="Bitstream Vera Sans" xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" />
  <style:font-face xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" svg:font-family="DejaVu Sans" xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" style:name="DejaVu Sans" xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" />
 </office:font-face-decls>
  [...]

Redefining xmlns:office in every element other than the toplevel office:document-content is unnecessary
since the definition in an element is applicable to any child element. The initial XML document simply
defined all the namespaces in the <office:document-content> element.

Even worse, the <style:font-face> elements got xmlns:style defined twice.
Checking the validity of this document with jing and a relaxng schema shows that as an error:
content.xml:4:234: fatal: Attribute "xmlns:style" was already specified for element "style:font-face".

The initial document was
<?xml version="1.0" encoding="UTF-8"?>
<office:document-content xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0" xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0">
  <office:font-face-decls>
    <style:font-face style:name="Bitstream Vera Sans" svg:font-family="Bitstream Vera Sans"/>
    <style:font-face style:name="DejaVu Sans" svg:font-family="DejaVu Sans"/>
  </office:font-face-decls>
  [...]

QDomElementPrivate::save() has a TODO comment about this.
To "only declare namespaces that are not yet declared" should indeed be implemented.


(*)
    QXmlInputSource source( inputDevice );
    QXmlSimpleReader reader;
    reader.setFeature( "http://xml.org/sax/features/namespaces", true )
    reader.setFeature( "http://xml.org/sax/features/namespace-prefixes", false );
    QString errorMsg;
    int errorLine, errorColumn;
    bool ok = doc.setContent( &source, &reader, &errorMsg, &errorLine, &errorColumn );

    Then using doc.toByteArray() when saving.

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel