bugfixes for redirect and form handling

[roundup.git] / roundup / cgi / PageTemplates / Expressions.py

diff --git a/roundup/cgi/PageTemplates/Expressions.py b/roundup/cgi/PageTemplates/Expressions.py
index ccfc6c733c6f2db0f4c03ed6dedb56fbd1a4f09b..054e29344b2793c1c1b79997ed48aee1eb816282 100644 (file)
--- a/roundup/cgi/PageTemplates/Expressions.py
+++ b/roundup/cgi/PageTemplates/Expressions.py
@@ -25,7 +25,7 @@ Modified for Roundup 0.5 release:
 
 """
 
-__version__='$Revision: 1.3 $'[11:-2]
+__version__='$Revision: 1.6 $'[11:-2]
 
 import re, sys
 from TALES import Engine, CompilerError, _valid_name, NAME_RE, \
@@ -121,7 +121,7 @@ class SubPathExpr:
                     # of path names.
                     path[i:i+1] = list(val)
         base = self._base
-        __traceback_info__ = 'sub path expression "%s"'%base
+        __traceback_info__ = 'path expression "%s"'%('/'.join(self._path))
         if base == 'CONTEXTS':
             ob = econtext.contexts
         else:
@@ -264,6 +264,10 @@ class DeferExpr:
     def __repr__(self):
         return 'defer:%s' % `self._s`
 
+class TraversalError:
+    def __init__(self, path, name):
+        self.path = path
+        self.name = name
 
 def restrictedTraverse(self, path, securityManager,
                        get=getattr, has=hasattr, N=None, M=[],
@@ -279,17 +283,18 @@ def restrictedTraverse(self, path, securityManager,
     path.reverse()
     object = self
     #print 'TRAVERSE', (object, path)
+    done = []
     while path:
         name = path.pop()
-        __traceback_info__ = 'looking for "%s"'%name
+        __traceback_info__ = TraversalError(done, name)
 
         if isinstance(name, TupleType):
             object = apply(object, name)
             continue
 
-        if name[0] == '_':
-            # Never allowed in a URL.
-            raise AttributeError, name
+#        if name[0] == '_':
+#            # Never allowed in a URL.
+#            raise AttributeError, name
 
         # Try an attribute.
         o = get(object, name, M)
@@ -319,6 +324,7 @@ def restrictedTraverse(self, path, securityManager,
                 raise
         #print '... object is now', `o`
         object = o
+        done.append((name, o))
 
     return object