[Zope-CMF] Re: "_v_skindata" fix

Florent Guillaume fg at nuxeo.com
Wed Feb 23 06:54:54 EST 2005

Dieter Maurer wrote:
> yesterday you posted a fix for the "_v_skindata" problem.
> Unfortunately, I looked over it, found it acceptable and then deleted it.
> Meanwhile, I may have noticed a problem:
>   When my memory is right, then it does not work for nested
>   "Skinnable"s. Because the information is stored
>   in a global structure keyed by thread id only.

That's right.

> We have a use case for nested "Skinnable" objects:
> a portal with nested sub-portals.
> The main portal provides most of the skin resources,
> the sub-portals override part of it to get their corporate identity.
> Your modification is not directly a problem for us,
> because we use the "_p_sticky" anyway.
> However, other people with similar use cases may be surprised.

I agree but I'm not sure I know a way to fix this. From __getattr__ 
there is very little that's accessible, and nothing about self's context 

Hm maybe we could store the portal's full path in a persistent 
attribute? And update it after add/clone ?


Florent Guillaume, Nuxeo (Paris, France)   CTO, Director of R&D
+33 1 40 33 71 59   http://nuxeo.com   fg at nuxeo.com

More information about the Zope-CMF mailing list