Feature #67884

display 'lost' CEs

Added by Bernd Wilke almost 4 years ago. Updated 9 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend User Interface
Target version:
Start date:
2015-07-02
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
Sprint Focus:
On Location Sprint

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.


Related issues

Related to TYPO3 Core - Feature #76564: add a system to retrieve "lost" content elements if list view is not available Closed 2016-06-09

Associated revisions

Revision fb2fb5c4 (diff)
Added by Jo Hasenau about 2 years ago

[FEATURE] Collect lost elements and show them separately from backend layout

Resolves: #67884
Releases: master
Change-Id: I7c14886e5dc1c81d7d816e5f2eb5d36afb8cdbf9
Reviewed-on: https://review.typo3.org/53040
Tested-by: TYPO3com <>
Reviewed-by: Mona Muzaffar <>
Tested-by: Mona Muzaffar <>
Tested-by: Riccardo De Contardi <>
Tested-by: Jasmina Ließmann Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch> Tested-by: Christian Kuhn <lolli@schwarzbu.ch>

Revision 7c13dc36 (diff)
Added by Jo Hasenau about 2 years ago

[FOLLOWUP][FEATURE] Collect lost elements and show them separately

Add missing RST file for new feature

Resolves: #67884
Releases: master
Change-Id: I9d7ffa4203cbcec5dac2b5359f418faeaf5462fa
Reviewed-on: https://review.typo3.org/53082
Reviewed-by: Susanne Moog <>
Tested-by: TYPO3com <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Jo Hasenau almost 4 years ago

  • Status changed from New to Accepted
  • Assignee set to Jo Hasenau

#2 Updated by Susanne Moog almost 4 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#3 Updated by Benni Mack over 3 years ago

  • Target version changed from 7.5 to 7 LTS

#4 Updated by Benni Mack over 3 years ago

  • Target version changed from 7 LTS to 8 LTS

#5 Updated by Riccardo De Contardi about 3 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!]

#6 Updated by Riccardo De Contardi over 2 years ago

  • Target version changed from 8 LTS to 9.0

#7 Updated by Benjamin Kott about 2 years ago

  • Sprint Focus set to On Location Sprint

#8 Updated by Gerrit Code Review about 2 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

#9 Updated by Gerrit Code Review about 2 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

#10 Updated by Gerrit Code Review about 2 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

#11 Updated by Gerrit Code Review about 2 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

#12 Updated by Gerrit Code Review about 2 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

#13 Updated by Gerrit Code Review about 2 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

#14 Updated by Gerrit Code Review about 2 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

#15 Updated by Gerrit Code Review about 2 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

#16 Updated by Gerrit Code Review about 2 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

#17 Updated by Gerrit Code Review about 2 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

#18 Updated by Gerrit Code Review about 2 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

#19 Updated by Gerrit Code Review about 2 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

#20 Updated by Gerrit Code Review about 2 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

#21 Updated by Jo Hasenau about 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#22 Updated by Gerrit Code Review about 2 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

#23 Updated by Gerrit Code Review about 2 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

#24 Updated by Gerrit Code Review about 2 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

#25 Updated by Jo Hasenau about 2 years ago

  • Status changed from Under Review to Resolved

#26 Updated by Claus Due almost 2 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.

#27 Updated by Nicole Cordes over 1 year 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

#28 Updated by Coders.Care Extension Team over 1 year ago

  • Related to deleted (Task #67134: The core must provide structured content elements which are configurable similar to backend layouts out of the box)

#29 Updated by Benni Mack 9 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF