Project

General

Profile

Actions

Task #97654

open

[BUGFIX] Load only LIVE records in checkIfPageIsHidden

Added by TYPO3 GmbH TYPO3com about 2 years ago. Updated 12 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Workspaces
Target version:
-
Start date:
2022-05-18
Due date:
% Done:

0%

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

Description

The `checkIfPageIsHidden` queries the page and afterwards performs the workspace overlay.
However the first query can also load workspaced records, because the restriction is missing, which means changes in the workspace can affect the preview functionality of LIVE.
Most of the times this won't break, because of the `LIMIT 1` and normally in the database the WS records will be stored after the LIVE record.

We first identified this bug in TYPO3 v10, where this bug is in `TypoScriptFrontendController`: https://github.com/TYPO3/typo3/blob/11.5/typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php#L852
There you can see there is a check for `pid > 0` in the query which is the legacy way of saying "no workspace version".

This issue was automatically created from https://github.com/TYPO3/typo3/pull/380

Actions

Also available in: Atom PDF