- accept message_filter and return_fields

[plcapi.git] / PLC / Methods / UpdateAddress.py

diff --git a/PLC/Methods/UpdateAddress.py b/PLC/Methods/UpdateAddress.py
index e6885d9..3573573 100644 (file)
--- a/PLC/Methods/UpdateAddress.py
+++ b/PLC/Methods/UpdateAddress.py
@@ -2,7 +2,7 @@ from PLC.Faults import *
 from PLC.Method import Method
 from PLC.Parameter import Parameter, Mixed
 from PLC.Addresses import Address, Addresses
-from PLC.Auth import PasswordAuth
+from PLC.Auth import Auth
 
 can_update = lambda (field, value): field in \
              ['line1', 'line2', 'line3',
@@ -21,11 +21,9 @@ class UpdateAddress(Method):
     roles = ['admin', 'pi']
 
     address_fields = dict(filter(can_update, Address.fields.items()))
-    for field in address_fields.values():
-        field.optional = True
 
     accepts = [
-        PasswordAuth(),
+        Auth(),
         Address.fields['address_id'],
         address_fields
         ]
@@ -36,7 +34,7 @@ class UpdateAddress(Method):
         address_fields = dict(filter(can_update, address_fields.items()))
 
         # Get associated address details
-        addresses = Addresses(self.api, [address_id]).values()
+        addresses = Addresses(self.api, [address_id])
         if not addresses:
             raise PLCInvalidArgument, "No such address"
         address = addresses[0]
@@ -47,5 +45,10 @@ class UpdateAddress(Method):
 
         address.update(address_fields)
         address.sync()
-
-        return 1
+       
+       # Logging variables
+       self.object_ids = [address['address_id']]
+       self.message = 'Address %d updated: %s' % \
+               (address['address_id'], ", ".join(address_fields.keys()))
+        
+       return 1