Zope Collector fixes.

- merge Zope Collector #538 fix from ZPT CVS trunk (path expressions with a
  non-path final alternate no longer try to call a value returned by that
  alternate)
- merge Zope Collector #573 fix from ZPT CVS trunk
- added 1000-user/issue benchmark test

Author: Richard Jones

CHANGES.txt

@@ -36,7 +36,13 @@ are given with the most recent entry first.
 - handle stupid mailers that QUOTE their Re; 'Re: "[issue1] bla blah"'
 - giving a user a Role that doesn't exist doesn't break stuff any more
 - revamped user guide, customisation guide, added maintenance guide
+- merge Zope Collector #538 fix from ZPT CVS trunk (path expressions with a
+  non-path final alternate no longer try to call a value returned by that
+  alternate)
+- merge Zope Collector #573 fix from ZPT CVS trunk
 - merge Zope Collector #580 fix from ZPT CVS trunk
+- added "crypt" password encoding and ability to set password with
+  already encrypted password through roundup-admin
 
 
 2002-09-13 0.5.0 beta2

roundup/cgi/PageTemplates/Expressions.py

@@ -25,7 +25,7 @@
 
 """
 
-__version__='$Revision: 1.7 $'[11:-2]
+__version__='$Revision: 1.8 $'[11:-2]
 
 import re, sys
 from TALES import Engine, CompilerError, _valid_name, NAME_RE, \
@@ -136,6 +136,7 @@ class PathExpr:
     def __init__(self, name, expr, engine):
         self._s = expr
         self._name = name
+        self._hybrid = 0
         paths = split(expr, '|')
         self._subexprs = []
         add = self._subexprs.append
@@ -145,6 +146,7 @@ def __init__(self, name, expr, engine):
                 # This part is the start of another expression type,
                 # so glue it back together and compile it.
                 add(engine.compile(lstrip(join(paths[i:], '|'))))
+                self._hybrid = 1
                 break
             add(SubPathExpr(path)._eval)
 
@@ -169,8 +171,11 @@ def _eval(self, econtext,
             else:
                 break
         else:
-            # On the last subexpression allow exceptions through.
+            # On the last subexpression allow exceptions through, and
+            # don't autocall if the expression was not a subpath.
             ob = self._subexprs[-1](econtext)
+            if self._hybrid:
+                return ob
 
         if self._name == 'nocall' or isinstance(ob, StringType):
             return ob

roundup/cgi/ZTUtils/Iterator.py

@@ -18,8 +18,8 @@
 iterator.  The next() method fetches the next item, and returns
 true if it succeeds.
 
-$Id: Iterator.py,v 1.1 2002-09-05 00:37:09 richard Exp $'''
-__version__='$Revision: 1.1 $'[11:-2]
+$Id: Iterator.py,v 1.2 2002-09-26 21:54:17 richard Exp $'''
+__version__='$Revision: 1.2 $'[11:-2]
 
 import string
 
@@ -137,7 +137,7 @@ class SeqInner(InnerBase):
 
     def _supports(self, ob):
         try: ob[0]
-        except TypeError: return 0
+        except (TypeError, AttributeError): return 0
         except: pass
         return 1
 

test/benchmark.py

@@ -141,4 +141,7 @@ def main(backendname, time=time.time, numissues=10):
         main(name, numissues=20)
     for name in 'anydbm bsddb bsddb3 metakit sqlite'.split():
         main(name, numissues=100)
+    # don't even bother benchmarking the dbm backends > 100!
+    for name in 'metakit sqlite'.split():
+        main(name, numissues=1000)