From 40cb672d9937574f53f87e177d67e82f3659b832 Mon Sep 17 00:00:00 2001 From: Tony Mack Date: Tue, 8 Jan 2013 21:12:01 -0500 Subject: [PATCH] use set instead of list to avoid potential duplicate entries --- sfa/importer/plimporter.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sfa/importer/plimporter.py b/sfa/importer/plimporter.py index 4b5c7c1d..0a485e4a 100644 --- a/sfa/importer/plimporter.py +++ b/sfa/importer/plimporter.py @@ -250,7 +250,7 @@ class PlImporter: pass node_record.stale=False - site_pis=[] + site_pis=set() # import persons for person_id in site['person_ids']: try: @@ -330,12 +330,12 @@ class PlImporter: # this is valid for all sites she is in.. # PI is coded with role_id==20 if 20 in person['role_ids']: - site_pis.append (user_record) + site_pis.add (user_record) except: self.logger.log_exc("PlImporter: failed to import person %d %s"%(person['person_id'],person['email'])) # maintain the list of PIs for a given site - site_record.reg_pis = site_pis + site_record.reg_pis = list(site_pis) dbsession.commit() # import slices -- 2.43.0