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

Plone 5 and upwards needs this index to figure out the icon. #194

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maethu
Copy link
Contributor

@maethu maethu commented Jan 15, 2021

The older getIcon Index returns bool (Not used from solr anymore) for either showing or not showing a thumbnail in folder_contents.

I keep the getIcon index in solr for plone 4 support and it dies not harm the plone 5 installation either.

Quote from plone's indexesr.py

geticon redefined in Plone > 5.0 see plone/Products.CMFPlone#1226
reuse of metadata field,
now used for showing thumbs in content listings etc.
when obj is an image or has a lead image
or has an image field with name 'image': true else false

@maethu maethu force-pushed the mle-add-mime-type branch from e80215a to 9603417 Compare January 15, 2021 20:45
@maethu maethu requested a review from buchi January 15, 2021 20:47
The older getIcon Index return bool for either showing or not showing a thumbnail in folder_contents.
@maethu maethu force-pushed the mle-add-mime-type branch from 9603417 to 169d8f7 Compare January 18, 2021 14:01
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

Successfully merging this pull request may close these issues.

1 participant