remove never used ``suds.mx.appender.DictAppender`` class
All code paths that could potentially lead to this class getting used convert any encountered dictionaries to ``suds.sudsobject.Object`` instances and report an error in case a corresponding XSD type can not be found.
This commit is contained in:
parent
c9a23c4a1e
commit
89c6b068c5
|
@ -171,6 +171,12 @@ version 0.7 (development)
|
|||
* Kudos to Kevin Pors (`krpors` on BitBucket) for detecting, analysing &
|
||||
reporting the issue, plus preparing a working quick-fix.
|
||||
|
||||
* Removed never actually used ``suds.mx.appender.DictAppender`` class.
|
||||
|
||||
* All code paths that could potentially lead to this class getting used
|
||||
convert any encountered dictionaries to ``suds.sudsobject.Object`` instances
|
||||
and report an error in case a corresponding XSD type can not be found.
|
||||
|
||||
* Made ``suds`` no longer eat up, log & ignore exceptions raised from registered
|
||||
user plugins (detected & reported by Ezequiel Ruiz & Bouke Haarsma, patch &
|
||||
test case contributed by Bouke Haarsma).
|
||||
|
|
|
@ -69,8 +69,7 @@ class ContentAppender:
|
|||
(Matcher(Element), ElementAppender(marshaller)),
|
||||
(Matcher(Text), TextAppender(marshaller)),
|
||||
(Matcher(list), ListAppender(marshaller)),
|
||||
(Matcher(tuple), ListAppender(marshaller)),
|
||||
(Matcher(dict), DictAppender(marshaller)))
|
||||
(Matcher(tuple), ListAppender(marshaller)))
|
||||
|
||||
def append(self, parent, content):
|
||||
"""
|
||||
|
@ -229,22 +228,6 @@ class ObjectAppender(Appender):
|
|||
Appender.append(self, child, cont)
|
||||
|
||||
|
||||
class DictAppender(Appender):
|
||||
"""
|
||||
An python I{dict} appender.
|
||||
"""
|
||||
|
||||
def append(self, parent, content):
|
||||
d = content.value
|
||||
if self.optional(content) and len(d) == 0:
|
||||
return
|
||||
child = self.node(content)
|
||||
parent.append(child)
|
||||
for item in d.items():
|
||||
cont = Content(tag=item[0], value=item[1])
|
||||
Appender.append(self, child, cont)
|
||||
|
||||
|
||||
class ElementWrapper(Element):
|
||||
"""
|
||||
Element wrapper.
|
||||
|
|
Loading…
Reference in New Issue