hasattr() second arg must be a string
authorTony Mack <tmack@paris.CS.Princeton.EDU>
Tue, 15 Nov 2011 03:29:27 +0000 (22:29 -0500)
committerTony Mack <tmack@paris.CS.Princeton.EDU>
Tue, 15 Nov 2011 21:25:00 +0000 (16:25 -0500)
sfa/rspecs/elements/element.py

index 6f15d60..5789a9c 100644 (file)
@@ -20,9 +20,8 @@ class Element(dict):
         """
         if not element_class:
             element_class = Element
-# xxx this obviously is wrong...
-#        if not fields and hasattr(element_class, fields):
-#           fields = element_class.fields
+        if not fields and hasattr(element_class, 'fields'):
+           fields = element_class.fields
         elems = xml.xpath(xpath)
         objs = []
         for elem in elems:
@@ -34,11 +33,8 @@ class Element(dict):
                     if field in elem.attrib:
                         obj[field] = elem.attrib[field]    
             objs.append(obj)
-# xxx seems unused
-        generic_elems = [element_class(elem.attrib, elem) for elem in elems]
         return objs
 
-
     @staticmethod
     def add_elements(xml, name, objs, fields=None):
         """