Project

General

Profile

Actions

Bug #75014

closed

l18n_cfg not respected in fluid_styled_contents of type special menu

Added by Daniel Siepmann about 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid Styled Content
Start date:
2016-03-10
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

The result
Pages in default location are shown in menus, even if they should not.

How to reproduce
Create a page in default language, select option "Hide default translation of page" under "Behaviour".
Create a content element of kind "special menus" and select the created page. Create the element in default language.
The page will be shown and link is broken as page should not be visible because it's not available in default language.
https://docs.typo3.org/typo3cms/FrontendLocalizationGuide/BasicSetupOfALocalizedWebsite/LocalizationOverviewOfThePageTree/Index.html#hiding-default-translation-of-pages

For our own implementation we had to add the following SQL to fix the issue:

        if ($this->arguments[static::ARGUMENT_TABLE] === 'pages') {
            $where .= ' AND pages.l18n_cfg != 1';
        }


Related issues 2 (0 open2 closed)

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 #82122: Typo3DbBackend::doLanguageAndWorkspaceOverlay ignores pages l18n_cfgClosed2017-08-17

Actions
Actions #1

Updated by Andreas Kienast about 8 years ago

  • Status changed from New to Needs Feedback
  • Assignee deleted (Benni Mack)

Which TYPO3 version are you using? I can't reproduce the issue in 7.6.4.

Actions #2

Updated by Daniel Siepmann about 8 years ago

We are running 7.6.4 installed via composer.

Take note that this is related to fluid styled content, not TypoScript rendering.
The ViewHelpers are using Core Functionality which doesn't take this option into account.
So

getSubpagesForPages
should add this condition in my opinion.

If needed, I can show you via screen sharing. Just contact me on slack: https://typo3.slack.com/team/danielsiepmann

Actions #3

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Needs Feedback 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/47268

Actions #4

Updated by Gerrit Code Review about 8 years ago

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

Actions #5

Updated by Gerrit Code Review about 8 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/47269

Actions #6

Updated by Gerrit Code Review about 8 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/47269

Actions #7

Updated by Gerrit Code Review about 8 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/47269

Actions #8

Updated by Benni Mack almost 8 years ago

  • Target version changed from 7.6.5 to Candidate for patchlevel
Actions #9

Updated by Andreas Kienast about 7 years ago

  • Status changed from Under Review to New
Actions #10

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from New to Needs Feedback

Is it still true for version 8.7.9? I tried to reproduce the issue with the following test:

Prerequisites

- TYPO3 8.7.9
- 2 languages (default (ITA)) and English (ID=1)
- the following pagetree

-+[ID=1] Home
  |
  +-- [ID=2] TEST 1
  |
  +-- [ID=21] TEST 2
  |
  +-- [ID=24] TEST 3

- All the pages are translated in English

Test

- Edit the page with ID=24 in Default Language > Tab Language > check "Hide default translation of page"
- Home page > Create a new content element, type "Menu of these pages"
- Select the three pages to be included in the menu

Result

When viewing the page, only the pages ID=2 and ID=21 are visible in the menu.

Could you confirm? Or add some more info about how to reproduce the issue, including your configuration and a step-by-step guide would be great. Thanks .

Actions #11

Updated by Alexander Opitz almost 6 years ago

  • Status changed from Needs Feedback to Closed

No feedback within the last 90 days => closing this issue.

If you think that this is the wrong decision or experience this issue again, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.

Thank you and best regards

Actions #12

Updated by Tymoteusz Motylewski almost 6 years ago

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

Updated by Tymoteusz Motylewski almost 6 years ago

  • Related to Bug #82122: Typo3DbBackend::doLanguageAndWorkspaceOverlay ignores pages l18n_cfg added
Actions

Also available in: Atom PDF