Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[BUGFIX] Make meta data editable for non-writable storages
Decouple check for writable files/storage from permission to edit meta data. Permission to edit meta data is now only denied when users have only access to the file via a readonly file mount. Resolves: #65636 Resolves: #66507 Releases: master, 8.7 Change-Id: I25a0fbc9cf761898dbdb95dec1d3d39bb2f4b7fd Reviewed-on: https://review.typo3.org/42874 Tested-by: TYPO3com <no-reply@typo3.com> Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by: Joerg Kummer <typo3@enobe.de> Reviewed-by: Josef Glatz <josef.glatz@typo3.org> Reviewed-by: Nicole Cordes <typo3@cordes.co> Tested-by: Nicole Cordes <typo3@cordes.co> Reviewed-by: Benni Mack <benni@typo3.org> Tested-by: Benni Mack <benni@typo3.org>
- Loading branch information
1 parent
d01d615
commit c3fef10
Showing
5 changed files
with
107 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 changes: 28 additions & 0 deletions
28
...tion/Changelog/9.5.x/Important-65636-AllowMetaDataEditingOnReadOnlyStorages.rst
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
.. include:: ../../Includes.txt | ||
|
||
========================================================================== | ||
Important: #65636 - File meta data can now be edited on read only storages | ||
========================================================================== | ||
|
||
See :issue:`65636` | ||
|
||
Description | ||
=========== | ||
|
||
Whether meta data editing of files is allowed or not, must not be bound to whether a file is | ||
physically writable in a storage, or whether the storage itself is set read only. | ||
|
||
Editing meta data should on the other hand be forbidden, when the file is within a read only | ||
file mount. | ||
|
||
Allowing meta data editing on read only storage | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
To allow to distinguish between read access to a file and write access to file meta data, | ||
a new file action permission `editMeta` is introduced, which is automatically checked and | ||
enforced when saving a meta data record. | ||
|
||
When having to check for editing meta data permission in userland code, it is recommended | ||
to use the new file action permission instead of the previously used permission `read`. | ||
|
||
.. index:: ext:core, NotScanned |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters