Skip to content

Commit

Permalink
Restore Plone Site default in ISearchSchema
Browse files Browse the repository at this point in the history
  • Loading branch information
jensens committed May 30, 2024
1 parent 510e50e commit a9ed087
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
4 changes: 2 additions & 2 deletions news/65.bugfix
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Remove ISearchSchemas types_not_searched "Discussion Item" value from plone.app.discussion and other impossible values too.
Default is empty now, since there will never the construction allowed of these three.
Remove ISearchSchemas types_not_searched "Discussion Item" value to make plone.app.discussion a core addon.
It is actually not needed anyway, also not part of the underlying vocabulary and would be lost on first save in control-panel.
See https://github.com/zopefoundation/Products.CMFCore/blob/8d765b8ce7ec4e053e58f5c8dc45d08db01ce3e0/src/Products/CMFCore/TypesTool.py#L768
[@jensens]
8 changes: 7 additions & 1 deletion src/plone/base/interfaces/controlpanel.py
Original file line number Diff line number Diff line change
Expand Up @@ -1015,7 +1015,13 @@ class ISearchSchema(Interface):
"off here or by the relevant installer."
),
required=False,
default=(),
# XXX: Actually, the Plone Site and TempFolder are not part of the vocabulary
# and would be removed from the list of types not searched on first save in ControlPanel!
# Both are used in many tests and kept for now.
default=(
"Plone Site",
"TempFolder",
),
missing_value=(),
value_type=schema.Choice(source="plone.app.vocabularies.PortalTypes"),
)
Expand Down

0 comments on commit a9ed087

Please sign in to comment.