Bug #92188

Deferred image processing in the backend is only partly implemented

Added by Helmut Hummel 5 months ago. Updated 30 days 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

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 SolrNew2020-12-21

Actions
#1

Updated by Gerrit Code Review 5 months 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

#2

Updated by Gerrit Code Review 5 months 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

#3

Updated by Gerrit Code Review 5 months 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

#4

Updated by Gerrit Code Review 5 months 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

#5

Updated by Gerrit Code Review 5 months 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

#6

Updated by Gerrit Code Review 5 months 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

#7

Updated by Gerrit Code Review 5 months 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

#8

Updated by Gerrit Code Review 4 months 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

#9

Updated by Gerrit Code Review 4 months 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

#10

Updated by Gerrit Code Review 4 months 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

#11

Updated by Gerrit Code Review 4 months 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

#12

Updated by Gerrit Code Review 4 months 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

#13

Updated by Gerrit Code Review 4 months 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

#14

Updated by Markus Klein 3 months ago

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

Updated by Gerrit Code Review 3 months 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

#16

Updated by Gerrit Code Review 2 months 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

#17

Updated by Gerrit Code Review 2 months 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

#18

Updated by Gerrit Code Review 2 months 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

#19

Updated by Gerrit Code Review 2 months 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

#20

Updated by Gerrit Code Review 2 months 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

#21

Updated by Gerrit Code Review 2 months 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

#22

Updated by Gerrit Code Review 2 months 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

#23

Updated by Helmut Hummel 2 months ago

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

Updated by Tymoteusz Motylewski about 1 month ago

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

Updated by Benni Mack 30 days ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF