commenting out normalize(), should no longer need it

[sfa.git] / sfa / plc / plcsfaapi.py

diff --git a/sfa/plc/plcsfaapi.py b/sfa/plc/plcsfaapi.py
index e136053..a1712e5 100644 (file)
--- a/sfa/plc/plcsfaapi.py
+++ b/sfa/plc/plcsfaapi.py
@@ -234,7 +234,7 @@ class PlcSfaApi(SfaApi):
         for record in records:
             if 'site_id' in record:
                 site_ids.append(record['site_id'])
-            if 'site_ids' in records:
+            if 'site_ids' in record:
                 site_ids.extend(record['site_ids'])
             if 'person_ids' in record:
                 person_ids.extend(record['person_ids'])
@@ -428,14 +428,14 @@ class PlcSfaApi(SfaApi):
         # xxx ugly hack - somehow we receive here a list of {'text':value} 
         # instead of an expected list of strings
         # please remove once this is issue is cleanly fixed
-        def normalize (value):
-            from types import StringTypes
-            if isinstance(value,StringTypes): return value
-            elif isinstance(value,dict): 
-                newvalue=value['text']
-                logger.info("Normalizing %s=>%s"%(value,newvalue))
-                return newvalue
-        newList=[normalize(v) for v in newList]
+        #def normalize (value):
+        #    from types import StringTypes
+        #    if isinstance(value,StringTypes): return value
+        #    elif isinstance(value,dict): 
+        #        newvalue=value['text']
+        #        logger.info("Normalizing %s=>%s"%(value,newvalue))
+        #        return newvalue
+        #newList=[normalize(v) for v in newList]
 
         # if the lists are the same, then we don't have to update anything
         if (oldList == newList):