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)
 
         # sync with the geni db
         sfa = SFA(self.api)

-        sfa.update_record(person, 'person') 
+        sfa.update_record(person, 'user') 

         
         return key['key_id']
         
         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.Persons import Person, Persons
 from PLC.Sites import Site, Sites
 from PLC.Auth import Auth

+from PLC.SFA import SFA

 
 class AddPersonToSite(Method):
     """
 
 class AddPersonToSite(Method):
     """


@@ -50,9 +51,13 @@ class AddPersonToSite(Method):
         if site['site_id'] not in person['site_ids']:
             site.add_person(person)
 
         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
         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.Persons import Person, Persons
 from PLC.Sites import Site, Sites
 from PLC.Auth import Auth

+from PLC.SFA import SFA

 
 class DeletePersonFromSite(Method):
     """
 
 class DeletePersonFromSite(Method):
     """


@@ -50,9 +51,13 @@ class DeletePersonFromSite(Method):
         if site['site_id'] in person['site_ids']:
             site.remove_person(person)
 
         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
         return 1