[issue20714] Allow for ]]> in CDATA in minidom

Peter Otten added the comment: Perhaps a look at the competition is still in order: Java silently breaks such an invalid CDATA in two, as suggested. http://www.w3.org/TR/2004/REC-DOM-Level-3-Core-20040407/core.htm says """ No lexical check is done on the content of a CDATA section and it is the

[issue20714] Allow for ]]> in CDATA in minidom

Martin v. Löwis added the comment: I don't think your proposal can be implemented. createXYZ, in DOM, only creates a single node, which is not yet attached at all into a tree. So if you would want a convenience function to create multiple CDATA section, the application would then still have to

[issue20714] Allow for ]]> in CDATA in minidom

Artur R. Czechowski added the comment: Martin, I partially agree with you. After rethinking the issue I agree that changing the behavior of createCDATASection maybe is not a good idea. On the other hand anyone in need of adding ]]> into CDATA must write a few lines of code, which will be essent

[issue20714] Allow for ]]> in CDATA in minidom

Martin v. Löwis added the comment: I fail to see the bug in Python. minidom is behaving correctly. The error is in xmlrunner, which does error_info = str(test_result.get_error_info()) failureText = xml_document.createCDATASection(error_info) This is incorrect - it would have to check that

[issue20714] Allow for ]]> in CDATA in minidom

Changes by Martin v. Löwis : Removed file: http://bugs.python.org/file34167/minidom.patch ___ Python tracker ___ ___ Python-bugs-list mailing

[issue20714] Allow for ]]> in CDATA in minidom

Artur R. Czechowski added the comment: Martin, the exact information you need are: 1. this is what I did: #!/usr/bin/env python import unittest import xmlrunner class Foo(unittest.TestCase): def testFoo(self): self.assertTrue(False, ']]>') unittest.main(testRunner=xmlrunner.XMLTes

[issue20714] Allow for ]]> in CDATA in minidom

Changes by Peter Otten <__pete...@web.de>: -- nosy: +peter.otten ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue20714] Allow for ]]> in CDATA in minidom

Martin v. Löwis added the comment: Artur: Please provide the follwing information (in this bug report, or in any other bug report you create in the future) 1. this is what I did 2. this is what happened 3. this is what should have happened instead In this case, a Python script that ought to work

[issue20714] Allow for ]]> in CDATA in minidom

Changes by Martin v. Löwis : Added file: http://bugs.python.org/file34167/minidom.patch ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue20714] Allow for ]]> in CDATA in minidom

Changes by Artur R. Czechowski : Removed file: http://bugs.python.org/file34167/minidom.patch ___ Python tracker ___ ___ Python-bugs-list mail

[issue20714] Allow for ]]> in CDATA in minidom

Artur R. Czechowski added the comment: Proper patch with tests available in remote hg repo attached to this comment. -- hgrepos: +217 ___ Python tracker ___ _

[issue20714] Allow for ]]> in CDATA in minidom

Changes by Serhiy Storchaka : -- nosy: +serhiy.storchaka stage: -> test needed ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue20714] Allow for ]]> in CDATA in minidom

Artur R. Czechowski added the comment: Eric, I'm not sure what exactly your concern is, but I'll try to address two issues I can see. First: both strings and are a correct and valid examples of CDATA usage as per specification[1]. Second: is it allowed to have two occurences of CDATA inside

[issue20714] Allow for ]]> in CDATA in minidom

Éric Araujo added the comment: Does the XML spec allow that? -- nosy: +eric.araujo title: Please allow for ]]> in CDATA in minidom.py -> Allow for ]]> in CDATA in minidom versions: +Python 3.5 -Python 2.7, Python 3.3 ___ Python tracker