Project

General

Profile

Actions

Bug #79934

closed

Localized one/many to many not mapped correctly

Added by Axel Helmert almost 8 years ago. Updated about 6 years ago.

Status:
Rejected
Priority:
Must have
Assignee:
-
Category:
Extbase + l10n
Target version:
-
Start date:
2017-02-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
persistence mapping
Complexity:
medium
Is Regression:
Yes
Sprint Focus:

Description

We've experienced issues when defining a reference like a file reference with \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::getFileFieldTCAConfig() in TCA and translating the record in the TYPO3 list Module.
This is not limited to file references, it also affects any field defined with type inline and [behaviour][localizationMode] = 'select'.

Steps to reproduce:

  1. Define a domain model with a file reference.
  2. Create a page with multiple languages
  3. Create a record for your domain model in the TYPO3 list module
  4. Translate this record to another language
  5. In the translation record change the file reference to another file

Actual behavior:

In backend everything looks file the Default language of the record has file A and the translation record has file B.
In the frontend however, the record itself seems to be translated when switching languages, but always references file A.

Expected behavior:
The translated record should have a reference to file B in frontend.

I have implemented a fix and will push it to gerrit.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #82363: Make Extbase translation handling consistent with typoscriptClosedTymoteusz Motylewski2017-09-07

Actions
Is duplicate of TYPO3 Core - Bug #57272: Extbase doesn't handle FAL translations correctlyClosed2014-03-25

Actions
Actions #1

Updated by Gerrit Code Review almost 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51778

Actions #2

Updated by Axel Helmert almost 8 years ago

I just saw there already is an issue for this. I linked it with a duplication note.

Actions #3

Updated by Riccardo De Contardi about 7 years ago

  • Category changed from Extbase to Extbase + l10n
Actions #4

Updated by Gerrit Code Review almost 7 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/55646

Actions #5

Updated by Gerrit Code Review almost 7 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/55646

Actions #6

Updated by Gerrit Code Review almost 7 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/55646

Actions #7

Updated by Alexander Opitz over 6 years ago

  • Status changed from Under Review to Needs Feedback

Hi,

thanks for the patch. As you already marked this issue as duplicate of #57272 and the patch series there looks better, I like to close this issue and abandone the patch series here.
Is this ok?

Actions #8

Updated by Axel Helmert over 6 years ago

As long as the Patch for #57272 does fix the issue for all relations and not just FAL, I'm fine with it.

Actions #9

Updated by Alexander Opitz over 6 years ago

The patch goes in the same direction, did you test the patch?

Actions #10

Updated by Axel Helmert over 6 years ago

Alexander Opitz wrote:

The patch goes in the same direction, did you test the patch?

Not, yet. But as far as I can see, this patch will only go to Version 8.x and above and 7.x will stay broken?

Actions #11

Updated by Markus Klein about 6 years ago

  • Related to Bug #82363: Make Extbase translation handling consistent with typoscript added
Actions #12

Updated by Christian Kuhn about 6 years ago

  • Status changed from Needs Feedback to Rejected

rejected in favor of #82363

Actions #13

Updated by Tymoteusz Motylewski about 6 years ago

please check if https://review.typo3.org/#/c/53974/ solves the issue for you

Actions

Also available in: Atom PDF