Project

General

Profile

Actions

Bug #92188

closed

Deferred image processing in the backend is only partly implemented

Added by Helmut Hummel over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-09-03
Due date:
% Done:

100%

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

Description

The following issues exist with the current implementation:

1. Image processing is only deferred, when implemented in code

Every place in the backend that processes images (inkluding third party code) needs to use
the provided API to pass the images through a processing script

2. Even if the images are already processed, the request goes through PHP and the TYPO3 bootstrap

This fact is a waste of resources on every side. Images aren't cached by browsers, thus downloaded on every request,
PHP is triggered with heavy TYPO3 boostrap consuming resources on the server side

3. Target width and height aren't known upfront, causing layout shifts while loading the images

Due to the current implementation of just generating a processing URL, the final dimensions of the image are not known
and thus not pssed into the resulting HTML code. This leads to layout shifts once the browser finally fetches the image with its dimensions


Related issues 4 (1 open3 closed)

Related to TYPO3 Core - Feature #81989: ImageMagick - Prevent Concurrent Processing of Image FilesAccepted2017-07-27

Actions
Related to TYPO3 Core - Bug #93133: Not processing images when indexing in SolrClosed2020-12-21

Actions
Related to TYPO3 Core - Bug #93245: Images not processed in Scheduler task since version 10.4.11Closed2021-01-08

Actions
Related to TYPO3 Core - Task #101992: Remove unused ThumbnailControllerClosedOliver Bartsch2023-09-21

Actions
Actions #1

Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

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/c/Packages/TYPO3.CMS/+/65237

Actions #2

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #3

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #4

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #5

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #6

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #7

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #8

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #9

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #10

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #11

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/65237

Actions #12

Updated by Gerrit Code Review over 3 years ago

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

Actions #13

Updated by Gerrit Code Review over 3 years ago

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

Actions #14

Updated by Markus Klein over 3 years ago

  • Related to Feature #81989: ImageMagick - Prevent Concurrent Processing of Image Files added
Actions #15

Updated by Gerrit Code Review over 3 years ago

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

Actions #16

Updated by Gerrit Code Review over 3 years ago

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

Actions #17

Updated by Gerrit Code Review over 3 years ago

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

Actions #18

Updated by Gerrit Code Review over 3 years ago

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

Actions #19

Updated by Gerrit Code Review over 3 years ago

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

Actions #20

Updated by Gerrit Code Review over 3 years ago

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

Actions #21

Updated by Gerrit Code Review over 3 years ago

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

Actions #22

Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch 10.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/+/66587

Actions #23

Updated by Helmut Hummel over 3 years ago

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

Updated by Tymoteusz Motylewski over 3 years ago

  • Related to Bug #93133: Not processing images when indexing in Solr added
Actions #25

Updated by Benni Mack over 3 years ago

  • Status changed from Resolved to Closed
Actions #26

Updated by Daniel Goerz about 3 years ago

  • Related to Bug #93245: Images not processed in Scheduler task since version 10.4.11 added
Actions #27

Updated by Oliver Bartsch 6 months ago

  • Related to Task #101992: Remove unused ThumbnailController added
Actions

Also available in: Atom PDF