- Initial checkin of new API implementation

[plcapi.git] / PLC / Methods / DeletePersonFromSite.py

diff --git a/PLC/Methods/DeletePersonFromSite.py b/PLC/Methods/DeletePersonFromSite.py
index 4479183..b497463 100644 (file)
--- a/PLC/Methods/DeletePersonFromSite.py
+++ b/PLC/Methods/DeletePersonFromSite.py
@@ -25,6 +25,7 @@ class DeletePersonFromSite(Method):
         ]
 
     returns = Parameter(int, '1 if successful')
+    
 
     def call(self, auth, person_id_or_email, site_id_or_login_base):
         # Get account information
@@ -32,16 +33,21 @@ class DeletePersonFromSite(Method):
         if not persons:
             raise PLCInvalidArgument, "No such account"
 
-        person = persons.values()[0]
+        person = persons[0]
 
         # Get site information
         sites = Sites(self.api, [site_id_or_login_base])
         if not sites:
             raise PLCInvalidArgument, "No such site"
 
-        site = sites.values()[0]
+        site = sites[0]
+       PLCCheckLocalSite(site,"DeletePersonFromSite")
 
         if site['site_id'] in person['site_ids']:
             site.remove_person(person)
 
+       # Logging variables
+       self.object_ids = [site['site_id']]
+       self.message = 'Person %d deleted from site %d  ' % \
+               (person['person_id'], site['site_id'])
         return 1