git://git.onelab.eu
/
sfa.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
a9cf6f1
)
use set instead of list to avoid potential duplicate entries
author
Tony Mack
<tmack@paris.CS.Princeton.EDU>
Wed, 9 Jan 2013 02:12:01 +0000
(21:12 -0500)
committer
Tony Mack
<tmack@paris.CS.Princeton.EDU>
Wed, 9 Jan 2013 02:12:01 +0000
(21:12 -0500)
sfa/importer/plimporter.py
patch
|
blob
|
history
diff --git
a/sfa/importer/plimporter.py
b/sfa/importer/plimporter.py
index
4b5c7c1
..
0a485e4
100644
(file)
--- a/
sfa/importer/plimporter.py
+++ b/
sfa/importer/plimporter.py
@@
-250,7
+250,7
@@
class PlImporter:
pass
node_record.stale=False
pass
node_record.stale=False
- site_pis=
[]
+ site_pis=
set()
# import persons
for person_id in site['person_ids']:
try:
# 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']:
# this is valid for all sites she is in..
# PI is coded with role_id==20
if 20 in person['role_ids']:
- site_pis.a
ppen
d (user_record)
+ site_pis.a
d
d (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
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
dbsession.commit()
# import slices