Project

General

Profile

Actions

Bug #103591

closed

Downloading single file in Filelist module triggers zipfile creation

Added by Andreas Kießling 10 months ago. Updated 2 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2024-04-10
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
On Location Sprint

Description

After updating a project to 12.4 i was not able to download large files in the backend anymore.
It seems like with https://review.typo3.org/c/Packages/TYPO3.CMS/+/77592 the direct link to the file with a download attribute was substituted with a link to the /typo3/file/download middleware (\TYPO3\CMS\Filelist\Controller\FileDownloadController::class . '::handleRequest')

See
https://review.typo3.org/c/Packages/TYPO3.CMS/+/77592/41/typo3/sysext/filelist/Classes/FileList.php , search for download=" to get to the previous implementation.

This is an unnecessary processing step, waste of CPU power and introduces wait times for editors


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #99861: Add tile view to element browserClosed2023-02-07

Actions
Actions #1

Updated by Andreas Kießling 10 months ago

Actions #2

Updated by Georg Ringer 7 months ago

  • Sprint Focus set to Needs Decision
Actions #3

Updated by Benni Mack 7 months ago

  • Status changed from New to Accepted
  • Sprint Focus changed from Needs Decision to On Location Sprint
Actions #4

Updated by Gerrit Code Review 6 months ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/85474

Actions #5

Updated by Marc Willmann 6 months ago

For the record: as a workaround an editor may download the file independent from the proposed patch with download from the context menu (this is always unzipped!).

But the case is valid, if only one file is selected it would be much better to not zip an archive around this file; this is what the proposed patch is doing.

Actions #6

Updated by Gerrit Code Review 6 months ago

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

Actions #7

Updated by Gerrit Code Review 6 months ago

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

Actions #8

Updated by Gerrit Code Review 6 months ago

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

Actions #9

Updated by Gerrit Code Review 4 days ago

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87853

Actions #10

Updated by Gerrit Code Review 2 days ago

Patch set 1 for branch 13.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87869

Actions #11

Updated by Anonymous 2 days ago

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

Also available in: Atom PDF