assert primary key exists in record before attempting to delete it
[plcapi.git] / PLC / NodeGroups.py
index 3915590..bbe20a8 100644 (file)
@@ -1,19 +1,14 @@
 #
 # Functions for interacting with the nodegroups table in the database
 #
-# Mark Huang <mlhuang@cs.princeton.edu>
-# Copyright (C) 2006 The Trustees of Princeton University
-#
-# $Id$
 #
 
 from types import StringTypes
 
 from PLC.Faults import *
 from PLC.Parameter import Parameter, Mixed
-from PLC.Filter import Filter
 from PLC.Debug import profile
-from PLC.Table import Row, Table
+from PLC.Storage.AlchemyObject import AlchemyObj
 from PLC.Nodes import Node, Nodes
 
 class NodeGroup(Row):
@@ -23,82 +18,67 @@ class NodeGroup(Row):
     dict. Commit to the database with sync().
     """
 
-    table_name = 'nodegroups'
-    primary_key = 'nodegroup_id'
+    tablename = 'nodegroups'
     join_tables = ['conf_file_nodegroup']
-    primary_field = 'nodegroup_id'
     fields = {
-        'nodegroup_id': Parameter(int, "Node group identifier"),
+        'nodegroup_id': Parameter(int, "Node group identifier", primary_key),
         'groupname': Parameter(str, "Node group name", max = 50),
         'tag_type_id': Parameter (int, "Node tag type id"),
         'value' : Parameter(str, "value that the nodegroup definition is based upon"),
         'tagname' : Parameter(str, "Tag name that the nodegroup definition is based upon"),
-        'conf_file_ids': Parameter([int], "List of configuration files specific to this node group"),
-        'node_ids' : Parameter([int], "List of node_ids that belong to this nodegroup"),
-       }
+        'conf_file_ids': Parameter([int], "List of configuration files specific to this node group", joined=True),
+        'node_ids' : Parameter([int], "List of node_ids that belong to this nodegroup", joined=True),
+        }
     related_fields = {
         }
 
     def validate_name(self, name):
-       # Make sure name is not blank
+        # Make sure name is not blank
         if not len(name):
-                raise PLCInvalidArgument, "Invalid node group name"
-       
-       # Make sure node group does not alredy exist
-       conflicts = NodeGroups(self.api, [name])
-       for nodegroup in conflicts:
+            raise PLCInvalidArgument, "Invalid node group name"
+
+        # Make sure node group does not alredy exist
+        conflicts = NodeGroups(self.api, name)
+        for nodegroup in conflicts:
             if 'nodegroup_id' not in self or self['nodegroup_id'] != nodegroup['nodegroup_id']:
-               raise PLCInvalidArgument, "Node group name already in use"
+                raise PLCInvalidArgument, "Node group name already in use"
 
-       return name
+        return name
 
-    def associate_conf_files(self, auth, field, value):
-        """
-        Add conf_files found in value list (AddConfFileToNodeGroup)
-        Delets conf_files not found in value list (DeleteConfFileFromNodeGroup)
-        """
+    def sync(self, commit=True, validate=True):
+        AlchemyObj.sync(self, commit, validate)
+        if 'nodegroup_id' not in self:
+            AlchemyObj.insert(self, dict(self))
+        else:
+            AlchemyObj.update(self, {'nodegroup_id': self['nodegroup_id']}, dict(self))
 
-        assert 'conf_file_ids' in self
+    def delete(self, commit=True):
         assert 'nodegroup_id' in self
-        assert isinstance(value, list)
-
-        conf_file_ids = self.separate_types(value)[0]
-
-        if self['conf_file_ids'] != conf_file_ids:
-            from PLC.Methods.AddConfFileToNodeGroup import AddConfFileToNodeGroup
-            from PLC.Methods.DeleteConfFileFromNodeGroup import DeleteConfFileFromNodeGroup
-            new_conf_files = set(conf_file_ids).difference(self['conf_file_ids'])
-            stale_conf_files = set(self['conf_file_ids']).difference(conf_file_ids)
+        AlchemyObj.delete(self, dict(self))
 
-            for new_conf_file in new_conf_files:
-                AddConfFileToNodeGroup.__call__(AddConfFileToNodeGroup(self.api), 
-                                                auth, new_conf_file, self['nodegroup_id'])
-            for stale_conf_file in stale_conf_files:
-                DeleteConfFileFromNodeGroup.__call__(DeleteConfFileFromNodeGroup(self.api), 
-                                                     auth, stale_conf_file, self['nodegroup_id'])
-
-
-class NodeGroups(Table):
+class NodeGroups(list):
     """
     Representation of row(s) from the nodegroups table in the
     database.
     """
 
     def __init__(self, api, nodegroup_filter = None, columns = None):
-        Table.__init__(self, api, NodeGroup, columns)
-
-        sql = "SELECT %s FROM view_nodegroups WHERE True" % \
-              ", ".join(self.columns)
-
-        if nodegroup_filter is not None:
-            if isinstance(nodegroup_filter, (list, tuple, set)):
-                # Separate the list into integers and strings
-                ints = filter(lambda x: isinstance(x, (int, long)), nodegroup_filter)
-                strs = filter(lambda x: isinstance(x, StringTypes), nodegroup_filter)
-                nodegroup_filter = Filter(NodeGroup.fields, {'nodegroup_id': ints, 'groupname': strs})
-                sql += " AND (%s) %s" % nodegroup_filter.sql(api, "OR")
-            elif isinstance(nodegroup_filter, dict):
-                nodegroup_filter = Filter(NodeGroup.fields, nodegroup_filter)
-                sql += " AND (%s) %s" % nodegroup_filter.sql(api, "AND")
 
-        self.selectall(sql)
+        if not nodegroup_filter is not None:
+            nodegroups = NodeGroup().select()
+        if isinstance(nodegroup_filter, (list, tuple, set)):
+            # Separate the list into integers and strings
+            ints = filter(lambda x: isinstance(x, (int, long)), nodegroup_filter)
+            strs = filter(lambda x: isinstance(x, StringTypes), nodegroup_filter)
+            nodegroups = NodeGroup().select(filter={'nodegroup_id': ints, 'groupname': strs})
+        elif isinstance(nodegroup_filter, dict):
+            nodegroups = NodeGroup().select(filter=nodegroup_filter)
+        elif isinstance(nodegroup_filter, (int, long)):
+            nodegroups = NodeGroup().select(filter={'nodegroup_id': nodegroup_filter})
+        elif isinstance(nodegroup_filter, StringTypes):
+            nodegroups = NodeGroup().select(filter={'groupname': nodegroup_filter})
+        else:
+            raise PLCInvalidArgument, "Wrong node group filter %r"%nodegroup_filter
+
+        for nodegroup in nodegroups:
+            self.append(nodegroup)