Actions
Bug #93078
closed
9.5/10.4: DataHandler::getPreviousLocalizedRecordUid() does not check pid
Status:
Closed
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.
Updated by Dmitry Dulepov almost 4 years ago
Compare:
- With pid: https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/typo3/sysext/core/Classes/DataHandling/DataHandler.php#L4856
- Without pid: https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/typo3/sysext/core/Classes/DataHandling/DataHandler.php#L7627 - same call, pid should be added here to produce the same result
Updated by Gerrit Code Review almost 4 years 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
Updated by Gerrit Code Review almost 4 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/+/67140
Updated by Christian Kuhn about 3 years ago
- Related to Bug #92783: Record inserted on wrong pid added
Updated by Christian Kuhn about 3 years ago
- Related to Bug #92198: Localize method gets wrong PID from records generated during former free mode translations added
Updated by Christian Kuhn about 3 years ago
- Status changed from Under Review to Resolved
Updated by
Actions