Bug #22359
closed
No FE editing with only field edit icons and with "old" feedit extension
Added by Mickaël PERRIN over 14 years ago.
Updated almost 5 years ago.
Description
Hello,
I was unable to make FE editing work with this UserTS setup :
admPanel {
enable {
edit = 1
}
hide = 1
override {
edit {
displayFieldIcons = 1
displayIcons = 0
}
}
}
Setting displayIcons = 1 does the trick however this enable features I don't need.
It seems there is an issue with the test function isFrontendEditingActive which is testing only if $GLOBALS['TSFE']->displayEditIcons 1 but not if $GLOBALS['TSFE']->displayFieldEditIcons 1.
A simple patch is attached.
Many thanks for your review
(issue imported from #M13978)
Files
Your problem might root to the same source as Issue 0015194. It seems you may not unset displayEditIcons.
2 Daniel Mueller
No. It's seems to be an another problem. Existing verification is not complete, it checks only the case when edit icons are rendered aside content records, and not the case when edit icons are rendered aside individual fields of content.
So the proposed patch should resolve the bug. I've made a little bit more clean version of the patch. Check in the attachment.
Just ran into the same issue, summarized:
for non-admin be-users, you cannot enable admin panel displayFieldIcons without also enabling displayIcons.
The problem still exists with v4.4 and 4.5
The second patch, 13978-2.patch, is clean and working for TYPO3 v4.5.0
+1 by reading and testing, should be a no-brainer.
- Target version deleted (
0)
Hi,
i can confirm this issue with TYPO3 6.1.1
a Workaround is, to remove the editPanel via TypoScript. If you need to have it different on usergroup, you can use conditions for checking for hat.
tt_content.stdWrap.editPanel >
- Description updated (diff)
- Category changed from Communication to Frontend
- Target version set to 7.2 (Frontend)
- TYPO3 Version set to 4.5
- Is Regression set to No
- Target version changed from 7.2 (Frontend) to 7.4 (Backend)
- Target version changed from 7.4 (Backend) to 7.5
- Target version changed from 7.5 to 7 LTS
- Target version deleted (
7 LTS)
Thank you for your report.
Even though it has been some time, would you consider checking if your patch idea is still up to date and upload it to our Gerrit review server?
Someone could do this for you, but I am thinking you might like the opportunity to contribute to TYPO3 yourself.
You can find a description of the TYPO3 contribution workflow here: https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/
Hint: If you get stuck anywhere, ask on Slack in the #typo3-cms-coredev channel. You can register in the TYPO3 slack workspace here: https://forger.typo3.com/slack
Thank you in advance!
- Category changed from Frontend to AdminPanel
- Status changed from New to Closed
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by