Project

General

Profile

Actions

Task #104725

closed

Epic #104723: ext:adminpanel creates a huge mess

Massive amount of data stored and transmitted by admin panel

Added by Christian Kuhn 3 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
AdminPanel
Target version:
-
Start date:
2024-08-24
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
13
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

ModuleData of admin panel can be huge.

Especially the event module tends to have massive data, when this is enabled, and a non-cached page is called, it catapults the size of the HTML response from ~100k to 1,2MB - with a small page. Also, the cache_adminpanel_requestdata "cache" row is not ~200kb, but 16MB!

We need to revamp this mess and improve the data serialization plus the html creation side to be way more efficient, and to not go amok.

At the moment, I actually tend to at least disable the event sub module since it not only kills serverside parsetime, but browser rendering times easily, as well. Imho, we need to redesign the 'data storage' side api from scratch to prevent it from catapulting in size when anything but tiny object trees occur.

Also, the html rendering of the 'clicky' event tree is "not satisfiable" to put it mildly.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #90265: Show events triggered in current request in AdminPanelClosedSusanne Moog2020-01-31

Actions
Actions #1

Updated by Christian Kuhn 3 months ago

  • Related to Feature #90265: Show events triggered in current request in AdminPanel added
Actions #2

Updated by Christian Kuhn 3 months ago

  • Description updated (diff)
Actions #3

Updated by Christoph Lehmann 3 months ago

I think the idea of the caching table was to be able to compare requests, later. Since this functionality never went into core, i would prefer to disable the cache by default.

Actions #4

Updated by Benjamin Franzke about 1 month ago

  • Tracker changed from Bug to Task
Actions #5

Updated by Gerrit Code Review about 1 month ago

  • Status changed from New 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/+/86576

Actions #6

Updated by Gerrit Code Review about 1 month 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/+/86576

Actions #7

Updated by Gerrit Code Review about 1 month 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/+/86576

Actions #8

Updated by Gerrit Code Review about 1 month 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/+/86576

Actions #9

Updated by Gerrit Code Review about 1 month ago

Patch set 5 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/+/86576

Actions #10

Updated by Christian Kuhn about 1 month ago

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

Updated by Benni Mack about 1 month ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF