Bug #56982: files copied for local processing are not deleted - TYPO3 Core - TYPO3 Forge

Custom queries

12 LTS
Accessibility
Dashboard issues
Easy tasks
Extbase Issues
FAL Issues
FAL Sprint 2023
Hierarchical Issues
Issues reported for v11
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
Most wanted Bugfixes
Most wanted Features
My open issues
New & Unassigned
No feedback within 90 days
Open Bugs
Open issues of 10
Page Tree Regressions after 2020-07-28
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (2)

Christian Weiske
Stefan Neufeind

Actions

Copy link

Bug #56982

closed

files copied for local processing are not deleted

Added by Christian Weiske about 10 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

File Abstraction Layer (FAL)

Target version:

Start date:

2014-03-17

Due date:

% Done:

Estimated time:

TYPO3 Version:

6.2

PHP Version:

Tags:

Complexity:

no-brainer

Is Regression:

Sprint Focus:

Remote Sprint

Description

TYPO3\CMS\Core\Resource\Processing\LocalPreviewHelper makes FAL storage adapters copy files locally to generate a preview image. It uses getFileForLocalProcessing for that.

After generating the preview, the file stays on the local file system and does not get deleted. This causes the local file system to fill up with unneeded files.

The temporarily locally copied file should get deleted after the preview image has been generated.

This happens with TYPO3 6.2beta7.

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Bug #62098: typo3temp folder is flooded with fal-tempfiles-*.*

Closed

2014-10-08

Actions

Issue # Delay: days Cancel

History
Notes
Property changes

Actions

Copy link

Updated by Steffen Ritter about 10 years ago

Status changed from New to Accepted
Complexity set to no-brainer

$originalFileName = $sourceFile->getForLocalProcessing(FALSE);

means: file is not copied - this only would happen for remote storages....

so we could fix the LOCALpreviewhelper for REMOTE storages - but that's not urgent atm.

Actions

Copy link

Updated by Steffen Ritter about 10 years ago

Target version set to 2702

Actions

Copy link

Updated by Steffen Ritter about 10 years ago

Target version deleted (~~2702~~)

Actions

Copy link

Updated by Frans Saris over 9 years ago

Sprint Focus set to On Location Sprint

In TYPO3\CMS\Core\Resource\Processing\LocalPreviewHelper::process() a check should be added that removes $originalFileName if it isn't the original file but a copy of it in typo3temp.

Actions

Copy link

Updated by Frans Saris over 9 years ago

Status changed from Accepted to New

Actions

Copy link

Updated by Mathias Schreiber about 9 years ago

Status changed from New to In Progress

Actions

Copy link

Updated by Gerrit Code Review about 9 years ago

Status changed from In Progress to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

Updated by Gerrit Code Review about 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

Updated by Gerrit Code Review about 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#10

Updated by Gerrit Code Review about 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#11

Updated by Gerrit Code Review about 9 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#12

Updated by Gerrit Code Review about 9 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#13

Updated by Gerrit Code Review about 9 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#14

Updated by Gerrit Code Review about 9 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#15

Updated by Anja Leichsenring over 8 years ago

Sprint Focus changed from On Location Sprint to Remote Sprint

Actions

Copy link

#16

Updated by Gerrit Code Review over 8 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36446

Actions

Copy link

#17

Updated by Gerrit Code Review about 8 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/36446

Actions

Copy link

#18

Updated by Riccardo De Contardi over 5 years ago

Status changed from Under Review to Closed

Closed for now - patch has been abandoned - see comments on https://review.typo3.org/36446

If you think that this is the wrong decision please reopen or ping me. Thank you.

Actions

Copy link

Also available in: Atom PDF

Project

General

Profile

TYPO3 Core

Custom queries

Watchers (2)

Bug #56982

files copied for local processing are not deleted

Updated by Steffen Ritter about 10 years ago

Updated by Steffen Ritter about 10 years ago

Updated by Steffen Ritter about 10 years ago

Updated by Frans Saris over 9 years ago

Updated by Frans Saris over 9 years ago

Updated by Mathias Schreiber about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Gerrit Code Review about 9 years ago

Updated by Anja Leichsenring over 8 years ago

Updated by Gerrit Code Review over 8 years ago

Updated by Gerrit Code Review about 8 years ago

Updated by Riccardo De Contardi over 5 years ago