Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix localization bug for page-layout (TYPO3 9) #413

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

ikonplant
Copy link

Add 'l10n_mode' => 'exclude' to the additional fields of page

Add 'l10n_mode' => 'exclude' to the additional fields of page
@NamelessCoder
Copy link
Member

Hi @ikonplant,

What is the bug you intend to fix? Why is the solution to make these fields not translatable?

@ikonplant
Copy link
Author

ikonplant commented Jul 1, 2019

Hi @NamelessCoder !

The reason is the new localization logic in TYPO3 9.

With your original TCA the value of tx_fed_page_controller_action is not inherited within translated pages (field value in the database).

For example, we need this value in frontend in fluid template:
<v:variable.set value="{v:page.info(field: 'tx_fed_page_controller_action')}" name="pageLayout"/>

With your original TCA, the db-field tx_fed_page_controller_action and the variable pageLayout have only value within original page/language, the localization is not synchronized.

'l10n_mode' => 'exclude', causes that the according db-field tx_fed_page_controller_action is filled within translated page (synchronized), once the layout is set/changed via backend => pageLayout has correct value.

@NamelessCoder
Copy link
Member

Thanks @ikonplant - have you checked how this affects TYPO3 8.7? I hesitate to merge since I suspect the instruction causes the field to not be used from pages_language_overlay on 8.7, which would constitute a breaking change.

For the time being, please set this from your provider extension's Configuration/TCA/Overrides/pages.php file - your provider's configuration will be loaded after fluidpages so you can freely manipulate the TCA. That way you get the desired behavior and we don't risk changing the current default behavior for people ;)

I'll leave the PR open in case others come across the use case and considers it a bug / for when the next major version of Fluidpages gets made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants