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-2.1-22~3
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=eb9e9478fc18f70c18a3279a4bf0d46e31785a39;p=sfa.git

AddPerson is more picky
---

diff --git a/sfa/planetlab/pldriver.py b/sfa/planetlab/pldriver.py
index 9274f39e..c3d14ea0 100644
--- a/sfa/planetlab/pldriver.py
+++ b/sfa/planetlab/pldriver.py
@@ -94,7 +94,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']