From 1eadaf32074b1e3c987da135728f8d81fe8763fd Mon Sep 17 00:00:00 2001 From: Mark Huang Date: Wed, 11 Oct 2006 17:20:24 +0000 Subject: [PATCH] - valid_fields is a list --- PLC/Methods/UpdatePerson.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/PLC/Methods/UpdatePerson.py b/PLC/Methods/UpdatePerson.py index d45a6044..5e84f7d5 100644 --- a/PLC/Methods/UpdatePerson.py +++ b/PLC/Methods/UpdatePerson.py @@ -38,11 +38,12 @@ class UpdatePerson(Method): returns = Parameter(int, '1 if successful') def call(self, auth, person_id_or_email, update_fields): - valid_fields = self.update_fields + valid_fields = self.update_fields.keys() + # Remove admin only fields if 'admin' not in self.caller['roles']: - for key in ['enabled']: - valid_fields.remove(key) + for key in ['enabled']: + valid_fields.remove(key) if filter(lambda field: field not in valid_fields, update_fields): raise PLCInvalidArgument, "Invalid field specified" -- 2.47.0