Bug #89180
closedRemove sys_file_metadata width and height from excludefields
100%
Description
The fields sys_file_metadata.width
and sys_file_metadata.height
are marked as readOnly. But they are excludeField too.
Even though this is not a logical combination at all, it wouldn't hurt much so far. Until now, when a user tried to translate a sys_file_metadata record and the width and height (which she did not have access to) fields are then set to zero (which then effectively disabled the crop config for the translation of the record to be rendered).
I'd suggest to just remove the 'exclude' => true
from those two fields from sysext core and filemetadata. The height/width are managed internally anyway (for example when a user uploads an image the width/height are set correctly even though the user has no access according to the excludeFields). The visibility of the cols should be left to the typedef.
Updated by Christian Futterlieb about 5 years ago
I just saw: this is resolved by #81964, but for master only. Is there any chance we could get this for v8 and v9 too?
Updated by Christian Futterlieb about 5 years ago
- Related to Bug #81964: No crop information given in localized FAL version If translated meta data is provided added
Updated by Georg Ringer about 5 years ago
- Status changed from New to In Progress
will check if a backport will be done.
Updated by Georg Ringer over 4 years ago
- Status changed from In Progress to Accepted
- TYPO3 Version changed from 8 to 9
Updated by Gerrit Code Review over 4 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63467
Updated by Gerrit Code Review over 4 years ago
Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63467
Updated by Gerrit Code Review over 4 years ago
Patch set 3 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63467
Updated by Anonymous over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 6afc789f6684f3c3b72ec60a5bf51b913ff994d4.