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

Bugfix for unused elements message in backend (introduced in TYPO3 9) #62

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

Conversation

schoeppe
Copy link

@schoeppe schoeppe commented Aug 4, 2020

@IchHabRecht
Copy link
Owner

Hi @schoeppe,

Thank you for your pull request. Instead of implementing an own solution I would suggest to use a default extension like https://extensions.typo3.org/extension/hide_used_content/

@schoeppe
Copy link
Author

schoeppe commented Aug 4, 2020

Hi,
thanks for pointing me to this extension, didn't know about it.

My implementation is based on the code I found in the flux-Extension. In my opinion this should be handled by the extension itself - the extension just knows best which content elements are used in the backend context.

@IchHabRecht
Copy link
Owner

Hi @schoeppe,

To be honest I was against the core implementation because I don't understand why every extension should ship its own hook. Everything is set up as TCA information. This is why I wrote the hide_used_content extension. IHMO this solution should be implemented core wise.

@joximu
Copy link

joximu commented Apr 28, 2021

Just want to tell:
I installed "hide_used_content" but the message about unused content didn't disappear.
With the patch/bugfix from schoeppe it's fine.

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.

3 participants