Bug #45065
closed
Remove delete button for sys_file
100%
Description
The edit view of file meta data currently has
a delete button, which sets the record of this
file to deleted, but does not delete the file itself.
It might be useful at some point
to use this to implement some recycler
functionality, but currently the FAL layer
has no support for it and a file with a deleted
meta data record is an inconsistent state
which has to be avoided by all means.
Updated by Gerrit Code Review about 11 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/17916
Updated by Gerrit Code Review about 11 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17917
Updated by Anonymous about 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset dc1c32715077cb9eb0f31569f0e4ca49a03c252f.
Updated by Ingo Renner about 11 years ago
- Category set to File Abstraction Layer (FAL)
- Status changed from Resolved to Needs Feedback
- Assignee set to Helmut Hummel
I agree with the problem of the inconsistent state, but wouldn't the correct solution be to hook into TCEmain, monitor sys_file deletes and then also remove the file? Simply merging this one doesn't seem right to me...
Updated by Helmut Hummel about 11 years ago
- Status changed from Needs Feedback to Resolved
- Assignee deleted (
Helmut Hummel)
Ingo Renner wrote:
I agree with the problem of the inconsistent state, but wouldn't the correct solution be to hook into TCEmain, monitor sys_file deletes and then also remove the file? Simply merging this one doesn't seem right to me...
The whole file/ data handling needs to be re-thought with FAL. There are a still a lot of things to do in this area.
Any help in this area is appreciated. Feel free to work on that.
Hiding the delete button is very effective with minimum amount of impact and effort to avoid accidental removal of the record.
Please open new tickets for further improvements in this area.
Updated by Ingo Renner about 11 years ago
Agreed. I might have a 6.0 project soon, would be happy to help.
Updated by