Bug #32967

Bug #46389: Faulty Workspace Placeholders

Backend workspace copy/cut paste column content fails on publish

Added by Jakob Berlin over 1 year ago. Updated 2 months ago.

Status:Under Review Start date:2012-01-04
Priority:Should have Due date:
Assignee:- % Done:

0%
Category:-
Target version:4.5.12
TYPO3 Version:4.5 Complexity:
PHP Version:5.3
Votes: 1 (View)

Description

1. Go to draft workspace
2. Cut content from normal column
3. Paste content into right column
4. Publish changes

-> Copied content appears in normal column again.

Any idea where this happens? I'm maybe able to fix it if I know where this happens...

1.png - Step 1 (53.7 kB) Jakob Berlin, 2012-01-04 15:41

3.png - Step 2 (54.6 kB) Jakob Berlin, 2012-01-04 15:41

wrongtextelementcolumns.JPG - DB behavoir in Typo3 4.5.5 (78.6 kB) Jakob Berlin, 2012-01-04 15:41

bug_32967.diff - Changes in typo3_src-4.5.10/typo3/sysext/version/class.tx_version_tcemain.php which fixes bug for me (1.6 kB) Jakob Berlin, 2012-01-04 17:54

Related issues

related to Core - Bug #44470: Content elements in wrong column in page module Resolved 2013-01-13

History

#1
Updated by Jakob Berlin over 1 year ago

In Typo3 4.5.10 the screenshot of DB behavoir is not reproduceable for me. It seems that something changed here. Now a row with MOVE-TO PLACEHOLDER and setup "t3ver_move_id" appears before publish and disappears after publish. In the "new" row the colpos value is right but not seems to be used...

#2
Updated by Jakob Berlin over 1 year ago

After hours of researching through the core I found the (a?) location where I can fix it. See diff file. Is this the right/best/only location? Someone with more Typo3-Core-Skill should check this.

If fine I can commit this to git repo.

#3
Updated by Simon Schaufelberger over 1 year ago

If fine I can commit this to git repo.

you can ALWAYS push to gerrit. code review will be done there and not HERE in forge so go ahead and push!
Then it is also easier to review since you can just checkout your commit into your own local repo.

#4
Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7691

#5
Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7691

#6
Updated by Gerrit Code Review over 1 year ago

removed (created new change by accident)

#7
Updated by Gerrit Code Review over 1 year ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7691

#8
Updated by Ernesto Baschny over 1 year ago

  • Target version changed from 4.5.11 to 4.5.12

#9
Updated by Gerrit Code Review over 1 year ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7691

#10
Updated by Gerrit Code Review over 1 year ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7691

#11
Updated by Gerrit Code Review 3 months ago

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18107

#12
Updated by Oliver Hader 2 months ago

  • Parent task set to #46389

Also available in: Atom PDF