Bug #82122

Typo3DbBackend::doLanguageAndWorkspaceOverlay ignores pages l18n_cfg

Added by Christian Weiske almost 4 years ago. Updated over 2 years ago.

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

0%

Estimated time:
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.phpClosed2018-03-05

Actions
Related to TYPO3 Core - Bug #85063: \TYPO3\CMS\Frontend\Typolink\PageLinkBuilder::build doesn't honor l18n_cfg settingsClosed2018-05-22

Actions
Related to TYPO3 Core - Bug #75014: l18n_cfg not respected in fluid_styled_contents of type special menuClosed2016-03-10

Actions
#1

Updated by Gerrit Code Review almost 4 years 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 almost 4 years 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 almost 4 years 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 almost 4 years 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 almost 4 years 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 over 3 years ago

  • Sprint Focus set to On Location Sprint
#7

Updated by Gerrit Code Review over 3 years 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 over 3 years 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 over 3 years 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 over 3 years 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 over 3 years 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 over 3 years ago

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

Updated by Mathias Brodala over 3 years ago

  • Sprint Focus deleted (On Location Sprint)
#14

Updated by Gerrit Code Review about 3 years 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 about 3 years ago

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

Updated by Tymoteusz Motylewski about 3 years ago

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

Updated by Susanne Moog over 2 years ago

  • Status changed from Under Review to Closed

Fixed with https://review.typo3.org/c/Packages/TYPO3.CMS/+/57038 (Patch has been abandoned)

Also available in: Atom PDF