corrected GetChildren() docs

[wxWidgets.git] / wxPython / samples / ide / activegrid / util / xmlmarshaller.py

diff --git a/wxPython/samples/ide/activegrid/util/xmlmarshaller.py b/wxPython/samples/ide/activegrid/util/xmlmarshaller.py
index f305f0ada82f6b3c515f37f74783385f03c1d501..266418a1ebd73dfb9594f31fc9bf527091a923a2 100644 (file)
--- a/wxPython/samples/ide/activegrid/util/xmlmarshaller.py
+++ b/wxPython/samples/ide/activegrid/util/xmlmarshaller.py
@@ -9,20 +9,22 @@
 # Copyright:    (c) 2004-2005 ActiveGrid, Inc.
 # License:      wxWindows License
 #----------------------------------------------------------------------------
 # Copyright:    (c) 2004-2005 ActiveGrid, Inc.
 # License:      wxWindows License
 #----------------------------------------------------------------------------

-from activegrid import util
-import inspect
+import __builtin__
+import sys

 from types import *
 from types import *

+import logging

 import xml.sax
 import xml.sax.handler
 import xml.sax
 import xml.sax.handler

-import __builtin__
-from xml.sax import saxutils
+import xml.sax.saxutils as saxutils
+from activegrid.util.lang import *
+import activegrid.util.aglogging as aglogging

 
 

-### ToDO remove maxOccurs "unbounded" resolves to -1 hacks after bug 177 is fixed
+MODULE_PATH = "__main__"

 
 

-"""
+## ToDO remove maxOccurs "unbounded" resolves to -1 hacks after bug 177 is fixed

 
 

-More documentation later, but here are some special Python attributes
-that McLane recognizes:
+"""
+Special attributes that we recognize:

 
 name: __xmlname__
 type: string
 
 name: __xmlname__
 type: string


@@ -30,26 +32,26 @@ description: the name of the xml element for the marshalled object
 
 name: __xmlattributes__
 type: tuple or list
 
 name: __xmlattributes__
 type: tuple or list

-description: the name(s) of the Python string attribute(s) to be
+description: the name(s) of the Lang string attribute(s) to be

 marshalled as xml attributes instead of nested xml elements. currently
 marshalled as xml attributes instead of nested xml elements. currently

-these can only be strings since there's not a way to get the type
+these can only be strings since there"s not a way to get the type

 information back when unmarshalling.
 
 name: __xmlexclude__
 type: tuple or list
 information back when unmarshalling.
 
 name: __xmlexclude__
 type: tuple or list

-description: the name(s) of the python attribute(s) to skip when
+description: the name(s) of the lang attribute(s) to skip when

 marshalling.
 
 name: __xmlrename__
 type: dict
 marshalling.
 
 name: __xmlrename__
 type: dict

-description: describes an alternate Python <-> XML name mapping.  
+description: describes an alternate Lang <-> XML name mapping.  

 Normally the name mapping is the identity function.  __xmlrename__
 Normally the name mapping is the identity function.  __xmlrename__

-overrides that.  The keys are the Python names, the values are their
+overrides that.  The keys are the Lang names, the values are their

 associated XML names.
 
 name: __xmlflattensequence__
 type: dict, tuple, or list
 associated XML names.
 
 name: __xmlflattensequence__
 type: dict, tuple, or list

-description: the name(s) of the Python sequence attribute(s) whose
+description: the name(s) of the Lang sequence attribute(s) whose

 items are to be marshalled as a series of xml elements (with an
 optional keyword argument that specifies the element name to use) as
 opposed to containing them in a separate sequence element, e.g.:
 items are to be marshalled as a series of xml elements (with an
 optional keyword argument that specifies the element name to use) as
 opposed to containing them in a separate sequence element, e.g.:


@@ -57,12 +59,12 @@ opposed to containing them in a separate sequence element, e.g.:
 myseq = (1, 2)
 <!-- normal way of marshalling -->
 <myseq>
 myseq = (1, 2)
 <!-- normal way of marshalling -->
 <myseq>

-  <item objtype='int'>1</item>
-  <item objtype='int'>2</item>
+  <item objtype="int">1</item>
+  <item objtype="int">2</item>

 </myseq>
 </myseq>

-<!-- with __xmlflattensequence__ set to {'myseq': 'squish'} -->
-<squish objtype='int'>1</squish>
-<squish objtype='int'>2</squish>
+<!-- with __xmlflattensequence__ set to {"myseq": "squish"} -->
+<squish objtype="int">1</squish>
+<squish objtype="int">2</squish>

 
 name: __xmlnamespaces__
 type: dict
 
 name: __xmlnamespaces__
 type: dict


@@ -79,16 +81,31 @@ should be used as the default namespace for the object.
 
 name: __xmlattrnamespaces__
 type: dict
 
 name: __xmlattrnamespaces__
 type: dict

-description: a dict assigning the Python object's attributes to the namespaces
+description: a dict assigning the Lang object"s attributes to the namespaces

 defined in __xmlnamespaces__.  Each item in the dict should consist of a
 prefix,attributeList combination where the key is the prefix and the value is
 defined in __xmlnamespaces__.  Each item in the dict should consist of a
 prefix,attributeList combination where the key is the prefix and the value is

-a list of the Python attribute names.  e.g.:
+a list of the Lang attribute names.  e.g.:

 
 __xmlattrnamespaces__ = { "ag":["firstName", "lastName", "addressLine1", "city"] }
 
 
 __xmlattrnamespaces__ = { "ag":["firstName", "lastName", "addressLine1", "city"] }
 

+name: __xmlattrgroups__
+type: dict
+description: a dict specifying groups of attributes to be wrapped in an enclosing tag.
+The key is the name of the enclosing tag; the value is a list of attributes to include
+within it. e.g.
+
+__xmlattrgroups__ = {"name": ["firstName", "lastName"], "address": ["addressLine1", "city", "state", "zip"]}

 
 """
 
 
 """
 

+global xmlMarshallerLogger
+xmlMarshallerLogger = logging.getLogger("activegrid.util.xmlmarshaller.marshal")
+xmlMarshallerLogger.setLevel(aglogging.LEVEL_WARN)
+# INFO  : low-level info
+# DEBUG : debugging info
+
+global knownGlobalTypes
+

 ################################################################################
 #
 # module exceptions
 ################################################################################
 #
 # module exceptions


@@ -109,7 +126,7 @@ class UnhandledTypeException(Error):
         return "%s is not supported for marshalling." % str(self.typename)
 
 class XMLAttributeIsNotStringType(Error):
         return "%s is not supported for marshalling." % str(self.typename)
 
 class XMLAttributeIsNotStringType(Error):

-    """Exception raised when an object's attribute is specified to be
+    """Exception raised when an object"s attribute is specified to be

     marshalled as an XML attribute of the enclosing object instead of
     a nested element.
     """
     marshalled as an XML attribute of the enclosing object instead of
     a nested element.
     """


@@ -118,37 +135,33 @@ class XMLAttributeIsNotStringType(Error):
         self.typename = typename
     def __str__(self):
         return """%s was set to be marshalled as an XML attribute
         self.typename = typename
     def __str__(self):
         return """%s was set to be marshalled as an XML attribute

-        instead of a nested element, but the object's type is %s, not
+        instead of a nested element, but the object"s type is %s, not

         string.""" % (self.attrname, self.typename)
 
         string.""" % (self.attrname, self.typename)
 

