The Plone site root is now cataloged

Fixes #3314

Products/CMFPlone/CatalogTool.py

@@ -488,6 +488,7 @@ def indexObject(obj, path):
                     pass
         self.manage_catalogClear()
         portal = aq_parent(aq_inner(self))
+        indexObject(portal, '')
         portal.ZopeFindAndApply(
             portal,
             search_sub=True,

Products/CMFPlone/Portal.py

@@ -222,19 +222,6 @@ def isEffective(self, date):
         # Override DefaultDublinCoreImpl's test, since we are always viewable.
         return 1
 
-    # Ensure portals don't get cataloged.
-    def indexObject(self):
-        pass
-
-    def unindexObject(self):
-        pass
-
-    def reindexObject(self, idxs=None):
-        pass
-
-    def reindexObjectSecurity(self, skip_self=False):
-        pass
-
 
 # Remove the IContentish interface so we don't listen to events that won't
 # apply to the site root, ie handleUidAnnotationEvent

Products/CMFPlone/tests/testCatalogTool.py

@@ -31,7 +31,7 @@
 group2 = 'g2'
 
 base_content = ['Members', 'aggregator', 'aggregator',
-                'events', 'news', TEST_USER_ID, 'doc']
+                'events', 'news', 'plone', TEST_USER_ID, 'doc']
 
 
 class TestCatalogSetup(PloneTestCase):

Products/CMFPlone/tests/testPortalCreation.py

@@ -86,6 +86,13 @@ def testWorkflowIsActionProvider(self):
             'portal_workflow' in self.actions.listActionProviders()
         )
 
+    def testPortalIsIndexed(self):
+        # The Plone site should be cataloged
+        res = self.catalog(getId="plone")
+        self.assertEqual(len(res), 1)
+        self.assertEqual(res[0].getId, "plone")
+        self.assertEqual(res[0].Title, "Welcome to Plone")
+
     def testMembersFolderMetaType(self):
         # Members folder should have meta_type 'Dexterity Container'
         members = self.membership.getMembersFolder()

news/3314.bugfix

@@ -0,0 +1 @@
+The Plone site root is cataloged