Bug #61559
closedAllowing to edit certain plugins without "Insert plugin" gives no warning
100%
Description
For an editor you might assign the right to edit a certain plugin that's placed on a page. But if you fail to also allow the CType "Insert plugin" that content will still not be editable. This might lead to questions why editing is not possible although those specific plugins desired are allowed.
Updated by Stefan Neufeind about 10 years ago
How do we want to/can we deal with this? Automatically also select "Insert plugin" upon safe if one or more plugins are allowed? It makes no real sense to be able to allow "Insert plugin" without activating a certain plugin. But even more it does not make sense to allow one or more plugins and not activate "Insert plugin", right?
Updated by Stefan Neufeind about 10 years ago
- Category set to Backend User Interface
Updated by Stefan Neufeind about 10 years ago
- Subject changed from BE: Allowing to edit certain plugins without "Insert plugin" gives no warning to Allowing to edit certain plugins without "Insert plugin" gives no warning
Updated by Philipp Gampe about 10 years ago
I suggest to just add a static textbox (flashmessage) above the allow plugin dialog, stating that the administrator should make sure to set allow plugin too.
Updated by Riccardo De Contardi over 9 years ago
this is still valid for 6.2.14 and 7 (latest master)
steps to reproduce:
1) install a plugin like "news"
2) add an editors usergroup
3) assign everything to the group (modules, tables listing, tables modify, allowed excludefields, etc) BUT...
3.1) be sure to set this:
Explicitly allow/deny field values:
- insert plugin: NOT CHECKED
- Page Content: Plugin:News system: CHECKED
4) create user editor and assign the group
5) put the news plugin to page
6) login as editor
Result: you can't modify the plugin CE from "page" module; if you try to modify it from "list" module you get the error:
Sorry, you didn't have proper permissions to perform this change. Reason: ERROR: authMode "explicitAllow" failed for field "CType" with value "list" evaluated
...quite cryptic...
Updated by Gerrit Code Review over 9 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41116
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41116
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41116
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41116
Updated by Nicole Cordes over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1e54574246a386e7f46e7740dbe8fc512e8b1608.