Task #56994
closed
Epic #55070: Workpackages
Epic #54851: WP: Workspaces IRRE & MM bugfixes
Story #54853: As a user I want to correctly create, modify, preview and publish IRRE & MM related workspace changes
Task #56376: Invalid relations of IRRE records in workspaces
Introduce shadowColumnsForMovePlaceholders property
100%
Description
$TCA[<table>]['ctrl']['shadowColumnsForMovePlaceholders'] behaves similar to shadowColumnsForNewPlaceholders but is only used for move placeholders.
Updated by Gerrit Code Review over 10 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/28473
Updated by Thorsten Kahler over 10 years ago
- Status changed from Under Review to Needs Feedback
Why do we need another config option for move placeholders? What's the difference to new placeholders in this context?
Updated by Timo Webler over 10 years ago
Available changes for move placeholder:
Updated by Oliver Hader over 10 years ago
Thorsten Kahler wrote:
Why do we need another config option for move placeholders? What's the difference to new placeholders in this context?
The difference is "it is used for move placeholders" only... as written...
The reason for that is the way move-placeholders are overlaid in the frontend.
See https://review.typo3.org/#/c/27774/ and the accordant CSV tests on sorting/moving things...
Updated by Oliver Hader over 10 years ago
Timo Webler wrote:
For IRRE that's not enough - the action happens in DataHandlerHook::moveRecordAvailable changes for move placeholder:
- DataHandler::versionizeRecord (creates version and the new placeholder)
- DataHanderHook::moveRecord_wsPlaceholders (creates move placeholder after everything was processed and remapped (remapStack))
- https://travis-ci.org/ohader/TYPO3.CMS/jobs/20950988 (the expected values are not set, thus misusing the newPlaceHolder property is not enough)
- https://travis-ci.org/ohader/TYPO3.CMS/jobs/20951081 (disabled the DataSet assertions - thus behaviour in frontend is not as expected)
Updated by Gerrit Code Review over 10 years ago
- Status changed from Needs Feedback to Under Review
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28473
Updated by Oliver Hader over 10 years ago
Gerrit Code Review wrote:
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28473
This change combines the new property for "move" with the old property for "new" as a fallback...
Thus, one can decide whether to override/disable on demand.
Updated by Gerrit Code Review over 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 https://review.typo3.org/28473
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/28473
Updated by Gerrit Code Review over 10 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28473
Updated by Oliver Hader over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 6e9fd7da1b26e2860d4ce9ee9dd01f666d89be34.
Updated by