From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Wed, 12 Dec 2012 08:59:42 +0000 (+0100)
Subject: AddPerson is more picky
X-Git-Tag: sfa-3.0-0~45
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=e694a97b15e64f0141119bd68502e486d7fc9dec;p=sfa.git

AddPerson is more picky
---

diff --git a/sfa/planetlab/pldriver.py b/sfa/planetlab/pldriver.py
index db9a2ddf..c47d170c 100644
--- a/sfa/planetlab/pldriver.py
+++ b/sfa/planetlab/pldriver.py
@@ -91,7 +91,10 @@ class PlDriver (Driver):
             if not persons:
                 for key in ['first_name','last_name']:
                     if key not in sfa_record: sfa_record[key]='*from*sfa*'
-                pointer = self.shell.AddPerson(dict(sfa_record))
+                # AddPerson does not allow everything to be set
+                can_add = ['first_name', 'last_name', 'title','email', 'password', 'phone', 'url', 'bio']
+                add_person_dict=dict ( [ (k,sfa_record[k]) for k in sfa_record if k in can_add ] )
+                pointer = self.shell.AddPerson(add_person_dict)
             else:
                 pointer = persons[0]['person_id']