Bug #92783: Record inserted on wrong pid - TYPO3 Core - TYPO3 Forge

Custom queries

Accessibility
Easy tasks
Forge Triage
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
My open issues
No feedback within 90 days
Open Bugs
Open issues of 10
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (2)

Karina Helena Reinhardt
Markus Klein

Actions

Copy link

Bug #92783

closed

Record inserted on wrong pid

Added by Philipp Parzer about 4 years ago. Updated over 3 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Philipp Parzer

Category:

Target version:

Candidate for patchlevel

Start date:

2020-11-06

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

If a record is localized and then moved to another page(A), the l10n_source field of the translated record on the old page(B) points to a record on a different pid(A).

If you create a new content element after the moved item on the new page(A) and want to translate it, the translated record of the new element is inserted on the old pid(B) because TYPO3 selects the record before itself by order and then selects the localized record by l10n_source which is on the old page(B).

This can lead to strange situations if records are copied and/or moved to different pages.

In this commit, the behaviour to use the l10n_source rather than the l18n_parent was introduced:
https://github.com/TYPO3/TYPO3.CMS/commit/3c6cdaedd288a3514d48e99fe298870606769201

Related issues 2 (0 open — 2 closed)

Related to TYPO3 Core - Bug #92198: Localize method gets wrong PID from records generated during former free mode translations

Closed

2020-09-04

Actions

Related to TYPO3 Core - Bug #93078: 9.5/10.4: DataHandler::getPreviousLocalizedRecordUid() does not check pid

Closed

2020-12-15

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Christian Kuhn about 4 years ago

Patch pending with https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552 - no idea why forge did not pick that one up.

Actions

Copy link

Updated by Gerrit Code Review about 4 years ago

Status changed from New to Under Review

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/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

Updated by Markus Klein about 4 years ago

I didn't get the problem completely:

What is moved, a translation or the original record?
Where does the source point to, the default language or some other translation?
Is the translation a free mode translation?

Actions

Copy link

Updated by Philipp Parzer about 4 years ago

The original record is moved - it is translated in free mode, so the translation doesn't get moved and stays on the old page (connected mode would move the translation too)
The source of the translation still points to the original record which is now on another page
As mentioned above its in free mode

If the moving process is correct or not, I cannot say - but nevertheless it should never ever happen, that a translation is inserted on another page, than the one I'm currently translating.

Actions

Copy link

Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

Updated by Philipp Parzer about 4 years ago

Related to Bug #92198: Localize method gets wrong PID from records generated during former free mode translations added

Actions

Copy link

Updated by Philipp Parzer about 4 years ago

Assignee set to Philipp Parzer

Actions

Copy link

Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

Updated by Gerrit Code Review almost 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#10

Updated by Gerrit Code Review almost 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#11

Updated by Christian Kuhn over 3 years ago

Seems we're suffering from that in a project, too. Will fix / finish the patch now.

Actions

Copy link

#12

Updated by Gerrit Code Review over 3 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#13

Updated by Gerrit Code Review over 3 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#14

Updated by Gerrit Code Review over 3 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#15

Updated by Gerrit Code Review over 3 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#16

Updated by Gerrit Code Review over 3 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#17

Updated by Gerrit Code Review over 3 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#18

Updated by Gerrit Code Review over 3 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#19

Updated by Gerrit Code Review over 3 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66552

Actions

Copy link

#20

Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/68612

Actions

Copy link

#21

Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/68612

Actions

Copy link

#22