Project

General

Profile

Actions

Bug #103591

open

Downloading single file in Filelist module triggers zipfile creation

Added by Andreas Kießling 8 months ago. Updated 4 months ago.

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

0%

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 8 months ago

Actions #2

Updated by Georg Ringer 5 months ago

  • Sprint Focus set to Needs Decision
Actions #3

Updated by Benni Mack 5 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 4 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 4 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 4 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 4 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 4 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

Also available in: Atom PDF