X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FUpdatePerson.py;h=816f98e780c0c6e7c5e678719f5e2f5fc59a12be;hb=fbc8feb0a370d9da9088f84b18a458a029d0f979;hp=7f769f5fb8a878a629836b884dd99bd0a24139aa;hpb=e347fc823bbba9d88a3fddf07d5c21024dfd1e55;p=plcapi.git

diff --git a/PLC/Methods/UpdatePerson.py b/PLC/Methods/UpdatePerson.py
index 7f769f5..816f98e 100644
--- a/PLC/Methods/UpdatePerson.py
+++ b/PLC/Methods/UpdatePerson.py
@@ -40,9 +40,11 @@ class UpdatePerson(Method):
         persons = Persons(self.api, [person_id_or_email])
         if not persons:
             raise PLCInvalidArgument, "No such account"
-
         person = persons[0]
 
+        if person['peer_id'] is not None:
+            raise PLCInvalidArgument, "Not a local account"
+
         # Authenticated function
         assert self.caller is not None
 
@@ -52,5 +54,16 @@ class UpdatePerson(Method):
 
         person.update(person_fields)
         person.sync()
+	
+	# Logging variables
+	self.event_objects = {'Person': [person['person_id']]}
+
+        # Redact password
+        if 'password' in person_fields:
+            person_fields['password'] = "Removed by API"
+        self.message = 'Person %d updated: %s.' % \
+                       (person['person_id'], person_fields.keys())
+	if 'enabled' in person_fields:
+            self.message += ' Person enabled' 	
 
         return 1