From: Mark Huang <mlhuang@cs.princeton.edu>
Date: Thu, 7 Sep 2006 23:49:01 +0000 (+0000)
Subject: - use Person.remove_role
X-Git-Tag: pycurl-7_13_1~779
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=78e90d38a2f56adc97bfd91d01eb9b3e3bfde646;p=plcapi.git

- use Person.remove_role
---

diff --git a/PLC/Methods/AdmRevokeRoleFromPerson.py b/PLC/Methods/AdmRevokeRoleFromPerson.py
index 61dea2f0..61dbda51 100644
--- a/PLC/Methods/AdmRevokeRoleFromPerson.py
+++ b/PLC/Methods/AdmRevokeRoleFromPerson.py
@@ -52,10 +52,6 @@ class AdmRevokeRoleFromPerson(Method):
             raise PLCPermissionDenied, "Not allowed to revoke that role"
 
         if role_id in person['role_ids']:
-            person_id = person['person_id']
-            self.api.db.do("DELETE FROM person_roles" \
-                           " WHERE person_id = %(person_id)d" \
-                           " AND role_id = %(role_id)d",
-                           locals())
+            person.remove_role(role_id)
 
         return 1