diff --git a/README.rst b/README.rst index 4a605a3..face5bd 100644 --- a/README.rst +++ b/README.rst @@ -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). diff --git a/suds/mx/appender.py b/suds/mx/appender.py index 726e262..0f0e973 100644 --- a/suds/mx/appender.py +++ b/suds/mx/appender.py @@ -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.