Project

General

Profile

Actions

Bug #81836

closed

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

Added by Alexander Grein over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Recycler
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 open4 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 #1

Updated by Alexander Grein over 6 years ago

This problem also exists in TYPO3 7.6.20

Actions #2

Updated by Riccardo De Contardi about 6 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 #3

Updated by Alexander Grein about 6 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 #4

Updated by Riccardo De Contardi about 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 #5

Updated by Guido S. almost 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 #6

Updated by Gerrit Code Review almost 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 #7

Updated by Guido S. almost 6 years ago

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

Updated by Riccardo De Contardi almost 6 years ago

  • Status changed from Under Review to Closed
Actions #9

Updated by Riccardo De Contardi almost 6 years ago

  • Status changed from Closed to Under Review

sorry, wrong closing

Actions #10

Updated by Gerrit Code Review almost 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 #11

Updated by Gerrit Code Review almost 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 #12

Updated by Gerrit Code Review almost 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 #13

Updated by Gerrit Code Review almost 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 #14

Updated by Gerrit Code Review almost 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 #15

Updated by Gerrit Code Review almost 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 #16

Updated by Gerrit Code Review almost 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 #17

Updated by Gerrit Code Review almost 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 #18

Updated by Gerrit Code Review almost 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 #19

Updated by Gerrit Code Review almost 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 #20

Updated by Gerrit Code Review almost 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 #21

Updated by Gerrit Code Review almost 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 #22

Updated by Gerrit Code Review almost 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 #23

Updated by Gerrit Code Review almost 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 #24

Updated by Gerrit Code Review almost 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 #25

Updated by Gerrit Code Review almost 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 #26

Updated by Gerrit Code Review almost 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 #27

Updated by Gerrit Code Review almost 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 #28

Updated by Guido S. almost 6 years ago

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

Updated by Gerrit Code Review almost 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 #30

Updated by Nicole Cordes almost 6 years ago

  • Status changed from Under Review to Resolved
Actions #31

Updated by Alexander Stehlik almost 6 years ago

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

Updated by Nicole Cordes almost 6 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #34

Updated by Markus Klein almost 5 years ago

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

Also available in: Atom PDF