X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FAddRoleToPerson.py;h=3fc1df3d3b4171c3b1fa8dff4d815c32e9a2501f;hb=567b2eb0fcdf0d9c5e8024afdaaf30ea95bba0cf;hp=1e47033d176bf978a901ab6ac30fc4e17d588a06;hpb=ea6f96626a5049706f5946e45b74c7d0dd20222a;p=plcapi.git

diff --git a/PLC/Methods/AddRoleToPerson.py b/PLC/Methods/AddRoleToPerson.py
index 1e47033..3fc1df3 100644
--- a/PLC/Methods/AddRoleToPerson.py
+++ b/PLC/Methods/AddRoleToPerson.py
@@ -19,7 +19,7 @@ class AddRoleToPerson(Method):
 
     accepts = [
         Auth(),
-        Mixed(Role.fields['role_id'],
+        Mixed(Role.fields['id'],
               Role.fields['name']),
         Mixed(Person.fields['person_id'],
               Person.fields['email']),
@@ -40,9 +40,6 @@ class AddRoleToPerson(Method):
             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
 
@@ -58,9 +55,4 @@ class AddRoleToPerson(Method):
         if role['role_id'] not in person['role_ids']:
             person.add_role(role)
 
-        self.event_objects = {'Person': [person['person_id']],
-                              'Role': [role['role_id']]}
-        self.message = "Role %d granted to person %d" % \
-                       (role['role_id'], person['person_id'])
-
         return 1