Bug #105478: Sys log shows "[No title]" for updated records in details - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #105478

closed

Sys log shows "[No title]" for updated records in details

Added by Remo H. 5 months ago. Updated 3 months ago.

Status:

Resolved

Priority:

Should have

Assignee:

Category:

DataHandler aka TCEmain

Target version:

Start date:

2024-10-25

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

Create a "Header only" content element an set the "Header"
Save the element
In syslog you see now something like: Record "Test" (tt_content:2) was inserted on page "Home" (1)
Go back to the element and change the "Type" (header_layout)
In syslog you see now something like: Record "[No title]" (tt_content:2) was updated

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Sébastien Delcroix 5 months ago

After some quick investigation, it occurs when a content record is updated but its title is not modified.
As the header field value has not changed, DataHandler doesn't have this field when updating the record in the database and so the record title cannot be resolved and inserted in the log message.

It works in v12 though because DataHandler uses the checkStoredRecord() method that queries the record just after it has been updated in DB.
So the header field value is retrieved and the record title can be resolved and inserted in the log message.

This method has been removed in v13 with #101793 :

I'll try to provide a patch.

Actions

Copy link

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

Actions

Copy link

Updated by Gerrit Code Review 3 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/+/87496

Actions

Copy link

Updated by Garvin Hicking 3 months ago

Related to Task #101793: Remove DataHandler->checkStoredRecord() added

Actions

Copy link

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

Actions

Copy link