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

undo icontentish interface #3836

Merged
merged 10 commits into from
Oct 24, 2023
7 changes: 0 additions & 7 deletions Products/CMFPlone/Portal.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
from plone.base.permissions import View
from plone.dexterity.content import Container
from Products.CMFCore import permissions
from Products.CMFCore.interfaces import IContentish
from Products.CMFCore.interfaces import ISiteRoot
from Products.CMFCore.permissions import AccessContentsInformation
from Products.CMFCore.permissions import AddPortalMember
Expand All @@ -32,8 +31,6 @@
from Products.CMFPlone import bbb
from Products.Five.component.interfaces import IObjectManagerSite
from zope.event import notify
from zope.interface import classImplementsOnly
from zope.interface import implementedBy
from zope.interface import implementer
from zope.interface.interfaces import ComponentLookupError
from zope.traversing.interfaces import BeforeTraverseEvent
Expand Down Expand Up @@ -222,8 +219,4 @@ def isEffective(self, date):
return 1


# Remove the IContentish interface so we don't listen to events that won't
# apply to the site root, ie handleUidAnnotationEvent
classImplementsOnly(PloneSite, implementedBy(PloneSite) - IContentish)

InitializeClass(PloneSite)
1 change: 1 addition & 0 deletions news/3833.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Make PloneSite have IContentish again. @Akshat2Jain @jaroel