X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FUpdateNodeGroup.py;h=6b8018752c5472404f41775935deb015c58aa874;hb=bd0cbf4f7f2e4cf7ceda500bfa6f98c0a700018b;hp=c2853666653b4586bbc5d3ecea7c9b64635765eb;hpb=77e4f177dfee85705c36298c3230b2e4a3e73467;p=plcapi.git diff --git a/PLC/Methods/UpdateNodeGroup.py b/PLC/Methods/UpdateNodeGroup.py index c285366..6b80187 100644 --- a/PLC/Methods/UpdateNodeGroup.py +++ b/PLC/Methods/UpdateNodeGroup.py @@ -1,16 +1,15 @@ -# $Id$ from PLC.Faults import * from PLC.Method import Method from PLC.Parameter import Parameter, Mixed from PLC.NodeGroups import NodeGroup, NodeGroups from PLC.Auth import Auth -can_update = lambda (field, value): field in ['groupname','tagvalue'] +can_update = lambda (field, value): field in ['groupname','value'] class UpdateNodeGroup(Method): """ Updates a custom node group. - + Returns 1 if successful, faults otherwise. """ @@ -21,7 +20,7 @@ class UpdateNodeGroup(Method): accepts = [ Auth(), Mixed(NodeGroup.fields['nodegroup_id'], - NodeGroup.fields['groupname']), + NodeGroup.fields['groupname']), nodegroup_fields ] @@ -30,17 +29,17 @@ class UpdateNodeGroup(Method): def call(self, auth, nodegroup_id_or_name, nodegroup_fields): nodegroup_fields = dict(filter(can_update, nodegroup_fields.items())) - # Get nodegroup information - nodegroups = NodeGroups(self.api, [nodegroup_id_or_name]) - if not nodegroups: + # Get nodegroup information + nodegroups = NodeGroups(self.api, [nodegroup_id_or_name]) + if not nodegroups: raise PLCInvalidArgument, "No such nodegroup %r"%nodegroup_id_or_name - nodegroup = nodegroups[0] + nodegroup = nodegroups[0] - nodegroup.update(nodegroup_fields) + nodegroup.update(nodegroup_fields) nodegroup.sync() - - # Logging variables - self.event_objects = {'NodeGroup': [nodegroup['nodegroup_id']]} - self.message = 'Node group %d updated: %s' % \ - (nodegroup['nodegroup_id'], ", ".join(nodegroup_fields.keys())) + + # Logging variables + self.event_objects = {'NodeGroup': [nodegroup['nodegroup_id']]} + self.message = 'Node group %d updated: %s' % \ + (nodegroup['nodegroup_id'], ", ".join(nodegroup_fields.keys())) return 1