[Plone-cvs] r38717 - plone.app.viewletmanager/trunk/plone/app/viewletmanager/exportimport

[Rok G. <svn...@pl...>]

Author: garbas
Date: Thu Aug 12 23:22:57 2010
New Revision: 38717

Modified:
   plone.app.viewletmanager/trunk/plone/app/viewletmanager/exportimport/storage.py
Log:
fixes #10903, listing skins is not done right because in storage._hidden
there are only themes listing hidden viewlets. in default plone install
that is only "Plone Classic Theme". not sure this should only apply here
or in any other pard of the code. but it fixes my usecase described in
ticket.

Modified: plone.app.viewletmanager/trunk/plone/app/viewletmanager/exportimport/storage.py
==============================================================================
--- plone.app.viewletmanager/trunk/plone/app/viewletmanager/exportimport/storage.py	(original)
+++ plone.app.viewletmanager/trunk/plone/app/viewletmanager/exportimport/storage.py	Thu Aug 12 23:22:57 2010
@@ -95,7 +95,8 @@
             manager = child.getAttribute('manager')
             skins = getattr(storage, '_'+nodename)
             if skinname == '*':
-                for skinname in skins:
+                for skinname in tuple(set(storage._hidden.keys() + \
+                                          storage._order.keys())):
                     values = []
                     if not purgeChild:
                         values = list(skins[skinname].get(manager, []))