- fixed error in indexargs_url (thanks Patrick Ohly)

- fixed getnode (sf bug 684531)

git-svn-id: http://svn.roundup-tracker.org/svnroot/roundup/trunk@1495 57a73879-2fb5-44c3-a270-3262357dd7e2

diff --git a/CHANGES.txt b/CHANGES.txt
index 76da383a58cbf3ebebcaef023a52ccfa326ce361..d3d3da2acbcb7a92276ebd53b66e215239226db0 100644 (file)
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -43,8 +43,11 @@ are given with the most recent entry first.
   return positive values..." (literal 0xffff0000 in portalocker.py)
 - fixed ZPT code generating SyntaxWarning for assignment to None
 - add "ago" to intervals in the past (sf bug 679232)
+- clarified licensing
 - another attempt to fix cookie misbehaviour - customise cookie name using
   tracker name
+- fixed error in indexargs_url (thanks Patrick Ohly)
+- fixed getnode (sf bug 684531)
 
 
 2003-??-?? 0.5.6

diff --git a/doc/index.txt b/doc/index.txt
index 37ad702e7cb1464680ede6cde5897f9586ff8326..dec63c8cef8d5fa6d0ab5db04b59bc07e092a109 100644 (file)
--- a/doc/index.txt
+++ b/doc/index.txt
@@ -69,6 +69,7 @@ Luke Opperman,
 Will Partain,
 Bernhard Reiter,
 John P. Rouillard,
+Florian Schulze,
 Dougal Scott,
 Stefan Seefeld,
 Jeffrey P Shell,

diff --git a/roundup/backends/back_anydbm.py b/roundup/backends/back_anydbm.py
index 681698b389d29fe04aef13b101c938d224745e18..ace6c45e408f97bde97771242eb9d9bd10fbb9d6 100644 (file)
--- a/roundup/backends/back_anydbm.py
+++ b/roundup/backends/back_anydbm.py
@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-#$Id: back_anydbm.py,v 1.100 2003-02-06 05:43:47 richard Exp $
+#$Id: back_anydbm.py,v 1.101 2003-02-12 00:00:18 richard Exp $
 '''
 This module defines a backend that saves the hyperdatabase in a database
 chosen by anydbm. It is guaranteed to always be available in python
@@ -30,7 +30,7 @@ from sessions import Sessions
 from roundup.indexer import Indexer
 from roundup.backends import locking
 from roundup.hyperdb import String, Password, Date, Interval, Link, \
-    Multilink, DatabaseError, Boolean, Number
+    Multilink, DatabaseError, Boolean, Number, Node
 
 #
 # Now the database

diff --git a/roundup/backends/back_metakit.py b/roundup/backends/back_metakit.py
index 4d1fd4c4785490bff9a2aa0ffa208c56bffddbe8..711a41b57e7ffa0586830cf3f8a166d5592bba4d 100755 (executable)
--- a/roundup/backends/back_metakit.py
+++ b/roundup/backends/back_metakit.py
@@ -50,7 +50,7 @@ def Database(config, journaltag=None):
             pass
     return db
 
-class _Database(hyperdb.Database):
+class _Database(hyperdb.Database, roundupdb.Database):
     def __init__(self, config, journaltag=None):
         self.config = config
         self.journaltag = journaltag
@@ -1206,13 +1206,16 @@ class FileClass(Class):
 
     def get(self, nodeid, propname, default=_marker, cache=1):
         x = Class.get(self, nodeid, propname, default, cache)
+        poss_msg = 'Possibly an access right configuration problem.'
         if propname == 'content':
             if x.startswith('file:'):
                 fnm = x[5:]
                 try:
                     x = open(fnm, 'rb').read()
-                except Exception, e:
-                    x = repr(e)
+                except IOError, (strerror):
+                    # XXX by catching this we donot see an error in the log.
+                    return 'ERROR reading file: %s%s\n%s\n%s'%(
+                            self.classname, nodeid, poss_msg, strerror)
         return x
 
     def create(self, **propvalues):

diff --git a/roundup/backends/rdbms_common.py b/roundup/backends/rdbms_common.py
index 3d41cafeafb3e85ea2aa12f4c0f713c92fd96fc0..1214555c28444cb5f7f062079f7067ee56fdcfdc 100644 (file)
--- a/roundup/backends/rdbms_common.py
+++ b/roundup/backends/rdbms_common.py
@@ -1,4 +1,4 @@
-# $Id: rdbms_common.py,v 1.31 2003-02-08 15:31:28 kedder Exp $
+# $Id: rdbms_common.py,v 1.32 2003-02-12 00:00:25 richard Exp $
 ''' Relational database (SQL) backend common code.
 
 Basics:
@@ -27,7 +27,7 @@ import sys, os, time, re, errno, weakref, copy
 # roundup modules
 from roundup import hyperdb, date, password, roundupdb, security
 from roundup.hyperdb import String, Password, Date, Interval, Link, \
-    Multilink, DatabaseError, Boolean, Number
+    Multilink, DatabaseError, Boolean, Number, Node
 from roundup.backends import locking
 
 # support

diff --git a/roundup/cgi/client.py b/roundup/cgi/client.py
index 80874cc55355f9a081c575391be0d2f9b02bc67f..d2cb1a79943066a1cbc68e2e792f513fc9598ca9 100644 (file)
--- a/roundup/cgi/client.py
+++ b/roundup/cgi/client.py
@@ -1,4 +1,4 @@
-# $Id: client.py,v 1.77 2003-02-07 04:49:13 richard Exp $
+# $Id: client.py,v 1.78 2003-02-12 00:00:28 richard Exp $
 
 __doc__ = """
 WWW request handler (also used in the stand-alone server).
@@ -1166,10 +1166,6 @@ class Client:
 
            This is typically used on, eg. the file upload page to indicated
            which issue to link the file to.
-
-           TODO: I suspect that this and newfile will go away now that
-           there's the ability to upload a file using the issue :file form
-           element!
         '''
         cn = self.classname
         cl = self.db.classes[cn]

diff --git a/roundup/cgi/templating.py b/roundup/cgi/templating.py
index d69f68e937d14dcc65d6441c220da82dde05417d..fc9713cb49fec18a0ae02496c0ad9f6b84736072 100644 (file)
--- a/roundup/cgi/templating.py
+++ b/roundup/cgi/templating.py
@@ -1526,7 +1526,7 @@ env: %(env)s
             else:
                 val = self.group[1]
             l.append(':group=%s'%val)
-        if self.filter and not args.has_key(':columns'):
+        if self.filter and not args.has_key(':filter'):
             l.append(':filter=%s'%(','.join(self.filter)))
         for k,v in self.filterspec.items():
             if not args.has_key(k):