Story #82206: list module enhancements/bugfixes
Language and colpos changes on re-ordering of Content Elements
If content elements on a page are moved up and down in the order, they get the sys_language_uid assigned of the record that they jumped over.
Steps to reproduce:¶
- go to the list module (extended view + localization view active)
- create a page
- add a content element, name it "foo" and assign it the language "all languages"
- add a second content element, name it "bar" and leave it on language "default"
The list now looks like this:
- foo - all languages
- bar - default
Now click the arrow down, to move "foo" down after "bar". Expected result is that only the order changes. But now, the order changes AND the language "default" will also be set for "bar", resulting in the following list:
- foo - default
- bar - default
- Disabling the Localization View has no effect -> bug is reproducable independent of that checkbox
- Same mistake happens in the Page Module
#2 Updated by Mario Rimann over 7 years ago
Steffen Gebert wrote:
Can you reproduce this with older TYPO3 versions?
I did not explicitely test it today in older versions, but my colleague who found this issue in a customer project is working in a 4.5.x environment - so I'm pretty sure it's also reproducable in 4.5.x and up to master (tested with a 6.0-dev today).
#5 Updated by Mario Rimann over 6 years ago
Markus Klein wrote:
Can you please check whether there is a hook registered for:
On the installation where this was reported, the "multicolumn" extension is installed, which adds itself to exactly this hook. So I went to test it on another 4.5.x based setup, where multicolumn is not installed. So the only thing that is hooked into the mentioned hook (in both installations) is:
The issue is reproducable also in this setup with only this hook in place.
#7 Updated by Markus Klein about 6 years ago
- Category set to Backend API
- Status changed from New to Accepted
- Priority changed from Should have to Must have
- Target version set to next-patchlevel
- Complexity set to hard
- Is Regression set to No
I can reproduce this on current master (6.2) too. Strangely this only happens when moving things "down"!
#12 Updated by Markus Klein over 2 years ago
- Subject changed from Language changes on re-ordering of Content Elements to Language and colpos changes on re-ordering of Content Elements
- Target version set to Candidate for patchlevel
- Complexity changed from hard to medium
Can be reproduced in all versions independent of any third party extension.
The problem is that \TYPO3\CMS\Core\DataHandling\DataHandler::fixCopyAfterDuplFields() being called on a move command.
Specifically, when you have a CE list like that (list module):
- Title1 (colPos = 0)
- Title2 (colPos = 2)
- Title3 (colPos = 2)
If one clicks the up-button to move Title3 before Title2, the colPos AND sys_language_uid of Title3 is being set to zero.
=> Hence, it shows up below Title1, but in the wrong column