[Zope-Checkins] [Checkins] SVN: Zope/branches/haufe-legacy-integration/ - Launchpad #373658: traversal using (un)restrictedTraverse

[Wichert Akkerman]

Previously Andreas Jung wrote:
> Modified: Zope/branches/haufe-legacy-integration/src/OFS/Traversable.py
> ===================================================================
> --- Zope/branches/haufe-legacy-integration/src/OFS/Traversable.py	2009-05-10 16:14:42 UTC (rev 99828)
> +++ Zope/branches/haufe-legacy-integration/src/OFS/Traversable.py	2009-05-10 19:12:43 UTC (rev 99829)
> @@ -154,6 +154,7 @@
>              path = list(path)
>  
>          REQUEST = {'TraversalRequestNameStack': path}
> +        web_request = getattr(self, 'REQUEST', None)
>          path.reverse()
>          path_pop = path.pop
>  
> @@ -198,7 +199,7 @@
>                          ns, nm = nsParse(name)
>                          try:
>                              next = namespaceLookup(
> -                                ns, nm, obj, aq_acquire(self, 'REQUEST'))
> +                                ns, nm, obj, web_request).__of__(obj)
>                              if IAcquirer.providedBy(next):
>                                  next = next.__of__(obj)
>                              if restricted and not validate(

What is the rational for the extra aq wrapping you are introducing here?

Wichert.

-- 
Wichert Akkerman <wichert at wiggy.net>    It is simple to make things.
http://www.wiggy.net/                   It is hard to make things simple.