Bug #82122

Typo3DbBackend::doLanguageAndWorkspaceOverlay ignores pages l18n_cfg

Added by Christian Weiske over 1 year ago. Updated 7 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2017-08-17
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Page records have a l18n_cfg bit field that has one bit for "Hide page if no translation for current language exists".

TYPO3\CMS\Extbase\Persistence\Generic\Storage\Typo3DbBackend::doLanguageAndWorkspaceOverlay() does ignore this setting, so that pages without a translation into the current language are returned despite the "hide page if not translation" setting being active.

The method relies on
TYPO3\CMS\Frontend\Page\PageRepository::getPagesOverlay(), which also ignores l18n_cfg.

One of those methods should take care of handling the page l18n_cfg property.

Steps to reproduce:

  1. Setup a second language in TYPO3
  2. Create a page with "Hide page if no translation for current language exists" set
  3. In the frontend page with L=1, use Extbase\Persistence\Generic\Query to fetch this page
  4. The page will be returned, but should not

Related issues

Related to TYPO3 Core - Bug #84139: l18n_cfg not respected in sysext/frontend/Classes/Page/PageRepository.php Under Review 2018-03-05
Related to TYPO3 Core - Bug #85063: \TYPO3\CMS\Frontend\Typolink\PageLinkBuilder::build doesn't honor l18n_cfg settings Closed 2018-05-22
Related to TYPO3 Core - Bug #75014: l18n_cfg not respected in fluid_styled_contents of type special menu Closed 2016-03-10

History

#1 Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

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

#2 Updated by Gerrit Code Review over 1 year ago

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

#3 Updated by Gerrit Code Review about 1 year ago

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

#4 Updated by Gerrit Code Review about 1 year ago

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

#5 Updated by Gerrit Code Review about 1 year ago

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

#6 Updated by Benni Mack about 1 year ago

  • Sprint Focus set to On Location Sprint

#7 Updated by Gerrit Code Review about 1 year ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54966

#8 Updated by Gerrit Code Review about 1 year ago

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

#9 Updated by Gerrit Code Review 10 months ago

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

#10 Updated by Gerrit Code Review 10 months ago

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

#11 Updated by Gerrit Code Review 10 months ago

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/53729

#12 Updated by Wolfgang Klinger 9 months ago

  • Related to Bug #84139: l18n_cfg not respected in sysext/frontend/Classes/Page/PageRepository.php added

#13 Updated by Mathias Brodala 9 months ago

  • Sprint Focus deleted (On Location Sprint)

#14 Updated by Gerrit Code Review 7 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/53729

#15 Updated by Tymoteusz Motylewski 7 months ago

  • Related to Bug #85063: \TYPO3\CMS\Frontend\Typolink\PageLinkBuilder::build doesn't honor l18n_cfg settings added

#16 Updated by Tymoteusz Motylewski 7 months ago

  • Related to Bug #75014: l18n_cfg not respected in fluid_styled_contents of type special menu added

Also available in: Atom PDF