Bug #33873
closedl10n_parent wrong after moving translated CEs
100%
Description
Thanks to #33592 it is now possible (again?) to move content elements which got a translation. the tranlation is now moved and stays in the correct language but it seems there is another minor bug.
The moved translation (or better I guess its placeholder) doesn't have the correct l10n_parent as there is this button in page module "Copy default content elements [1]".
could be also possible that this is a problem of the page module but I doubt.
Files
Updated by Tolleiv Nietsch almost 13 years ago
- Status changed from New to Accepted
- Assignee set to Tolleiv Nietsch
Updated by Georg Ringer almost 13 years ago
just as a hint.
count comes from class.tx_cms_layout.php
=>
function getNonTranslatedTTcontentUids($defLanguageCount, $id, $lP) { if ($lP && count($defLanguageCount)) { // Select all translations here: $queryParts = $this->makeQueryArray('tt_content', $id, 'AND sys_language_uid=' . intval($lP) . ' AND l18n_parent IN (' . implode(',', $defLanguageCount) . ')'); $result = $GLOBALS['TYPO3_DB']->exec_SELECT_queryArray($queryParts); ...
funny is phpdoc:
Selects across columns, but within in the same PID. Columns are expect to be the same for translations and original but this may be a conceptual error (?)
a workspace overlay is missing IMO here?
Updated by Georg Ringer almost 13 years ago
- File ce-diff.png ce-diff.png added
are you sure the problem is not in tcemain/tx_version?
screenshot shows the 2 content elemnts + translation
Updated by Georg Ringer over 12 years ago
How to test: use a current master
- Create 2 pages, both with translations, both with one translated content element
- Switch to workspaces
- Cut 1 content element from one page and paste it into the 2nd page
- See that at the 2nd page that there are 4 content elements, 2 in default, 2 in translations and still the button "copy default content element"
- apply patch
- reload page module of page 2
- if works, review it
Updated by Gerrit Code Review over 12 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9451
Updated by Michael Stucki almost 11 years ago
- Category changed from Bugs to Workspaces
Updated by Michael Stucki almost 11 years ago
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/9451
Updated by Gerrit Code Review almost 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/9451
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/9451
Updated by Mathias Schreiber over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset bf097f8d8683ad792d19add613c49c0a9d261679.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed