handle deleted properties in RDBMS history

apply timezone in correct direction in user input [SF#1013097]

Author: Richard Jones

CHANGES.txt

@@ -46,6 +46,8 @@ Fixed:
 - fix lookup of REMOTE_USER (sf bug 1002923)
 - new Interval props weren't created properly in rdbms
 - date.Interval() now accepts an Interval as a spec (sf bug 1041266)
+- handle deleted properties in RDBMS history
+- apply timezone in correct direction in user input (sf bug 1013097)
 
 
 2004-07-21 0.7.6

roundup/backends/rdbms_common.py

@@ -1,4 +1,4 @@
-# $Id: rdbms_common.py,v 1.134 2004-10-08 00:18:28 richard Exp $
+# $Id: rdbms_common.py,v 1.135 2004-10-08 00:56:12 richard Exp $
 ''' Relational database (SQL) backend common code.
 
 Basics:
@@ -1099,7 +1099,10 @@ def getjournal(self, classname, nodeid):
                 for param, value in params.items():
                     if not value:
                         continue
-                    property = properties[param]
+                    property = properties.get(param, None)
+                    if property is None:
+                        # deleted property
+                        continue
                     cvt = self.sql_to_hyperdb_value[property.__class__]
                     if isinstance(property, Password):
                         params[param] = cvt(value)

roundup/hyperdb.py

@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: hyperdb.py,v 1.102 2004-07-20 23:24:26 richard Exp $
+# $Id: hyperdb.py,v 1.103 2004-10-08 00:56:11 richard Exp $
 
 """Hyperdatabase implementation, especially field types.
 """
@@ -647,7 +647,7 @@ def rawToHyperdb(db, klass, itemid, propname, value,
     elif isinstance(proptype, Date):
         try:
             tz = db.getUserTimezone()
-            value = date.Date(value).local(tz)
+            value = date.Date(value).local(-tz)
         except ValueError, message:
             raise HyperdbValueError, 'property %s: %r is an invalid '\
                 'date (%s)'%(propname, value, message)