#241 — Site error at viewlet refresh with poi installed

State Resolved
Version:
Area Functionality
Issue type Bug
Severity Medium
Submitted by Thomas Desvenain
Submitted on May 15, 2010
Responsible Maurits van Rees
Target release:
Once poi has been installed, we have this error when we refresh the viewlets, for example when removing a portlet

the viewlet can't adapt with plone site

2010-05-15 22:36:40 ERROR Zope.SiteErrorLog 1273955800.630.511188047378 http://localhost:8080/Plone/inlineDeletePortlet
Traceback (innermost last):
  Module ZPublisher.Publish, line 127, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module Products.PDBDebugMode.runcall, line 70, in pdb_runcall
  Module ZPublisher.Publish, line 47, in call_object
  Module plone.app.portlets.browser.kss, line 66, in delete_portlet
  Module plone.app.portlets.browser.kss, line 98, in _render_column
  Module kss.core.kssview, line 162, in render
  Module zope.event, line 23, in notify
  Module zope.component.event, line 26, in dispatch
  Module zope.component._api, line 138, in subscribers
  Module zope.component.registry, line 323, in subscribers
  Module zope.interface.adapter, line 575, in subscribers
  Module plone.app.kss.commands.zopecommands, line 68, in addRefreshViewletCommands
  Module zope.component.registry, line 252, in getAdapters
  Module Products.Poi.browser.response, line 343, in __init__
  Module Products.Poi.browser.response, line 81, in __init__
TypeError: ('Could not adapt', <PloneSite at /Plone>, <InterfaceClass Products.Poi.adapters.IResponseContainer>)
> /home/thomasdesvenain/buildouts/extranet/src/Products.Poi/Products/Poi/browser/response.py(81)__init__()
-> self.folder = IResponseContainer(context)
Steps to reproduce:
add a portlet
install poi
go to portlet manager and try to remove a portlet
Added by Maurits van Rees on May 17, 2010 09:20 AM
Issue state: UnconfirmedConfirmed
Responsible manager: (UNASSIGNED)maurits
I cannot reproduce this. But the traceback points to the poi.response.add viewlet manager (which does not actually manage viewlets but is just a provider, but this is the way you register that in zcml). This viewlet manager is defined for all Interfaces; it should help if I register it only for the IIssue interface.

Fixed on the 1.2 branch in r117680; and on trunk in r117681.
Added by Thomas Desvenain on May 17, 2010 10:02 AM
Yes it does : if you set viewletmanager

for="Products.Poi.Interfaces.IIssue"

you fix the problem

can i commit this ?
Added by Maurits van Rees on May 17, 2010 10:20 AM
Ehm, no you cannot commit this, as I have already fixed it, as noted in my previous reply. :-)
Added by Maurits van Rees on May 17, 2010 10:20 AM
Issue state: ConfirmedResolved

No responses can be added.