Project

General

Profile

Actions
Copy link

Bug #81836

closed

deleting a file inside _recycler_ folder generates new file with preview_ prefix

Added by Alexander Grein over 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Recycler
Target version:
Candidate for patchlevel
Start date:
2017-07-11
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

To reproduce:
  1. Add a folder called recycler
  2. Delete any image file somewhere inside fileadmin
  3. Take a look inside recycler folder and see (first bug) an additional preview_-File
  4. Try to delete one of those files inside recycler folder and see the new generated files with preview_ prefix
  5. Same problem exists if you try to delete old files using scheduler task

Related issues 4 (0 open — 4 closed)

Related to TYPO3 Core - Bug #85019: Recycler folder detection loop does not endClosed2018-05-16

Actions
Related to TYPO3 Core - Bug #85079: You are not allowed to access the given folder: "user_upload"ClosedNicole Cordes2018-05-25

Actions
Related to TYPO3 Core - Bug #88632: File cannot be deleted from a storage that is not browsableClosedMarkus Klein2019-06-26

Actions
Is duplicate of TYPO3 Core - Bug #79116: Delete-infinite-loop: fileadmin/_recycler_ preview imagesClosed2016-12-30

Actions
Actions
Copy link
 #1

Updated by Alexander Grein over 7 years ago

This problem also exists in TYPO3 7.6.20

Actions
Copy link
 #2

Updated by Riccardo De Contardi almost 7 years ago

  • Status changed from New to Needs Feedback

Where should the "recycler" folder be added? Inside fileadmin? Could you add a step-by-step guide about how to reproduce this issue? Thank you!

Actions
Copy link
 #3

Updated by Alexander Grein almost 7 years ago

Step-by-Step Guide for TYPO3 8.7.9
  1. Open Backend-Module Filelist
  2. Click on the root of "fileadmin/ (auto-created)" storage
  3. Click on the Plus-Sign to add a new folder named `_recycler_`
  4. Go to any other folder and upload a picture (e.g. a png image) to it
  5. Click on the same folder again, where you uploaded the picture and press the delete icon (bucket), available in the extended view
  6. Change to the former created `_recycler_` folder and realize two new files: the deleted one and a file named 'preview_' + filename_of_the_deleted_file_without_extension + '_' + hash + fileextension_of_the_deleted_file
  7. Deleting the deleted file inside this recycler folder again, by pressing the delete icon works perfect
  8. Try to delete the 'preview_'-file ends with a new generated preview-file with this schema: + 'preview_preview_' + filename_of_the_deleted_file_without_extension + '_' + hash + '_' + hash + fileextension_of_the_deleted_file
  9. If you try to delete this again you will get a new file 'preview_preview_preview_' + filename_of_the_deleted_file_without_extension + '_' + hash + '_' + hash + '_' + hash + fileextension_of_the_deleted_file
  10. And so on
Actions
Copy link
 #4

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Needs Feedback to New

I am able to reproduce it on 8.7.10, even deleting the "normal" <filename> I get a new file preview_<filename>_<hash>

It seems solved on version 9.2.0 can you confirm?

Actions
Copy link
 #5

Updated by Guido S. over 6 years ago

Can confirm it with latest master (9.2.-dev)

CASE A: "normal" <filename> I get a new file preview_<filename>_<hash>
If you upload a file TYPO3 generates a preview image in typo3\sysext\core\Classes\Resource\Processing\LocalPreviewHelper.php::process() So if you delete the parent, you should delete the child. That's fine. I think we can delete this image file directly. If you recover the original file, it generates a new preview image.

CASE B: Recursive preview files is reproducable with option "Display thumbnails"
If you have delete the preview image and activate "display thumbnails" the processTask generate a new preview image. If we handle Case A correctly (delete preview image), we haven't got this problem.

Actions
Copy link
 #6

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New 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 https://review.typo3.org/56503

Actions
Copy link
 #7

Updated by Guido S. over 6 years ago

  • Is duplicate of Bug #79116: Delete-infinite-loop: fileadmin/_recycler_ preview images added
Actions
Copy link
 #8

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Under Review to Closed
Actions
Copy link
 #9

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Closed to Under Review

sorry, wrong closing

Actions
Copy link
 #10

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #11

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #12

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #13

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #14

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #15

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #16

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #17

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #18

Updated by Gerrit Code Review over 6 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/56503

Actions
Copy link
 #19

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #20

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #21

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #22

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #23

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #24

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #25

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #26

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #27

Updated by Gerrit Code Review over 6 years ago

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

Actions
Copy link
 #28

Updated by Guido S. over 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #29

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

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

Actions
Copy link
 #30

Updated by Nicole Cordes over 6 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #31

Updated by Alexander Stehlik over 6 years ago

  • Related to Bug #85019: Recycler folder detection loop does not end added
Actions
Copy link
 #32

Updated by Nicole Cordes over 6 years ago

  • Related to Bug #85079: You are not allowed to access the given folder: "user_upload" added
Actions
Copy link
 #33

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link
 #34

Updated by Markus Klein over 5 years ago

  • Related to Bug #88632: File cannot be deleted from a storage that is not browsable added
Actions
Copy link

Also available in: Atom PDF