Bug #51698

Task #50876: Handling of deleted files in FAL

Delete sys_file entry when a file is deleted

Added by Ernesto Baschny about 6 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2013-09-03
Due date:
% Done:

100%

TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

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 TYPO3 Core - Bug #50508: Re-uploading file in backend fails Closed 2013-07-29
Blocks TYPO3 Core - Task #51838: Delete all processed files when a sys_file gets deleted Closed 2013-09-08

Associated revisions

Revision 1cfc65dc (diff)
Added by Frans Saris about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 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 about 6 years ago

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

#22 Updated by Gerrit Code Review almost 6 years 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 5 years 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 Benni Mack almost 5 years ago

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

fixed in latest

#25 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF