From e6caa9640584691b00a565c7a20309a1fb3fd579 Mon Sep 17 00:00:00 2001
From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Mon, 1 Oct 2012 23:03:23 -0400
Subject: [PATCH] implement sessions

---
 PLC/Sessions.py | 68 ++++++++++++++++++-------------------------------
 1 file changed, 25 insertions(+), 43 deletions(-)

diff --git a/PLC/Sessions.py b/PLC/Sessions.py
index 6a03068b..18b935c4 100644
--- a/PLC/Sessions.py
+++ b/PLC/Sessions.py
@@ -5,23 +5,21 @@ import time
 
 from PLC.Faults import *
 from PLC.Parameter import Parameter
-from PLC.Filter import Filter
 from PLC.Debug import profile
-from PLC.Table import Row, Table
+from PLC.Storage.AlchemyObject import AlchemyObj
 from PLC.Persons import Person, Persons
 from PLC.Nodes import Node, Nodes
 
-class Session(Row):
+class Session(AlchemyObj):
     """
     Representation of a row in the sessions table. To use, instantiate
     with a dict of values.
     """
 
-    table_name = 'sessions'
-    primary_key = 'session_id'
+    tablename = 'sessions'
     join_tables = ['person_session', 'node_session']
     fields = {
-        'session_id': Parameter(str, "Session key"),
+        'session_id': Parameter(str, "Session key", primary_key=True),
         'person_id': Parameter(int, "Account identifier, if applicable"),
         'node_id': Parameter(int, "Node identifier, if applicable"),
         'expires': Parameter(int, "Date and time when session expires, in seconds since UNIX epoch"),
@@ -33,17 +31,10 @@ class Session(Row):
 
         return time.strftime("%Y-%m-%d %H:%M:%S", time.gmtime(expires))
 
-    add_person = Row.add_object(Person, 'person_session')
+    #add_person = Row.add_object(Person, 'person_session')
 
-    def add_node(self, node, commit = True):
-        # Nodes can have only one session at a time
-        self.api.db.do("DELETE FROM node_session WHERE node_id = %d" % \
-                       node['node_id'])
-
-        add = Row.add_object(Node, 'node_session')
-        add(self, node, commit = commit)
-
-    def sync(self, commit = True, insert = None):
+    def sync(self, commit = True, insert = None, validate=True):
+        AlchemyObj.sync(commit=commit, validate=validate)
         if not self.has_key('session_id'):
             # Before a new session is added, delete expired sessions
             expired = Sessions(self.api, expires = -int(time.time()))
@@ -55,39 +46,30 @@ class Session(Row):
             # Base64 encode their string representation
             self['session_id'] = base64.b64encode("".join(map(chr, bytes)))
             # Force insert
-            insert = True
-
-        Row.sync(self, commit, insert)
+            AlchemyObj.insert(self, dict(self))
 
-class Sessions(Table):
+class Sessions(list):
     """
     Representation of row(s) from the session table in the database.
     """
 
     def __init__(self, api, session_filter = None, expires = int(time.time())):
-        Table.__init__(self, api, Session)
-
-        sql = "SELECT %s FROM view_sessions WHERE True" % \
-              ", ".join(Session.fields)
-
         if session_filter is not None:
-            if isinstance(session_filter, (list, tuple, set)):
-                # Separate the list into integers and strings
-                ints = filter(lambda x: isinstance(x, (int, long)), session_filter)
-                strs = filter(lambda x: isinstance(x, StringTypes), session_filter)
-                session_filter = Filter(Session.fields, {'person_id': ints, 'session_id': strs})
-                sql += " AND (%s) %s" % session_filter.sql(api, "OR")
-            elif isinstance(session_filter, dict):
-                session_filter = Filter(Session.fields, session_filter)
-                sql += " AND (%s) %s" % session_filter.sql(api, "AND")
-            elif isinstance(session_filter, (int, long)):
-                session_filter = Filter(Session.fields, {'person_id': session_filter})
-                sql += " AND (%s) %s" % session_filter.sql(api, "AND")
-            elif isinstance(session_filter, StringTypes):
-                session_filter = Filter(Session.fields, {'session_id': session_filter})
-                sql += " AND (%s) %s" % session_filter.sql(api, "AND")
-            else:
-                raise PLCInvalidArgument, "Wrong session filter"%session_filter
+            sessions = Session().select()
+        if isinstance(session_filter, (list, tuple, set)):
+            # Separate the list into integers and strings
+            ints = filter(lambda x: isinstance(x, (int, long)), session_filter)
+            strs = filter(lambda x: isinstance(x, StringTypes), session_filter)
+            session_filter = {'person_id': ints, 'session_id': strs}
+            sessions = Session().select(filter=session_filter)
+        elif isinstance(session_filter, dict):
+            sessions = Session().select(filter=session_filter)
+        elif isinstance(session_filter, (int, long)):
+            sessions = Session().select(filter={'person_id': session_filter})
+        elif isinstance(session_filter, StringTypes):
+            sessions = Session().select(filter={'session_id': session_filter})
+        else:
+            raise PLCInvalidArgument, "Wrong session filter"%session_filter
 
         if expires is not None:
             if expires >= 0:
@@ -96,4 +78,4 @@ class Sessions(Table):
                 expires = -expires
                 sql += " AND expires < %(expires)d"
 
-        self.selectall(sql, locals())
+        self.extend(sessions)
-- 
2.47.0