Bug #48939
closedMoving a content down moves it in another column
100%
Description
It seems there is a problem with sorting behavior on content element in page view.
It can be reproduced with the following steps :- Create a new page
- Add a new content in the normal column (text content with header set to "test1")
- Add a new content in the right column (text content with header set to "test2")
- Move test1 content up; this shouldn't be possible and it has no effect except displaying the down arrow on this content
- Move test1 content down; here again, it should not be possible. It moves this content from the normal column to the right column.
I guess this have something to do with the getResult function in class.tx_cms_layout.php (line 2061). The prev and next ids are set without taking in account the column position of the content element.
I use typo3 v4.7.12.
Files
Updated by Timo Bartholomes over 11 years ago
I have the same behavior in 4.5.27, but thought it might be related to the backend layout.
Updated by Thorsten Kahler about 11 years ago
- Category set to Backend User Interface
- Status changed from New to Accepted
- Is Regression set to Yes
This bug is caused by the faulty fix for #44470.
Updated by Daniel Wagner about 11 years ago
This also happens in TYPO3 v4.5.30, v4.5.29 and v4.5.27 (v4.5.28 not tested) - all without workspaces, but using backendlayout.
Some content elements in the first row in different columns have a move up button and some not.
After clicking the up or down button, the content element mostly will appear in a wrong colPos.
Updated by Nicole Cordes about 11 years ago
- Assignee set to Nicole Cordes
- Target version set to next-patchlevel
Updated by Gerrit Code Review about 11 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 https://review.typo3.org/25001
Updated by Gerrit Code Review about 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/25001
Updated by Gerrit Code Review about 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/25001
Updated by Siva Prasad about 11 years ago
Please check the attached diff file. For me the issue is fixed with these changes . If you guys have some better idea than this one please suggest
Updated by Sander Leeuwesteijn almost 11 years ago
It appears this issue also happens in 4.5.32!
Updated by Sander Leeuwesteijn almost 11 years ago
Can anyone tell me if this issue will be fixed in 4.5.33?
Updated by Nicole Cordes almost 11 years ago
As you can see, this patch is blocked by now. This means it cannot be merged until there is a decision which way to go (see attached review comments).
Updated by Daniel Wagner over 10 years ago
What is the actual state? Do the different opinions converge to a decision? The last entry is from Feb 26.
Updated by Gerrit Code Review over 10 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/25001
Updated by Gerrit Code Review over 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31492
Updated by Gerrit Code Review over 10 years ago
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31493
Updated by Gerrit Code Review over 10 years ago
Patch set 1 for branch TYPO3_4-5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31494
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch TYPO3_4-5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31494
Updated by Gerrit Code Review over 10 years ago
Patch set 1 for branch TYPO3_4-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31495
Updated by Nicole Cordes over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ebdd15ea62115329eb324ac29e5400e5806c01bf.
Updated by Gerrit Code Review over 10 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31497
Updated by Nicole Cordes over 10 years ago
- Status changed from Under Review to Resolved
Applied in changeset f9d05d26a3314f2354442ab3d8d0fa3da2b874c8.