+class MarshallerException(Exception):
+    pass 
+

 ################################################################################
 #
 # constants and such
 #
 ################################################################################
 
 ################################################################################
 #
 # constants and such
 #
 ################################################################################
 

-XMLNS = 'xmlns'
-XMLNS_PREFIX = XMLNS + ':'
+XMLNS = "xmlns"
+XMLNS_PREFIX = XMLNS + ":"

 XMLNS_PREFIX_LENGTH = len(XMLNS_PREFIX)
 
 XMLNS_PREFIX_LENGTH = len(XMLNS_PREFIX)
 

-BASETYPE_ELEMENT_NAME = 'item'
-MEMBERS_TO_SKIP = ('__module__', '__doc__', '__xmlname__', '__xmlattributes__',
-                   '__xmlexclude__', '__xmlflattensequence__', '__xmlnamespaces__',
-                   '__xmldefaultnamespace__', '__xmlattrnamespaces__')
-
-WELL_KNOWN_OBJECTS = { "xs:element"     : "activegrid.model.schema.XsdElement",
-                       "xs:complexType" : "activegrid.model.schema.XsdComplexType",
-                       "xs:complexType" : "activegrid.model.schema.XsdComplexType",
-                       "xs:element"     : "activegrid.model.schema.XsdElement",
-                       "xs:key"         : "activegrid.model.schema.XsdKey",
-                       "xs:field"       : "activegrid.model.schema.XsdKeyField",
-                       "xs:keyref"      : "activegrid.model.schema.XsdKeyRef",
-                       "xs:selector"    : "activegrid.model.schema.XsdKeySelector",              
-                       "xs:schema"      : "activegrid.model.schema.Schema",
-                       "ag:schemaOptions":"activegrid.model.schema.SchemaOptions",
-                       "ag:debug"       : "activegrid.model.processmodel.DebugOperation",
-                     }
+BASETYPE_ELEMENT_NAME = "item"
+DICT_ITEM_NAME = "qqDictItem"
+DICT_ITEM_KEY_NAME = "key"
+DICT_ITEM_VALUE_NAME = "value"

 
 

+# This list doesn"t seem to be used.
+#   Internal documentation or useless? You make the call!
+##MEMBERS_TO_SKIP = ("__module__", "__doc__", "__xmlname__", "__xmlattributes__",
+##                   "__xmlexclude__", "__xmlflattensequence__", "__xmlnamespaces__",
+##                   "__xmldefaultnamespace__", "__xmlattrnamespaces__",
+##                   "__xmlattrgroups__")

 
 ################################################################################
 #
 
 ################################################################################
 #


