Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recursion error because upgraded events portlets missing attribute #305

Open
rpatterson opened this issue Apr 30, 2019 · 0 comments
Open

Comments

@rpatterson
Copy link
Member

Some attributes were added to the events portlet assignment class. Portlets added before this result in the following recursion error when the code is upgraded:

2019-04-29 18:10:09 ERROR imageportlet Error while determining renderer availability of portlet ('context' '/...' 'events'): maximum recursion depth exceeded
Traceback (most recent call last):
  File "eggs/plone.portlets-2.3-py2.7.egg/plone/portlets/manager.py", line 117, in _lazyLoadPortlets
    isAvailable = renderer.available
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 141, in available
    return self.data.count > 0 and len(self.events)
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 154, in events
    if ICollection and ICollection.providedBy(self.search_base):
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 122, in search_base
    if not self._search_base and self.data.search_base_uid:
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 106, in _uid
    path = self.search_base
...
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 122, in search_base
    if not self._search_base and self.data.search_base_uid:
  File "src/plone.app.event/plone/app/event/portlets/portlet_events.py", line 106, in _uid
    path = self.search_base
RuntimeError: maximum recursion depth exceeded

We should add an upgrade step that fixes this on existing assignments.

rpatterson added a commit to rpatterson/plone.app.event that referenced this issue Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant