[Zope-CMF] Re: "_v_skindata" fix
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.
> 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