Project

General

Profile

Actions
Copy link

Bug #75014

closed

l18n_cfg not respected in fluid_styled_contents of type special menu

Added by Daniel Siepmann over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid Styled Content
Target version:
Candidate for patchlevel
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 open — 2 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
Copy link
 #1

Updated by Andreas Kienast over 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
Copy link
 #2

Updated by Daniel Siepmann over 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
Copy link
 #3

Updated by Gerrit Code Review over 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
Copy link
 #4

Updated by Gerrit Code Review over 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
Copy link
 #5

Updated by Gerrit Code Review over 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
Copy link
 #6

Updated by Gerrit Code Review over 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
Copy link
 #7

Updated by Gerrit Code Review over 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
Copy link
 #8

Updated by Benni Mack over 8 years ago

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

Updated by Andreas Kienast almost 8 years ago

  • Status changed from Under Review to New
Actions
Copy link
 #10

Updated by Riccardo De Contardi almost 7 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
Copy link
 #11

Updated by Alexander Opitz over 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
Copy link
 #12

Updated by Tymoteusz Motylewski over 6 years ago

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

Updated by Tymoteusz Motylewski over 6 years ago

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

Also available in: Atom PDF