Task #70857
closed
displayCond: HIDE_FOR_NON_ADMINS should be ADMIN_ONLY or IS_ADMIN
0%
Description
Having a condition named "hide ..." is a double-negation. Instead it should positively say what the real condition is - the fact that somebody is an admin/has admin-rights.
Consider adding a new, clean wording for the exact same behaviour and to properly deprecate the old one.
(The original proposal for this was raised by Benjamin Kott.)
Updated by Christian Kuhn almost 8 years ago
- Status changed from New to Closed
Well, it's not beautiful, but also not a blocker. the display condition stuff is documented pretty well and the naming is still ok'ish. It would be possible to rename that field, too, but that currently doesn't seem to happen.
For now, I'll close that issue. If you want to catch up on the issue and implement it: The change should be relatively easy in "EvaluateDisplayCondition" together with a couple of unit test fixes. Furthermore, it should have a fully automatic migration within "TcaMigration" so people at least get some "I'll change it for you" magic for this naming change.
Feel free to pick that issue up again when https://review.typo3.org/#/c/50879/ is done.