added sfa/geniwrapper upcalls to AddPersonToSite and DeletePersonFrimSite

diff --git a/PLC/Methods/AddPersonKey.py b/PLC/Methods/AddPersonKey.py
index 335b403..79cdfc6 100644 (file)
--- a/PLC/Methods/AddPersonKey.py
+++ b/PLC/Methods/AddPersonKey.py
@@ -61,6 +61,6 @@ class AddPersonKey(Method):
 
         # sync with the geni db
         sfa = SFA(self.api)
-        sfa.update_record(person, 'person') 
+        sfa.update_record(person, 'user') 
         
         return key['key_id']

diff --git a/PLC/Methods/AddPersonToSite.py b/PLC/Methods/AddPersonToSite.py
index e4d3aa4..e208b69 100644 (file)
--- a/PLC/Methods/AddPersonToSite.py
+++ b/PLC/Methods/AddPersonToSite.py
@@ -6,6 +6,7 @@ from PLC.Parameter import Parameter, Mixed
 from PLC.Persons import Person, Persons
 from PLC.Sites import Site, Sites
 from PLC.Auth import Auth
+from PLC.SFA import SFA
 
 class AddPersonToSite(Method):
     """
@@ -50,9 +51,13 @@ class AddPersonToSite(Method):
         if site['site_id'] not in person['site_ids']:
             site.add_person(person)
 
-       # Logging variables
-       self.event_objects = {'Site': [site['site_id']],
-                             'Person': [person['person_id']]}
-       self.message = 'Person %d added to site %d' % \
-               (person['person_id'], site['site_id'])
+        # Logging variables
+        self.event_objects = {'Site': [site['site_id']],
+                              'Person': [person['person_id']]}
+        self.message = 'Person %d added to site %d' % \
+                       (person['person_id'], site['site_id'])
+
+        sfa = SFA(self.api)
+        self.update_record(person, 'user')
+
         return 1

diff --git a/PLC/Methods/DeletePersonFromSite.py b/PLC/Methods/DeletePersonFromSite.py
index 58df556..7400dc3 100644 (file)
--- a/PLC/Methods/DeletePersonFromSite.py
+++ b/PLC/Methods/DeletePersonFromSite.py
@@ -6,6 +6,7 @@ from PLC.Parameter import Parameter, Mixed
 from PLC.Persons import Person, Persons
 from PLC.Sites import Site, Sites
 from PLC.Auth import Auth
+from PLC.SFA import SFA
 
 class DeletePersonFromSite(Method):
     """
@@ -50,9 +51,13 @@ class DeletePersonFromSite(Method):
         if site['site_id'] in person['site_ids']:
             site.remove_person(person)
 
-       # Logging variables
-       self.event_objects = {'Site': [site['site_id']],
-                             'Person': [person['person_id']]}  
-       self.message = 'Person %d deleted from site %d  ' % \
-               (person['person_id'], site['site_id'])
+        # Logging variables
+        self.event_objects = {'Site': [site['site_id']],
+                              'Person': [person['person_id']]} 
+        self.message = 'Person %d deleted from site %d ' % \
+                        (person['person_id'], site['site_id'])
+
+        sfa = SFA(self.api)
+        sfa.delete_record(person, 'user')
+
         return 1