Bug #93078

9.5/10.4: DataHandler::getPreviousLocalizedRecordUid() does not check pid

Added by Dmitry Dulepov 12 months ago. Updated about 1 month ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
DataHandler aka TCEmain
Target version:
-
Start date:
2020-12-15
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

When creating new localizations, DataHandler::getPreviousLocalizedRecordUid() does not check pid while all other places in this class do. This causes issues in some copy/paste cases, especially if gridelements is involved.

To solve these issues, this function should use pid cause just like every other place in this class.

This is not a case for TYPO3 11 where this method is removed.


Related issues

Related to TYPO3 Core - Bug #92783: Record inserted on wrong pidClosedPhilipp Parzer2020-11-06

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

Actions
#1

Updated by Dmitry Dulepov 12 months ago

Patch to follow.

#3

Updated by Gerrit Code Review 12 months ago

  • Status changed from New 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/c/Packages/TYPO3.CMS/+/67139

#4

Updated by Gerrit Code Review 12 months 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/+/67140

#5

Updated by Christian Kuhn about 1 month ago

  • Related to Bug #92783: Record inserted on wrong pid added
#6

Updated by Christian Kuhn about 1 month ago

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

Updated by Christian Kuhn about 1 month ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF