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

Respect TCA overrides for CE localization #438

Closed
wants to merge 1 commit into from
Closed

Respect TCA overrides for CE localization #438

wants to merge 1 commit into from

Conversation

kauz56
Copy link

@kauz56 kauz56 commented Jun 16, 2023

I have a few general purpose columns, that are shared between CTypes. The implementation differs between them, and is setup via overrides for the specific CType.

DataHandler->localize doesn't seem to respect column overrides for CTypes, resulting in settings like l10n_mode being ignored, if they are set via CType specific override.

The scenario where I ran into this, was trying to translate one of the general purpose columns via https://github.com/web-vision/wv_deepltranslate/. The columns were ignored, because it didn't detect the 'l10n_mode' => 'prefixLangTitle', which had to be set via overrides in my case.

This pull request adds the handling of CType specific overrides in DataHandler->localize.

@TYPO3IncTeam
Copy link

Thank you for your contribution to TYPO3. We are using Gerrit Code Review for our contributions and took the liberty to convert your pull request to a review in our review system.
You can find your patch at: https://review.typo3.org/c/Packages/TYPO3.CMS/+/79441
For further information on how to contribute have a look at https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/

@TYPO3 TYPO3 locked and limited conversation to collaborators Jun 16, 2023
@kauz56 kauz56 deleted the respect-tca-overrides-for-ce-localization branch July 16, 2023 19:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants