[Zope-dev] [Checkins] SVN: zope.traversing/trunk/src/zope/traversing/ Moved the publicationtraverse module from zope.app.publication and added tests.

Jim Fulton jim at zope.com
Sun Jun 21 10:58:30 EDT 2009


Gaaa.  As I did deeper, it's even more muddled that I feared.  I'll  
start a separate thread.

Jim

On Jun 21, 2009, at 8:48 AM, Jim Fulton wrote:

>
> On Jun 21, 2009, at 5:36 AM, Christian Theune wrote:
>
>> On Sat, 2009-06-20 at 13:04 -0600, Shane Hathaway wrote:
>>> Jim Fulton wrote:
>>>> Why?  traverseName is part of zope.app.publication's  
>>>> implementation.
>>>> Now it's oddly split off in a very separate package.
>>>
>>> The publisher traversal code is very similar to the code in
>>> zope.traversing, so I thought the best thing to do is put it in the
>>> same
>>> package as zope.traversing, so that traversal would be maintained
>>> in one
>>> place.
>>
>> I think there's a valid goal around here. Publisher traversal and
>> traversal as used in templates have two different implementations  
>> that
>> partly overlap at some points and its hard to explain why there are
>> two
>> of them.
>>
>> I'm not sure whether we can get a single unified implementation, but
>> for
>> me that would be nice.
>
>
> I don't agree. The semantics are different. For example, you often
> want to traverse to things in a template that you don't want to expose
> via URL.  We currently (or last time I checked) expose ++resource+
> +name in URLs and this is a bug.
>
> The only place this traversal code is used outside of publication is
> by the browser menu code that tries to traverse to a menu item to see
> if it is accessible to the user.  This was to avoid having to put
> security declarations on menu items (a worthy goal) but we've found
> that this entailed far too much expense and complexity.  I'm not sure
> anyone is even using this menu code any more and if they are, I bet
> they are or should be making explicit security declarations.
>
> Jim
>
> --
> Jim Fulton
> Zope Corporation
>
>
> _______________________________________________
> Zope-Dev maillist  -  Zope-Dev at zope.org
> http://mail.zope.org/mailman/listinfo/zope-dev
> **  No cross posts or HTML encoding!  **
> (Related lists -
> http://mail.zope.org/mailman/listinfo/zope-announce
> http://mail.zope.org/mailman/listinfo/zope )

--
Jim Fulton
Zope Corporation




More information about the Zope-Dev mailing list