twisted.words.xish.domish doesn't deal with namespaces properly

[ralphm]

xish has several issues with namespaces that yield incorrect serializations:

 1. The default namespace is not reflected in serialization if the element has
    no parent
 2. Using 'no' namespace via xmlns='' doesn't work
 3. Prefix declarations (xmlns:*) of namespaced attributes are not generated

ad 1. This comes from Xish' Jabber specific roots.

ad 2. In the current implementation, there is no difference between not
specifying a namespace and specifying the empty namespace on Element creation.

[ralphm]

These issues have been fixed in source:branches/domish-namespaces-1444-2.

[radix]

Ok

• I'm not very happy about the way test_domish is done. Generally, one should try not to put assertions in methods that aren't directly and specifically called by the unit test method. Specifically I'm referring to the assertions about matched elements and such in _docStarted, _elementMatched, and so forth. A bug which prevents these methods from being called or completed could make the test look OK. But, I know that this is how it was before you came to it, so it's not required to be fixed for this merge.

• As I mentioned on IRC, generateOnlyInterface can be replaced with reduce(IFace.providedBy, list)

• "raise AttributeError" in getattr shoudl be "raise AttributeError(key)"

Good docs!

Go ahead and merge after the AttributeError is improved.

[ralphm]

About the tests in DomishStreamTests, yes, I'm not too happy about those, either. Also, it'd be better to query the generated objects directly, instead of relying on the XPath stuff. Be sure that it will be addressed when I need add new tests there.

Assuming you meant filter instead of reduce. As there are other generate* functions in domish.py, too, I'll just leave it for now. Also, filter() does not return a generator, but a constructed list.

Fixed the AttributeError stuff in r16634.

Thanks!

[ralphm]

(In [16635]) Fix namespace issues in domish and add IElement

Author: ralphm Reviewer: radix Fixes #1444

• Make domish handle prefixed attributes properly
• Allow the use of the empty namespace
• Some other edgecases concerning namespaces
• Let Element.toXml() take a defaultUri parameter for partial serializations.
• Allow manually constructed Elements to take None as a namespace, meaning it inherits the default namespace from its (logical) parent.
• Parsing XML using elementStream().parse() will always yield Elements with defined namespaces.
• Add the IElement interface and check objects using providedBy instead of using isinstance.
• Add lots of documentation

[radix]

Milestone Twisted-2.3 deleted