Skip to content

Commit 4ab653a

Browse files
author
Richard Jones
committed
Fixed a bug in the filter
wrong variable names in the error message. Recognised that the filter has an outstanding bug. Hrm. we need a bug tracker for this project :)
1 parent 4e623ba commit 4ab653a

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

README

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,9 @@ date:
211211
. date subtraction doesn't work correctly "if the dates cross leap years,
212212
phases of the moon, ..."
213213

214+
filter:
215+
. incorrectly embeds hidden fields for filters being displayed - and
216+
doesn't use the existing values for filters being displayed either.
214217

215218

216219
6. Author

hyperdb.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# $Id: hyperdb.py,v 1.5 2001-07-20 07:35:55 richard Exp $
1+
# $Id: hyperdb.py,v 1.6 2001-07-20 08:20:24 richard Exp $
22

33
# standard python modules
44
import cPickle, re, string
@@ -486,7 +486,7 @@ def filter(self, filterspec, sort, group, num_re = re.compile('^\d+$')):
486486
entry = link_class.lookup(entry)
487487
except:
488488
raise ValueError, 'new property "%s": %s not a %s'%(
489-
key, entry, self.properties[key].classname)
489+
k, entry, self.properties[k].classname)
490490
u.append(entry)
491491

492492
l.append((0, k, u))
@@ -502,7 +502,7 @@ def filter(self, filterspec, sort, group, num_re = re.compile('^\d+$')):
502502
entry = link_class.lookup(entry)
503503
except:
504504
raise ValueError, 'new property "%s": %s not a %s'%(
505-
key, entry, self.properties[key].classname)
505+
k, entry, self.properties[k].classname)
506506
u.append(entry)
507507
l.append((1, k, u))
508508
elif propclass.isStringType:
@@ -737,4 +737,8 @@ def Choice(name, *options):
737737

738738
#
739739
# $Log: not supported by cvs2svn $
740+
# Revision 1.5 2001/07/20 07:35:55 richard
741+
# largish changes as a start of splitting off bits and pieces to allow more
742+
# flexible installation / database back-ends
743+
#
740744

0 commit comments

Comments
 (0)