From: Mark Huang <mlhuang@cs.princeton.edu>
Date: Wed, 11 Oct 2006 17:20:24 +0000 (+0000)
Subject: - valid_fields is a list
X-Git-Tag: pycurl-7_13_1~608
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=1eadaf32074b1e3c987da135728f8d81fe8763fd;p=plcapi.git

- valid_fields is a list
---

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"