Feature #67884
closeddisplay 'lost' CEs
Added by Bernd Wilke over 9 years ago. Updated about 6 years ago.
100%
Description
gather all CEs which are not assigned to a valid column of the current BE-layout (or one of those metacolumns like 18181(fluid_content) or -2(gridelements)) and show them at the end of the page IF the user has checked to display 'lost' CEs, which could be an additional option to the page view.
the list of special columns (metacolumns) should be changeable, so that extensions like fluid_content or gridelements add their special column to the exception list.
Updated by Jo Hasenau over 9 years ago
- Status changed from New to Accepted
- Assignee set to Jo Hasenau
Updated by Susanne Moog over 9 years ago
- Target version changed from 7.4 (Backend) to 7.5
Updated by Benni Mack about 9 years ago
- Target version changed from 7.5 to 7 LTS
Updated by Benni Mack about 9 years ago
- Target version changed from 7 LTS to 8 LTS
Updated by Riccardo De Contardi over 8 years ago
you don't even have to use gridelements of flux to stumble upon this kind of issue... Imagine the following scenario (I report here the text of #76564):
1) create two backend layouts, named "test" and "test2"
this is "test":
backend_layout { colCount = 1 rowCount = 2 rows { 1 { columns { 1 { name = Main colPos = 0 } } } 2 { columns { 1 { name = Bottom colPos = 1 } } } } }
and this is "test2":
backend_layout { colCount = 1 rowCount = 2 rows { 1 { columns { 1 { name = Main colPos = 0 } } } 2 { columns { 1 { name = Bottom colPos = 2 } } } } }
2) create a page and assign "test"
3) create a content element inside the "Bottom" column and save.
4) edit the page and switch backend layout to "test2"
Result:¶
in page view the content element disappears (is "lost"), as it doesn't have a valid colPos anymore; the only easy method to have it back is using the list view (which is not meant to be used to manipulate tt_content records and in the future this practice could be disallowed).
possible solution:¶
(thanks to Mathias Schreiber for the suggestion) Add a flash message in page view with a message like:
"There are CEs that cannot be displayed for Reason X, click here to migrate them to an available column"
or add two buttons:
"There are CEs that cannot be displayed for Reason X [Migrate to an available column] [delete them!]
Updated by Riccardo De Contardi over 7 years ago
- Target version changed from 8 LTS to 9.0
Updated by Benjamin Kott over 7 years ago
- Sprint Focus set to On Location Sprint
Updated by Gerrit Code Review over 7 years ago
- Status changed from Accepted 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/53040
Updated by Gerrit Code Review over 7 years ago
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/53040
Updated by Gerrit Code Review over 7 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/53040
Updated by Gerrit Code Review over 7 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/53040
Updated by Gerrit Code Review over 7 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/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Gerrit Code Review over 7 years ago
Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53040
Updated by Jo Hasenau over 7 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset fb2fb5c40858fd5b371364c3d4717cebe60ceab5.
Updated by Gerrit Code Review over 7 years ago
- Status changed from Resolved 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/53082
Updated by Gerrit Code Review over 7 years ago
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/53082
Updated by Gerrit Code Review over 7 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/53082
Updated by Jo Hasenau over 7 years ago
- Status changed from Under Review to Resolved
Applied in changeset 7c13dc3640b40e440de90da8dd6c92395f2b46c5.
Updated by Claus Due about 7 years ago
you don't even have to use gridelements of flux to stumble upon this kind of issue.
True, but after applying this patch, now if you use those extensions all child content gets reported as lost even though it was rendered and there's no way to avoid that without adding the column to the page layout and thereby render duplicated content elements.
I don't mean to sound harsh here but I find it quite odd that you would mention these extensions as a way to experience the symptom - and then go on to create a solution that completely breaks one of those solutions... unfortunately, this has caused quite a negative effect at least for Flux, to a degree where Flux will be broken unless I manage to sneak a new signal dispatching in before the release of v9.
I'm not too happy about this, to tell you the truth.
Updated by Nicole Cordes about 7 years ago
Is there any follow up available already? What is the plan for the extensions (content elements) using IRRE with tt_content elements? Would be cool if you would add me as reviewer to any additional patch.
Thanks
Updated by Coders.Care Extension Team almost 7 years ago
- Related to deleted (Task #67134: The core must provide structured content elements which are configurable similar to backend layouts out of the box)