only delete the person from plc if he/she has site ids. If he/she doesnt it probably...

[sfa.git] / sfa / methods / remove.py

diff --git a/sfa/methods/remove.py b/sfa/methods/remove.py
index c0f5c52..dade3fb 100644 (file)
--- a/sfa/methods/remove.py
+++ b/sfa/methods/remove.py
@@ -42,7 +42,10 @@ class remove(Method):
         
         type = record['type']
         if type == "user":
-            if self.api.plshell.GetPersons(self.api.plauth, record.get_pointer()):
+            persons = self.api.plshell.GetPersons(self.api.plauth, record.get_pointer())
+            # only delete this person if he has site ids. if he doesnt, it probably means 
+            # he was just removed from a site, not actually deleted
+            if persons and persons[0]['site_ids']:
                 self.api.plshell.DeletePerson(self.api.plauth, record.get_pointer())
         elif type == "slice":
             if self.api.plshell.GetSlices(self.api.plauth, record.get_pointer()):