- fix errors

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

diff --git a/PLC/Methods/AdmUpdateNodeGroup.py b/PLC/Methods/AdmUpdateNodeGroup.py
index 90aae6e..b53198a 100644 (file)
--- a/PLC/Methods/AdmUpdateNodeGroup.py
+++ b/PLC/Methods/AdmUpdateNodeGroup.py
@@ -2,19 +2,18 @@ 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 PasswordAuth
+from PLC.Auth import Auth
+from PLC.Methods.UpdateNodeGroup import UpdateNodeGroup
 
-class AdmUpdateNodeGroup(Method):
+class AdmUpdateNodeGroup(UpdateNodeGroup):
     """
-    Updates a custom node group.
-     
-    Returns 1 if successful, faults otherwise.
+    Deprecated. See UpdateNodeGroup.
     """
 
-    roles = ['admin']
+    status = "deprecated"
 
     accepts = [
-        PasswordAuth(),
+        Auth(),
         Mixed(NodeGroup.fields['nodegroup_id'],
              NodeGroup.fields['name']),
         NodeGroup.fields['name'],
@@ -24,20 +23,5 @@ class AdmUpdateNodeGroup(Method):
     returns = Parameter(int, '1 if successful')
 
     def call(self, auth, nodegroup_id_or_name, name, description):
-       # Get nodegroup information
-       nodegroups = NodeGroups(self.api, [nodegroup_id_or_name])
-       if not nodegroups:
-            raise PLCInvalidArgument, "No such nodegroup"
-
-       nodegroup = nodegroups.values()[0]
-       
-       # Modify node group
-        update_fields = {
-            'name': name,
-            'description': description
-            }
-
-       nodegroup.update(update_fields)
-        nodegroup.flush()
-
-        return 1
+        return UpdateNodeGroup.call(self, auth, nodegroup_id_or_name,
+                                    {'name': name, 'description': description})