Bug #51698

Task #50876: Handling of deleted files in FAL

Delete sys_file entry when a file is deleted

Added by Ernesto Baschny almost 2 years ago. Updated 8 months ago.

Status:Resolved Start date:2013-09-03
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:File Abstraction Layer (FAL) Spent time: -
Target version:next-patchlevel
TYPO3 Version:6.0 Is Regression:No
PHP Version: Sprint Focus:
Complexity:easy

Description

In order to keep sys_file as much in sync with the real file system as possible we should delete the relevant sys_file entry as soon as a file is deleted.

This is part of the "plan" in #50876 and should also be backported to 6.0/6.1 to keep the API straight and uniform throughout the releases.


Related issues

related to Core - Bug #50508: Re-uploading file in backend fails Resolved 2013-07-29
blocks Core - Task #51838: Delete all processed files when a sys_file gets deleted Closed 2013-09-08

Associated revisions

Revision 1cfc65dc
Added by Frans Saris almost 2 years ago

[TASK] Delete sys_file(_metadata) entries when a file is deleted

In order to keep sys_file and sys_file_processedfile
as much in sync with the real file system as possible
we delete the relevant sys_file and sys_file_metadata
entries as soon as a file is deleted.

Resolves: #51698
Releases: 6.2, 6.1, 6.0
Change-Id: Ifa6a565702c26dc02c01634076726ef9f2272f84
Reviewed-on: https://review.typo3.org/23671
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#2 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#3 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#4 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#5 Updated by Gerrit Code Review almost 2 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#6 Updated by Gerrit Code Review almost 2 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#7 Updated by Gerrit Code Review almost 2 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#8 Updated by Gerrit Code Review almost 2 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#9 Updated by Gerrit Code Review almost 2 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#10 Updated by Gerrit Code Review almost 2 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#11 Updated by Gerrit Code Review almost 2 years ago

Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#12 Updated by Gerrit Code Review almost 2 years ago

Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#13 Updated by Gerrit Code Review almost 2 years ago

Patch set 13 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#14 Updated by Gerrit Code Review almost 2 years ago

Patch set 14 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#15 Updated by Gerrit Code Review almost 2 years ago

Patch set 15 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#16 Updated by Gerrit Code Review almost 2 years ago

Patch set 16 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#17 Updated by Gerrit Code Review almost 2 years ago

Patch set 17 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#18 Updated by Gerrit Code Review almost 2 years ago

Patch set 18 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#19 Updated by Gerrit Code Review almost 2 years ago

Patch set 19 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#20 Updated by Gerrit Code Review almost 2 years ago

Patch set 20 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23671

#21 Updated by Frans Saris almost 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#22 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26995

#23 Updated by Gerrit Code Review over 1 year ago

Patch set 2 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26995

#24 Updated by Benjamin Mack 8 months ago

  • Status changed from Under Review to Resolved
  • Is Regression set to No

fixed in latest

Also available in: Atom PDF