@@ -156,73 +169,100 @@ WELL_KNOWN_OBJECTS = { "xs:element"     : "activegrid.model.schema.XsdElement",
 #
 ################################################################################
 
 #
 ################################################################################
 

+def setattrignorecase(object, name, value):
+    if (name not in object.__dict__):
+        namelow = name.lower()
+        for attr in object.__dict__:
+            if attr.lower() == namelow:
+                object.__dict__[attr] = value
+                return
+    object.__dict__[name] = value
+
+def getComplexType(obj):
+    if (hasattr(obj, "__xsdcomplextype__")):
+        return obj.__xsdcomplextype__
+    return None
+

 def _objectfactory(objname, objargs=None, xsname=None):
 def _objectfactory(objname, objargs=None, xsname=None):

-    try:
-        '''dynamically create an object based on the objname and return
-        it. look it up in the BASETYPE_ELEMENT_MAP first.
-        '''
-##        print "_objectfactory creating an object of type %s and value %s, xsname=%s" % (objname, objargs, xsname)
-        # split the objname into the typename and module path,
-        # importing the module if need be.
-        if not isinstance(objargs, list):
-            objargs = [objargs]
-            
-        if (xsname):
-            try:
-                objname = WELL_KNOWN_OBJECTS[xsname]
-            except KeyError:
-                pass
-
-        objtype = objname.split('.')[-1]
-        pathlist = objname.split('.')
-        modulename = '.'.join(pathlist[0:-1])
-
-##        print "[objectfactory] objtype is %s" % objtype
-##        print "[objectfactory] objargs is %s" % `objargs`
-
-        ## since the bool constructor will turn a string of non-zero
-        ## length into True, we call it with no argument (yielding a
-        ## False) if the string contains 'false'
-        if objtype == 'bool' and objargs[0].lower() == 'false':
-            objargs = None
-
-##        if objtype == 'str':
-##            print type(objargs)
-##            print "string we're unescaping: '%s'" % objargs[0]
-##            objargs = saxutils.unescape(objargs[0])
-        if objtype in ('float', 'int', 'str', 'long'):
-            objargs = [x.strip() for x in objargs]
-
-        if objtype == 'str':
-            objargs = [saxutils.unescape(x) for x in objargs]
-
-        if __builtin__.__dict__.has_key(objname):
-            module = __builtin__
+    "dynamically create an object based on the objname and return it."
+
+    if not isinstance(objargs, list):
+        objargs = [objargs]
+
+##    print "[objectfactory] xsname [%s]; objname [%s]" % (xsname, objname)
+
+    # (a) deal with tagName:knownTypes mappings
+    if (xsname != None):
+        objclass = knownGlobalTypes.get(xsname)
+        if (objclass != None):
+            if (objargs != None):
+                return objclass(*objargs)
+            else:
+                return objclass()                
+
+    # (b) next with intrinisic types
+    if objname == "str" or objname == "unicode": # don"t strip: blanks are significant
+        if len(objargs) > 0:
+            return saxutils.unescape(objargs[0]).encode()

         else:
         else:

-            if modulename:
-                module = __import__(modulename)
+            return ""
+    elif objname == "bool":
+        return not objargs[0].lower() == "false"
+    elif objname in ("float", "int", "long"):
+##        objargs = [x.strip() for x in objargs]
+        return __builtin__.__dict__[objname](*objargs)
+    elif objname == "None":
+        return None
+
+    # (c) objtype=path...module.class   
+    # split the objname into the typename and module path,
+    # importing the module if need be.
+##    print "[objectfactory] creating an object of type %s and value %s, xsname=%s" % (objname, objargs, xsname)
+    objtype = objname.split(".")[-1]
+    pathlist = objname.split(".")
+    modulename = ".".join(pathlist[0:-1])
+##    print "[objectfactory] object [%s] %s(%r)" % (objname, objtype, objargs)
+
+    try:
+        if modulename:
+            module = __import__(modulename)

             for name in pathlist[1:-1]:
                 module = module.__dict__[name]
             for name in pathlist[1:-1]:
                 module = module.__dict__[name]

+        elif __builtin__.__dict__.has_key(objname):
+            module = __builtin__
+        else:
+            raise MarshallerException("Could not find class %s" % objname)

         if objargs:
             return module.__dict__[objtype](*objargs)
         else:
         if objargs:
             return module.__dict__[objtype](*objargs)
         else:

-            if objtype == 'None':
-                return None

             return module.__dict__[objtype]()
     except KeyError:
             return module.__dict__[objtype]()
     except KeyError:

-        raise KeyError("Could not find class %s" % objname)
+        raise MarshallerException("Could not find class %s" % objname)

 
 class Element:
 
 class Element:

+    

     def __init__(self, name, attrs=None):
         self.name = name
         self.attrs = attrs
     def __init__(self, name, attrs=None):
         self.name = name
         self.attrs = attrs

-        self.content = ''
+        self.content = ""

         self.children = []
         self.children = []

+        

     def getobjtype(self):
     def getobjtype(self):

-        if self.attrs.has_key('objtype'):
-            return self.attrs.getValue('objtype')
-        else:
-            return 'str'
+        objtype = self.attrs.get("objtype")
+        if (objtype == None):
+            if (len(self.children) > 0):
+                objtype = "dict"
+            else:
+                objtype = "str"
+        return objtype
+            
+    def __str__(self):
+        print "    name = ", self.name, "; attrs = ", self.attrs, "number of children = ", len(self.children)
+        i = -1
+        for child in self.children:
+            i = i + 1
+            childClass = child.__class__.__name__
+            print "    Child ", i, " class: ",childClass

 
 
 class XMLObjectFactory(xml.sax.ContentHandler):
 
 
 class XMLObjectFactory(xml.sax.ContentHandler):


@@ -231,10 +271,25 @@ class XMLObjectFactory(xml.sax.ContentHandler):
         self.elementstack = []
         xml.sax.handler.ContentHandler.__init__(self)
 
         self.elementstack = []
         xml.sax.handler.ContentHandler.__init__(self)
 

+    def __str__(self):
+        print "-----XMLObjectFactory Dump-------------------------------"
+        if (self.rootelement == None):
+            print "rootelement is None"
+        else:
+            print "rootelement is an object"
+        i = -1
+        print "length of elementstack is: ", len(self.elementstack)
+        for e in self.elementstack:
+            i = i + 1
+            print "elementstack[", i, "]: "
+            str(e)
+        print "-----end XMLObjectFactory--------------------------------"
+        

     ## ContentHandler methods
     def startElement(self, name, attrs):
     ## ContentHandler methods
     def startElement(self, name, attrs):

-        if name.find(':') > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
-            name = name[name.index(':') + 1:]
+##        print "startElement for name: ", name
+        if name.find(":") > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
+            name = name[name.find(":") + 1:]

 ##        for attrname in attrs.getNames():
 ##            print "%s: %s" % (attrname, attrs.getValue(attrname))
         element = Element(name, attrs.copy())
 ##        for attrname in attrs.getNames():
 ##            print "%s: %s" % (attrname, attrs.getValue(attrname))
         element = Element(name, attrs.copy())


@@ -242,94 +297,142 @@ class XMLObjectFactory(xml.sax.ContentHandler):
 ##        print self.elementstack
 
     def characters(self, content):
 ##        print self.elementstack
 
     def characters(self, content):

-##        print "got content: %s" % content
-        if content:
+##        print "got content: %s (%s)" % (content, type(content))
+        if (content != None):

             self.elementstack[-1].content += content
 
     def endElement(self, name):
             self.elementstack[-1].content += content
 
     def endElement(self, name):

-##        print "[endElement] name of element we're at the end of: %s" % name
+##        print "[endElement] name of element we"re at the end of: %s" % name

         xsname = name
         xsname = name

-        if name.find(':') > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
-            name = name[name.index(':') + 1:]
+        if name.find(":") > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
+            name = name[name.find(":") + 1:]
+        oldChildren = self.elementstack[-1].children

         element = self.elementstack.pop()
         element = self.elementstack.pop()

+        if ((len(self.elementstack) > 1) and (self.elementstack[-1].getobjtype() == "None")):
+            parentElement = self.elementstack[-2]
+##            print "[endElement] %s: found parent with objtype==None: using its grandparent" % name
+        elif (len(self.elementstack) > 0):
+            parentElement = self.elementstack[-1]

         objtype = element.getobjtype()
         objtype = element.getobjtype()

+##        print "element objtype is: ", objtype
+        if (objtype == "None"):
+##            print "[endElement] %s: skipping a (objtype==None) end tag" % name
+            return

         constructorarglist = []
         constructorarglist = []

-        if element.content:
+        if (len(element.content) > 0):

             strippedElementContent = element.content.strip()
             strippedElementContent = element.content.strip()

-            if strippedElementContent:
+            if (len(strippedElementContent) > 0):

                 constructorarglist.append(element.content)
                 constructorarglist.append(element.content)

+##        print "[endElement] calling objectfactory"

         obj = _objectfactory(objtype, constructorarglist, xsname)
         obj = _objectfactory(objtype, constructorarglist, xsname)

-        complexType = None
-        if hasattr(obj, '__xsdcomplextype__'):
-            complexType = getattr(obj, '__xsdcomplextype__')
-        if len(self.elementstack) > 0:
-            self.elementstack[-1].children.append((name, obj))
-        else:
-            self.rootelement = obj
-        if element.attrs and not isinstance(obj, list):
+        complexType = getComplexType(obj)
+        if (obj != None):
+            if (hasattr(obj, "__xmlname__") and getattr(obj, "__xmlname__") == "sequence"):
+                self.elementstack[-1].children = oldChildren
+                return
+        if (len(element.attrs) > 0) and not isinstance(obj, list):
+##            print "[endElement] %s: element has attrs and the obj is not a list" % name

             for attrname, attr in element.attrs.items():
                 if attrname == XMLNS or attrname.startswith(XMLNS_PREFIX):
                     if attrname.startswith(XMLNS_PREFIX):
                         ns = attrname[XMLNS_PREFIX_LENGTH:]
                     else:
                         ns = ""
             for attrname, attr in element.attrs.items():
                 if attrname == XMLNS or attrname.startswith(XMLNS_PREFIX):
                     if attrname.startswith(XMLNS_PREFIX):
                         ns = attrname[XMLNS_PREFIX_LENGTH:]
                     else:
                         ns = ""

-                    if not hasattr(obj, '__xmlnamespaces__'):
+                    if not hasattr(obj, "__xmlnamespaces__"):

                         obj.__xmlnamespaces__ = {ns:attr}
                     elif ns not in obj.__xmlnamespaces__:
                         obj.__xmlnamespaces__ = {ns:attr}
                     elif ns not in obj.__xmlnamespaces__:

-                        if (hasattr(obj.__class__, '__xmlnamespaces__') 
-                           and obj.__xmlnamespaces__ is obj.__class__.__xmlnamespaces__):
-                                obj.__xmlnamespaces__ = dict(obj.__xmlnamespaces__)
+                        if (hasattr(obj.__class__, "__xmlnamespaces__") 
+                            and (obj.__xmlnamespaces__ is obj.__class__.__xmlnamespaces__)):
+                            obj.__xmlnamespaces__ = dict(obj.__xmlnamespaces__)

                         obj.__xmlnamespaces__[ns] = attr
                         obj.__xmlnamespaces__[ns] = attr

-                elif not attrname == 'objtype':
-                    if attrname.find(':') > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
-                        attrname = attrname[attrname.index(':') + 1:]
-                    if complexType:
+                elif not attrname == "objtype":
+                    if attrname.find(":") > -1:  # Strip namespace prefixes for now until actually looking them up in xsd
+                        attrname = attrname[attrname.find(":") + 1:]
+                    if (complexType != None):

                         xsdElement = complexType.findElement(attrname)
                         xsdElement = complexType.findElement(attrname)

-                        if xsdElement:
+                        if (xsdElement != None):

                             type = xsdElement.type
                             type = xsdElement.type

-                            if type:
-                                type = xsdToPythonType(type)
+                            if (type != None):
+                                type = xsdToLangType(type)

                                 ### ToDO remove maxOccurs hack after bug 177 is fixed
                                 if attrname == "maxOccurs" and attr == "unbounded":
                                     attr = "-1"
                                 attr = _objectfactory(type, attr)
                                 ### ToDO remove maxOccurs hack after bug 177 is fixed
                                 if attrname == "maxOccurs" and attr == "unbounded":
                                     attr = "-1"
                                 attr = _objectfactory(type, attr)

-                    util.setattrignorecase(obj, _toAttrName(obj, attrname), attr)
+                    try:
+                        setattrignorecase(obj, _toAttrName(obj, attrname), attr)
+                    except AttributeError:
+                        raise MarshallerException("Error unmarshalling attribute \"%s\" of XML element \"%s\": object type not specified or known" % (attrname, name))

 ##                    obj.__dict__[_toAttrName(obj, attrname)] = attr
         # stuff any child attributes meant to be in a sequence via the __xmlflattensequence__
         flattenDict = {}
 ##                    obj.__dict__[_toAttrName(obj, attrname)] = attr
         # stuff any child attributes meant to be in a sequence via the __xmlflattensequence__
         flattenDict = {}

-        if hasattr(obj, '__xmlflattensequence__'):
-            for sequencename, xmlnametuple in obj.__xmlflattensequence__.items():
-                for xmlname in xmlnametuple:
-                    flattenDict[xmlname] = sequencename
+        if hasattr(obj, "__xmlflattensequence__"):
+            flatten = obj.__xmlflattensequence__
+##            print "[endElement] %s: obj has __xmlflattensequence__" % name
+            if (isinstance(flatten, dict)):
+##                print "[endElement]  dict with flatten.items: ", flatten.items()
+                for sequencename, xmlnametuple in flatten.items():
+                    if (xmlnametuple == None):
+                        flattenDict[sequencename] = sequencename
+                    elif (not isinstance(xmlnametuple, (tuple, list))):
+                        flattenDict[str(xmlnametuple)] = sequencename
+                    else:
+                        for xmlname in xmlnametuple:
+    ##                        print "[endElement]: adding flattenDict[%s] = %s" % (xmlname, sequencename)
+                            flattenDict[xmlname] = sequencename
+            else:
+                raise "Invalid type for __xmlflattensequence___ : it must be a dict"

 
 

-        # reattach an object's attributes to it
+        # reattach an object"s attributes to it

         for childname, child in element.children:
         for childname, child in element.children:

-            if flattenDict.has_key(childname):
+##            print "[endElement] childname is: ", childname, "; child is: ", child
+            if (childname in flattenDict):

                 sequencename = _toAttrName(obj, flattenDict[childname])
                 sequencename = _toAttrName(obj, flattenDict[childname])

-                try:
-                    sequencevalue = obj.__dict__[sequencename]
-                except AttributeError:
-                    sequencevalue = None
-                if sequencevalue == None:
-                    sequencevalue = []
-                    obj.__dict__[sequencename] = sequencevalue
+##                print "[endElement] sequencename is: ", sequencename
+                if (not hasattr(obj, sequencename)):
+##                    print "[endElement] obj.__dict__ is: ", obj.__dict__
+                    obj.__dict__[sequencename] = []
+                sequencevalue = getattr(obj, sequencename)
+                if (sequencevalue == None):
+                    obj.__dict__[sequencename] = []
+                    sequencevalue = getattr(obj, sequencename)

                 sequencevalue.append(child)
                 sequencevalue.append(child)

-            elif isinstance(obj, list):
+            elif (objtype == "list"):

                 obj.append(child)
                 obj.append(child)

+            elif isinstance(obj, dict):
+                if (childname == DICT_ITEM_NAME):
+                    obj[child[DICT_ITEM_KEY_NAME]] = child[DICT_ITEM_VALUE_NAME]
+                else:
+                    obj[childname] = child

             else:
 ##                print "childname = %s, obj = %s, child = %s" % (childname, repr(obj), repr(child))
             else:
 ##                print "childname = %s, obj = %s, child = %s" % (childname, repr(obj), repr(child))

-                util.setattrignorecase(obj, _toAttrName(obj, childname), child)
+                try:
+                    setattrignorecase(obj, _toAttrName(obj, childname), child)
+                except AttributeError:
+                    raise MarshallerException("Error unmarshalling child element \"%s\" of XML element \"%s\": object type not specified or known" % (childname, name))

 ##                obj.__dict__[_toAttrName(obj, childname)] = child
 
 ##                obj.__dict__[_toAttrName(obj, childname)] = child
 

-        if complexType:
+        if (complexType != None):

             for element in complexType.elements:
                 if element.default:
                     elementName = _toAttrName(obj, element.name)
                     if ((elementName not in obj.__dict__) or (obj.__dict__[elementName] == None)):
             for element in complexType.elements:
                 if element.default:
                     elementName = _toAttrName(obj, element.name)
                     if ((elementName not in obj.__dict__) or (obj.__dict__[elementName] == None)):

-                        pythonType = xsdToPythonType(element.type)
-                        defaultValue = _objectfactory(pythonType, element.default)
+                        langType = xsdToLangType(element.type)
+                        defaultValue = _objectfactory(langType, element.default)

                         obj.__dict__[elementName] = defaultValue
 
                         obj.__dict__[elementName] = defaultValue
 

+        ifDefPy()
+        if (isinstance(obj, list)):
+            if ((element.attrs.has_key("mutable")) and (element.attrs.getValue("mutable") == "false")):
+                obj = tuple(obj)
+        endIfDef()
+            
+        if (len(self.elementstack) > 0):
+##            print "[endElement] appending child with name: ", name, "; objtype: ", objtype
+            parentElement.children.append((name, obj))
+##            print "parentElement now has ", len(parentElement.children), " children"
+        else:
+            self.rootelement = obj
+            

     def getRootObject(self):
         return self.rootelement
 
     def getRootObject(self):
         return self.rootelement
 


@@ -343,11 +446,11 @@ def _toAttrName(obj, name):
 ##        name = "_%s%s" % (obj.__class__.__name__, name)
     return name
 
 ##        name = "_%s%s" % (obj.__class__.__name__, name)
     return name
 

-__typeMappingXsdToPython = {
+__typeMappingXsdToLang = {

     "string": "str",
     "char": "str",
     "varchar": "str",
     "string": "str",
     "char": "str",
     "varchar": "str",

-    "date": "str", # ToDO Need to work out how to create python date types
+    "date": "str", # ToDO Need to work out how to create lang date types

     "boolean": "bool",
     "decimal": "float", # ToDO Does python have a better fixed point type?
     "int": "int",
     "boolean": "bool",
     "decimal": "float", # ToDO Does python have a better fixed point type?
     "int": "int",


@@ -356,103 +459,136 @@ __typeMappingXsdToPython = {
     "bool": "bool",
     "str": "str",
     "unicode": "unicode",
     "bool": "bool",
     "str": "str",
     "unicode": "unicode",

+    "short": "int",
+    "duration": "str", # see above (date)
+    "datetime": "str", # see above (date)
+    "time": "str", # see above (date)
+    "double": "float",

     }    
 
     }    
 

-def xsdToPythonType(xsdType):
-    try:
-        return __typeMappingXsdToPython[xsdType]
-    except KeyError:
+def xsdToLangType(xsdType):
+    langType = __typeMappingXsdToLang.get(xsdType)
+    if (langType == None):

         raise Exception("Unknown xsd type %s" % xsdType)
         raise Exception("Unknown xsd type %s" % xsdType)

+    return langType

 
 

-def _getXmlValue(pythonValue):
-    if (isinstance(pythonValue, bool)):
-        return str(pythonValue).lower()
+def _getXmlValue(langValue):
+    if (isinstance(langValue, bool)):
+        return str(langValue).lower()
+    elif (isinstance(langValue, unicode)):
+        return langValue.encode()

     else:
     else:

-        return str(pythonValue)
+        return str(langValue)

 
 

-def unmarshal(xmlstr):
+def unmarshal(xmlstr, knownTypes=None):
+    global knownGlobalTypes
+    if (knownTypes == None):
+        knownGlobalTypes = {}
+    else:
+        knownGlobalTypes = knownTypes

     objectfactory = XMLObjectFactory()
     xml.sax.parseString(xmlstr, objectfactory)
     return objectfactory.getRootObject()
 
 
     objectfactory = XMLObjectFactory()
     xml.sax.parseString(xmlstr, objectfactory)
     return objectfactory.getRootObject()
 
 

-def marshal(obj, elementName=None, nameSpacePrefix='', nameSpaces=None, prettyPrint=False, indent=0):
+def marshal(obj, elementName=None, prettyPrint=False, indent=0, knownTypes=None, encoding=-1):
+    xmlstr = "".join(_marshal(obj, elementName, prettyPrint=prettyPrint, indent=indent, knownTypes=knownTypes))
+    if (isinstance(encoding, basestring)):
+        return '<?xml version="1.0" encoding="%s"?>\n%s' % (encoding, xmlstr.encode(encoding))
+    elif (encoding == None):
+        return xmlstr
+    else:
+        return '<?xml version="1.0" encoding="%s"?>\n%s' % (sys.getdefaultencoding(), xmlstr)
+
+def _marshal(obj, elementName=None, nameSpacePrefix="", nameSpaces=None, prettyPrint=False, indent=0, knownTypes=None):
+    xmlMarshallerLogger.debug("--> _marshal: elementName=%s, type=%s, obj=%s" % (elementName, type(obj), str(obj)))
+    xmlString = None

     if prettyPrint or indent:
     if prettyPrint or indent:

-        prefix = ' '*indent
-        newline = '\n'
+        prefix = " "*indent
+        newline = "\n"

         increment = 4
     else:
         increment = 4
     else:

-        prefix = ''
-        newline = ''
+        prefix = ""
+        newline = ""

         increment = 0
 
         increment = 0
 

-    ## Determine the XML element name. If it isn't specified in the
-    ## parameter list, look for it in the __xmlname__ Python
+    ## Determine the XML element name. If it isn"t specified in the
+    ## parameter list, look for it in the __xmlname__ Lang

     ## attribute, else use the default generic BASETYPE_ELEMENT_NAME.
     if not nameSpaces: nameSpaces = {}  # Need to do this since if the {} is a default parameter it gets shared by all calls into the function
     ## attribute, else use the default generic BASETYPE_ELEMENT_NAME.
     if not nameSpaces: nameSpaces = {}  # Need to do this since if the {} is a default parameter it gets shared by all calls into the function

-    nameSpaceAttrs = ''
-    if hasattr(obj, '__xmlnamespaces__'):
-        for nameSpaceKey, nameSpaceUrl in getattr(obj, '__xmlnamespaces__').items():
-            if nameSpaceUrl in nameSpaces:
+    nameSpaceAttrs = ""
+    if knownTypes == None:
+        knownTypes = {}
+    if hasattr(obj, "__xmlnamespaces__"):
+        for nameSpaceKey, nameSpaceUrl in getattr(obj, "__xmlnamespaces__").items():
+            if nameSpaceUrl in asDict(nameSpaces):

                 nameSpaceKey = nameSpaces[nameSpaceUrl]
             else:
                 nameSpaceKey = nameSpaces[nameSpaceUrl]
             else:

-##                # TODO: Wait to do this until there is shared state for use when going through the object graph
-##                origNameSpaceKey = nameSpaceKey  # Make sure there is no key collision, ie: same key referencing two different URL's
+##                # TODO: Wait to do this until there is shared for use when going through the object graph
+##                origNameSpaceKey = nameSpaceKey  # Make sure there is no key collision, ie: same key referencing two different URL"s

 ##                i = 1
 ##                while nameSpaceKey in nameSpaces.values():
 ##                    nameSpaceKey = origNameSpaceKey + str(i)
 ##                    i += 1
                 nameSpaces[nameSpaceUrl] = nameSpaceKey
 ##                i = 1
 ##                while nameSpaceKey in nameSpaces.values():
 ##                    nameSpaceKey = origNameSpaceKey + str(i)
 ##                    i += 1
                 nameSpaces[nameSpaceUrl] = nameSpaceKey

-                if nameSpaceKey == '':
+                if nameSpaceKey == "":

                     nameSpaceAttrs += ' xmlns="%s" ' % (nameSpaceUrl)
                 else:
                     nameSpaceAttrs += ' xmlns:%s="%s" ' % (nameSpaceKey, nameSpaceUrl)
         nameSpaceAttrs = nameSpaceAttrs.rstrip()
                     nameSpaceAttrs += ' xmlns="%s" ' % (nameSpaceUrl)
                 else:
                     nameSpaceAttrs += ' xmlns:%s="%s" ' % (nameSpaceKey, nameSpaceUrl)
         nameSpaceAttrs = nameSpaceAttrs.rstrip()

-    if hasattr(obj, '__xmldefaultnamespace__'):
-        nameSpacePrefix = getattr(obj, '__xmldefaultnamespace__') + ':'
+    if hasattr(obj, "__xmldefaultnamespace__"):
+        nameSpacePrefix = getattr(obj, "__xmldefaultnamespace__") + ":"        

     if not elementName:
     if not elementName:

-        if hasattr(obj, '__xmlname__'):
+        if hasattr(obj, "__xmlname__"):

             elementName = nameSpacePrefix + obj.__xmlname__
         else:
             elementName = nameSpacePrefix + BASETYPE_ELEMENT_NAME
     else:
         elementName = nameSpacePrefix + elementName
             elementName = nameSpacePrefix + obj.__xmlname__
         else:
             elementName = nameSpacePrefix + BASETYPE_ELEMENT_NAME
     else:
         elementName = nameSpacePrefix + elementName

-        
+    if hasattr(obj, "__xmlsequencer__"):
+        elementAdd = obj.__xmlsequencer__
+    else:
+        elementAdd = None
+               
+##    print "marshal: entered with elementName: ", elementName

     members_to_skip = []
     ## Add more members_to_skip based on ones the user has selected
     ## via the __xmlexclude__ attribute.
     members_to_skip = []
     ## Add more members_to_skip based on ones the user has selected
     ## via the __xmlexclude__ attribute.

-    if hasattr(obj, '__xmlexclude__'):
-        members_to_skip += list(obj.__xmlexclude__)
+    if hasattr(obj, "__xmlexclude__"):
+##        print "marshal: found __xmlexclude__"
+        members_to_skip.extend(obj.__xmlexclude__)

     # Marshal the attributes that are selected to be XML attributes.
     # Marshal the attributes that are selected to be XML attributes.

-    objattrs = ''
-    className = obj.__class__.__name__
+    objattrs = ""
+    className = ag_className(obj)

     classNamePrefix = "_" + className
     classNamePrefix = "_" + className

-    if hasattr(obj, '__xmlattributes__'):
+    if hasattr(obj, "__xmlattributes__"):
+##        print "marshal: found __xmlattributes__"

         xmlattributes = obj.__xmlattributes__
         xmlattributes = obj.__xmlattributes__

-        members_to_skip += xmlattributes
+        members_to_skip.extend(xmlattributes)

         for attr in xmlattributes:
             internalAttrName = attr
         for attr in xmlattributes:
             internalAttrName = attr

+            ifDefPy()

             if (attr.startswith("__") and not attr.endswith("__")): 
                 internalAttrName = classNamePrefix + attr
             if (attr.startswith("__") and not attr.endswith("__")): 
                 internalAttrName = classNamePrefix + attr

+            endIfDef()

             # Fail silently if a python attribute is specified to be
             # an XML attribute but is missing.
             # Fail silently if a python attribute is specified to be
             # an XML attribute but is missing.

-            try:
-                value = obj.__dict__[internalAttrName]
-            except KeyError:
-                continue
-##                # But, check and see if it is a property first:
-##                if (hasPropertyValue(obj, attr)):
-##                    value = getattr(obj, attr)
-##                else:
-##                    continue
+##            print "marshal:   processing attribute ", internalAttrName
+            attrs = obj.__dict__
+            value = attrs.get(internalAttrName)

             xsdElement = None
             xsdElement = None

-            if hasattr(obj, '__xsdcomplextype__'):
-                complexType = getattr(obj, '__xsdcomplextype__')
+            complexType = getComplexType(obj)
+            if (complexType != None):
+##                print "marshal: found __xsdcomplextype__"

                 xsdElement = complexType.findElement(attr)
                 xsdElement = complexType.findElement(attr)

-            if xsdElement:
+            if (xsdElement != None):

                 default = xsdElement.default
                 default = xsdElement.default

-                if default == value or default == _getXmlValue(value):
-                    continue
+                if (default != None):
+                    if ((default == value) or (default == _getXmlValue(value))):
+                        continue
+                else:
+                    if (value == None):
+                        continue

             elif value == None:
                 continue
 
             elif value == None:
                 continue
 


@@ -466,184 +602,205 @@ def marshal(obj, elementName=None, nameSpacePrefix='', nameSpaces=None, prettyPr
                else:
                    value = "false"
 
                else:
                    value = "false"
 

-            attrNameSpacePrefix = ''
-            if hasattr(obj, '__xmlattrnamespaces__'):
-                for nameSpaceKey, nameSpaceAttributes in getattr(obj, '__xmlattrnamespaces__').items():
-                    if nameSpaceKey == nameSpacePrefix[:-1]: # Don't need to specify attribute namespace if it is the same as it selement
+            attrNameSpacePrefix = ""
+            if hasattr(obj, "__xmlattrnamespaces__"):
+##                print "marshal: found __xmlattrnamespaces__"
+                for nameSpaceKey, nameSpaceAttributes in getattr(obj, "__xmlattrnamespaces__").iteritems():
+                    if nameSpaceKey == nameSpacePrefix[:-1]: # Don't need to specify attribute namespace if it is the same as its element

                         continue
                     if attr in nameSpaceAttributes:
                         continue
                     if attr in nameSpaceAttributes:

-                        attrNameSpacePrefix = nameSpaceKey + ':'
+                        attrNameSpacePrefix = nameSpaceKey + ":"

                         break
                         break

-##            if attr.startswith('_'):
+##            if attr.startswith("_"):

 ##                attr = attr[1:]
 ##                attr = attr[1:]

-            if (hasattr(obj, "__xmlrename__") and attr in obj.__xmlrename__):
+            if (hasattr(obj, "__xmlrename__") and attr in asDict(obj.__xmlrename__)):
+##                print "marshal: found __xmlrename__ (and its attribute)"

                 attr = obj.__xmlrename__[attr]
 
                 attr = obj.__xmlrename__[attr]
 

-            objattrs += ' %s%s="%s"' % (attrNameSpacePrefix, attr, value)
+            objattrs += ' %s%s="%s"' % (attrNameSpacePrefix, attr, str(value))
+##            print "marshal:   new objattrs is: ", objattrs

 
 

-    objtype = type(obj)
-    if isinstance(obj, NoneType):
-        return ''
-#        return '%s<%s objtype="None"/>%s' % (prefix, elementName, newline)
+    if (obj == None):
+        xmlString = [""]

     elif isinstance(obj, bool):
     elif isinstance(obj, bool):

-        return '%s<%s objtype="bool">%s</%s>%s' % (prefix, elementName, obj, elementName, newline)
+        xmlString = ['%s<%s objtype="bool">%s</%s>%s' % (prefix, elementName, obj, elementName, newline)]

     elif isinstance(obj, int):
     elif isinstance(obj, int):

-        return '''%s<%s objtype="int">%s</%s>%s''' % (prefix, elementName, str(obj), elementName, newline)
+        xmlString = ['%s<%s objtype="int">%s</%s>%s' % (prefix, elementName, str(obj), elementName, newline)]

     elif isinstance(obj, long):
     elif isinstance(obj, long):

-        return '%s<%s objtype="long">%s</%s>%s' % (prefix, elementName, str(obj), elementName, newline)
+        xmlString = ['%s<%s objtype="long">%s</%s>%s' % (prefix, elementName, str(obj), elementName, newline)]

     elif isinstance(obj, float):
     elif isinstance(obj, float):

-        return '%s<%s objtype="float">%s</%s>%s' % (prefix, elementName, str(obj), elementName, newline)
+        xmlString = ['%s<%s objtype="float">%s</%s>%s' % (prefix, elementName, str(obj), elementName, newline)]
+    elif isinstance(obj, unicode): # have to check before basestring - unicode is instance of base string
+        xmlString = ['%s<%s>%s</%s>%s' % (prefix, elementName, saxutils.escape(obj.encode()), elementName, newline)]

     elif isinstance(obj, basestring):
     elif isinstance(obj, basestring):

-        return '''%s<%s>%s</%s>%s''' % (prefix, elementName, saxutils.escape(obj), elementName, newline)
-##    elif isinstance(obj, unicode):
-##        return '''%s<%s>%s</%s>%s''' % (prefix, elementName, obj, elementName, newline)
+        xmlString = ['%s<%s>%s</%s>%s' % (prefix, elementName, saxutils.escape(obj), elementName, newline)]

     elif isinstance(obj, list):
         if len(obj) < 1:
     elif isinstance(obj, list):
         if len(obj) < 1:

-            return ''
-        xmlString = '%s<%s objtype="list">%s' % (prefix, elementName, newline)
-        for item in obj:
-            xmlString += marshal(item, nameSpaces=nameSpaces, indent=indent+increment)
-        xmlString += '%s</%s>%s' % (prefix, elementName, newline)
-        return xmlString
+            xmlString = ""
+        else:
+            xmlString = ['%s<%s objtype="list">%s' % (prefix, elementName, newline)]
+            for item in obj:
+                xmlString.extend(_marshal(item, nameSpaces=nameSpaces, indent=indent+increment, knownTypes=knownTypes))
+            xmlString.append("%s</%s>%s" % (prefix, elementName, newline))

     elif isinstance(obj, tuple):
         if len(obj) < 1:
     elif isinstance(obj, tuple):
         if len(obj) < 1:

-            return ''
-        xmlString = '%s<%s objtype="list" mutable="false">%s' % (prefix, elementName, newline)
-        for item in obj:
-            xmlString += marshal(item, nameSpaces=nameSpaces, indent=indent+increment)
-        xmlString += '%s</%s>%s' % (prefix, elementName, newline)
-        return xmlString
+            xmlString = ""
+        else:
+            xmlString = ['%s<%s objtype="list" mutable="false">%s' % (prefix, elementName, newline)]
+            for item in obj:
+                xmlString.extend(_marshal(item, nameSpaces=nameSpaces, indent=indent+increment, knownTypes=knownTypes))
+            xmlString.append("%s</%s>%s" % (prefix, elementName, newline))

     elif isinstance(obj, dict):
     elif isinstance(obj, dict):

-        xmlString = '%s<%s objtype="dict">%s' % (prefix, elementName, newline)
-        subprefix = prefix + ' '*increment
+        xmlString = ['%s<%s objtype="dict">%s' % (prefix, elementName, newline)]
+        subprefix = prefix + " "*increment

         subindent = indent + 2*increment
         for key, val in obj.iteritems():
         subindent = indent + 2*increment
         for key, val in obj.iteritems():

-            xmlString += "%s<key>%s%s%s</key>%s%s<value>%s%s%s</value>%s" \
-                         % (subprefix, newline, marshal(key, indent=subindent), subprefix, newline, subprefix, newline, marshal(val, nameSpaces=nameSpaces, indent=subindent), subprefix, newline)
-        xmlString += '%s</%s>%s' % (prefix, elementName, newline)
-        return xmlString
+##            if (isinstance(key, basestring) and key is legal identifier):
+##                xmlString.extend(_marshal(val, elementName=key, nameSpaces=nameSpaces, indent=subindent, knownTypes=knownTypes))
+##            else:
+            xmlString.append("%s<%s>%s" % (subprefix, DICT_ITEM_NAME, newline))
+            xmlString.extend(_marshal(key, elementName=DICT_ITEM_KEY_NAME, indent=subindent, knownTypes=knownTypes))
+            xmlString.extend(_marshal(val, elementName=DICT_ITEM_VALUE_NAME, nameSpaces=nameSpaces, indent=subindent, knownTypes=knownTypes))
+            xmlString.append("%s</%s>%s" % (subprefix, DICT_ITEM_NAME, newline))
+        xmlString.append("%s</%s>%s" % (prefix, elementName, newline))

     else:
     else:

-        moduleName = obj.__class__.__module__
-        if (moduleName == "activegrid.model.schema"):
-            xmlString = '%s<%s%s%s' % (prefix, elementName, nameSpaceAttrs, objattrs)
+        # Only add the objtype if the element tag is unknown to us.
+        objname = knownTypes.get(elementName)
+        if (objname != None):
+            xmlString = ["%s<%s%s%s" % (prefix, elementName, nameSpaceAttrs, objattrs)]

         else:
         else:

-            xmlString = '%s<%s%s%s objtype="%s.%s"' % (prefix, elementName, nameSpaceAttrs, objattrs, moduleName, className)
-        # get the member, value pairs for the object, filtering out
-        # the types we don't support.
-        xmlMemberString = ''
-        if hasattr(obj, '__xmlbody__'):
-            xmlMemberString = getattr(obj, obj.__xmlbody__)
+            xmlString = ['%s<%s%s%s objtype="%s.%s"' % (prefix, elementName, nameSpaceAttrs, objattrs, obj.__class__.__module__, className)]
+        # get the member, value pairs for the object, filtering out the types we don"t support
+        if (elementAdd != None):
+            prefix += increment*" "
+            indent += increment
+            
+        xmlMemberString = []
+        if hasattr(obj, "__xmlbody__"):
+            xmlbody = getattr(obj, obj.__xmlbody__)
+            if xmlbody != None:
+                xmlMemberString.append(xmlbody)           

         else:
         else:

-            entryList = obj.__dict__.items()
-##            # Add in properties
-##            for key in obj.__class__.__dict__.iterkeys():
-##                if (key not in members_to_skip and key not in obj.__dict__
-##                    and hasPropertyValue(obj, key)):
-##                    value = getattr(obj, key)
-##                    entryList.append((key, value))
-            entryList.sort()
-            for name, value in entryList:
-##                # special name handling for private "__*" attributes:
-##                # remove the _<class-name> added by Python
-##                if name.startswith(classNamePrefix): name = name[len(classNamePrefix):]
-                if name in members_to_skip: continue
-                if name.startswith('__') and name.endswith('__'): continue
-##                idx = name.find('__')
-##                if idx > 0:
-##                    newName = name[idx+2:]
-##                    if newName:
-##                        name = newName
-                subElementNameSpacePrefix = nameSpacePrefix
-                if hasattr(obj, '__xmlattrnamespaces__'):
-                    for nameSpaceKey, nameSpaceValues in getattr(obj, '__xmlattrnamespaces__').items():
-                        if name in nameSpaceValues:
-                            subElementNameSpacePrefix = nameSpaceKey + ':'
-                            break
-                # handle sequences listed in __xmlflattensequence__
-                # specially: instead of listing the contained items inside
-                # of a separate list, as god intended, list them inside
-                # the object containing the sequence.
-                if hasattr(obj, '__xmlflattensequence__') and name in obj.__xmlflattensequence__ and value:
-                    try:
+            if hasattr(obj, "__xmlattrgroups__"):
+                attrGroups = obj.__xmlattrgroups__.copy()
+                if (not isinstance(attrGroups, dict)):
+                    raise "__xmlattrgroups__ is not a dict, but must be"
+                for n in attrGroups.iterkeys():
+                    members_to_skip.extend(attrGroups[n])
+            else:
+                attrGroups = {}
+            # add the list of all attributes to attrGroups
+            eList = obj.__dict__.keys()
+            eList.sort()
+            attrGroups["__nogroup__"] = eList
+            
+            for eName, eList in attrGroups.iteritems():
+                if (eName != "__nogroup__"):
+                    prefix += increment*" "
+                    indent += increment
+                    xmlMemberString.append('%s<%s objtype="None">%s' % (prefix, eName, newline))
+                for name in eList:
+                    value = obj.__dict__[name]
+                    if eName == "__nogroup__" and name in members_to_skip: continue
+                    if name.startswith("__") and name.endswith("__"): continue
+                    subElementNameSpacePrefix = nameSpacePrefix
+                    if hasattr(obj, "__xmlattrnamespaces__"):
+                        for nameSpaceKey, nameSpaceValues in getattr(obj, "__xmlattrnamespaces__").iteritems():
+                            if name in nameSpaceValues:
+                                subElementNameSpacePrefix = nameSpaceKey + ":"
+                                break
+                    # handle sequences listed in __xmlflattensequence__
+                    # specially: instead of listing the contained items inside
+                    # of a separate list, as God intended, list them inside
+                    # the object containing the sequence.
+                    if (hasattr(obj, "__xmlflattensequence__") and (value != None) and (name in asDict(obj.__xmlflattensequence__))):

                         xmlnametuple = obj.__xmlflattensequence__[name]
                         xmlnametuple = obj.__xmlflattensequence__[name]

+                        if (xmlnametuple == None):
+                            xmlnametuple = [name]
+                        elif (not isinstance(xmlnametuple, (tuple,list))):
+                            xmlnametuple = [str(xmlnametuple)]

                         xmlname = None
                         xmlname = None

-                        if len(xmlnametuple) == 1:
+                        if (len(xmlnametuple) == 1):

                             xmlname = xmlnametuple[0]
                             xmlname = xmlnametuple[0]

-                    except:
-                        xmlname = name
-##                        xmlname = name.lower()
-                    for seqitem in value:
-                        xmlMemberString += marshal(seqitem, xmlname, subElementNameSpacePrefix, nameSpaces=nameSpaces, indent=indent+increment)
-                else:
-                    if (hasattr(obj, "__xmlrename__") and name in obj.__xmlrename__):
-                        xmlname = obj.__xmlrename__[name]
+##                        ix = 0
+                        for seqitem in value:
+##                            xmlname = xmlnametuple[ix]
+##                            ix += 1
+##                            if (ix >= len(xmlnametuple)):
+##                                ix = 0
+                            xmlMemberString.extend(_marshal(seqitem, xmlname, subElementNameSpacePrefix, nameSpaces=nameSpaces, indent=indent+increment, knownTypes=knownTypes))

                     else:
                     else:

-                        xmlname = name
-##                    xmlname = name.lower()
-##                    # skip
-##                    if xmlname.startswith('_') and not xmlname.startswith('__'):
-##                        xmlname = xmlname[1:]
-##                    if (indent > 30):
-##                        print "getting pretty deep, xmlname = ", xmlname
-                    xmlMemberString += marshal(value, xmlname, subElementNameSpacePrefix, nameSpaces=nameSpaces, indent=indent+increment)
+                        if (hasattr(obj, "__xmlrename__") and name in asDict(obj.__xmlrename__)):
+                            xmlname = obj.__xmlrename__[name]
+                        else:
+                            xmlname = name
+                        xmlMemberString.extend(_marshal(value, xmlname, subElementNameSpacePrefix, nameSpaces=nameSpaces, indent=indent+increment, knownTypes=knownTypes))
+                if (eName != "__nogroup__"):
+                    xmlMemberString.append("%s</%s>%s" % (prefix, eName, newline))
+                    prefix = prefix[:-increment]
+                    indent -= increment
+

         # if we have nested elements, add them here, otherwise close the element tag immediately.
         # if we have nested elements, add them here, otherwise close the element tag immediately.

-        if xmlMemberString:
-            xmlString += '>'
-            if hasattr(obj, '__xmlbody__'):
-                xmlString += xmlMemberString
-                xmlString += '</%s>%s' % (elementName, newline)
+        newList = []
+        for s in xmlMemberString:
+            if (len(s) > 0): newList.append(s)
+        xmlMemberString = newList
+        if len(xmlMemberString) > 0:
+            xmlString.append(">")
+            if hasattr(obj, "__xmlbody__"):
+                xmlString.extend(xmlMemberString)
+                xmlString.append("</%s>%s" % (elementName, newline))

             else:
             else:

-                xmlString += newline
-                xmlString += xmlMemberString
-                xmlString += '%s</%s>%s' % (prefix, elementName, newline)
+                xmlString.append(newline)
+                if (elementAdd != None):
+                    xmlString.append("%s<%s>%s" % (prefix, elementAdd, newline))
+                xmlString.extend(xmlMemberString)
+                if (elementAdd != None):
+                    xmlString.append("%s</%s>%s" % (prefix, elementAdd, newline))
+                    prefix = prefix[:-increment]
+                    indent -= increment
+                xmlString.append("%s</%s>%s" % (prefix, elementName, newline))

         else:
         else:

-            xmlString = xmlString + '/>%s' % newline
-        return xmlString
-
-# We don't use this anymore but in case we want to get properties this is how
-# you do it
-def hasPropertyValue(obj, attr):
-    hasProp = False
-    try:
-        prop = obj.__class__.__dict__[attr]
-        if (isinstance(prop, property)):
-            hasProp = hasattr(obj, attr)
-            if (hasProp):
-                # It's a property and it has a value but sometimes we don't want it.
-                # If there is a _hasattr method execute it and the
-                # result will tell us whether to include this value
-                try:
-                    hasProp = obj._hasattr(attr)
-                except:
-                    pass
-    except KeyError:
-        pass
-    return hasProp
-
-if __name__ == '__main__':
-    from xmlmarshallertests import Person, marshalledint, marshalledlist
-
-    l = [1, 2, 3]
-    d = {'1': 1, '2': 2}
-    outerlist = [l]
-    xmlstr = marshal(d, "d", prettyPrint=True)
-    print xmlstr
-
-    person = Person()
-    person.firstName = "Albert"
-    person.lastName = "Camus"
-    person.addressLine1 = "23 Absurd St."
-    person.city = "Ennui"
-    person.state = "MO"
-    person.zip = "54321"
-    person._phoneNumber = "808-303-2323"
-    person.favoriteWords = ['angst', 'ennui', 'existence']
-    person.weight = 150
-
-    xmlstring = marshal(person, 'person', prettyPrint=True)
-    print xmlstring
-
-    obj = unmarshal(marshalledlist)
-    print "obj has type %s and value %s" % (type(obj), str(obj))
-    for item in obj:
-        print "item: %s" % str(item)
+            xmlString.append("/>%s" % newline)
+##        return xmlString
+    xmlMarshallerLogger.debug("<-- _marshal: %s" % str(xmlString))
+    return xmlString
+
+# A simple test, to be executed when the xmlmarshaller is run standalone
+class MarshallerPerson:
+    __xmlname__ = "person"
+    __xmlexclude__ = ["fabulousness",]
+    __xmlattributes__ = ("nonSmoker",)
+    __xmlrename__ = {"_phoneNumber": "telephone"}
+    __xmlflattensequence__ = {"favoriteWords": ("vocabulary",)}
+    __xmlattrgroups__ = {"name": ["firstName", "lastName"], "address": ["addressLine1", "city", "state", "zip"]}
+
+    def setPerson(self):
+        self.firstName = "Albert"
+        self.lastName = "Camus"
+        self.addressLine1 = "23 Absurd St."
+        self.city = "Ennui"
+        self.state = "MO"
+        self.zip = "54321"
+        self._phoneNumber = "808-303-2323"
+        self.favoriteWords = ["angst", "ennui", "existence"]
+        self.phobias = ["war", "tuberculosis", "cars"]
+        self.weight = 150
+        self.fabulousness = "tres tres"
+        self.nonSmoker = False
+
+if isMain(__name__):
+    p1 = MarshallerPerson()
+    p1.setPerson() 
+    xmlP1 = marshal(p1, prettyPrint=True, encoding="utf-8")        
+    print "\n########################"
+    print   "# testPerson test case #"
+    print   "########################"
+    print xmlP1
+    p2 = unmarshal(xmlP1)
+    xmlP2 = marshal(p2, prettyPrint=True, encoding="utf-8")
+    if xmlP1 == xmlP2:
+        print "Success: repeated marshalling yields identical results"
+    else:
+        print "Failure: repeated marshalling yields different results"
+        print xmlP2