Task #69721
closed
Story #69712: Further FormEngine development
Inline BackendUtility::getRecordTitle as data provider
100%
Description
BackendUtility::getRecordTitle() is used within FormEngine at various places. It does tons of different data fetching operations and is in general a rather huge headache.
Get rid of this in FormEngine scope:
- Add a "recordTitle" field to the data array in FormDataCompiler that can be used by render part.
- Extend removeObsoleteColumns() in TcaTypesShowItem method to not remove fields that are used in one of the title ctrl fields mentioned in getRecordTitle. Maybe extract this method to an own provider that is executed right after TcaTypesShowItem provider. This has the effect, that the following field specific providers like TcaSelect and friends will calculated items and values for fields that are used within title, but that are now within showitem, too.
- Add a provider that is executed after the Tca* providers that fills the "recordTitle" field of the data array.
Updated by Christian Kuhn about 9 years ago
- Complexity changed from medium to hard
Updated by Nicole Cordes about 9 years ago
- Status changed from New to Needs Feedback
Hey Christian,
I had a look at the used BackendUtility::getRecordTitle and had to confess that the title is crawled for related records (e.g. parent page, inline elements) but not for the record itself. IMHO we cannot get rid of the calls of getRecordTitle but maybe we should introduce an internal caching in there.
Updated by Christian Kuhn about 9 years ago
- Status changed from Needs Feedback to New
Still needs to be tackled. getRecordTitle within FormEngine usage should vanish and substituted by some proper data providing ... not 100% sure about all details, though.
Updated by Christian Kuhn about 9 years ago
- Assignee deleted (
Nicole Cordes)
Morton picked up this topic.
Updated by Anja Leichsenring about 9 years ago
- Sprint Focus changed from On Location Sprint to Stabilization Sprint
Updated by Morton Jonuschat about 9 years ago
- Category set to FormEngine aka TCEforms
- Status changed from New to In Progress
- Assignee set to Morton Jonuschat
Updated by Gerrit Code Review about 9 years ago
- Status changed from In Progress to Under Review
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Benni Mack about 9 years ago
- Target version changed from 7.5 to 7 LTS
- Sprint Focus deleted (
Stabilization Sprint)
Updated by Gerrit Code Review about 9 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Gerrit Code Review about 9 years ago
Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43485
Updated by Morton Jonuschat about 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 4ff5509b47ce4887ff9ff2104a7ac0d9f5fd75cc.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed