[Zope-CMF] Re: "_v_skindata" fix

[Florent Guillaume]

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 
wrappers.

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

Florent

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