Task #84353
closedEpic #84272: Use PSR-7 ServerRequestInterface in backend controllers
Decouple TreeController from UserSettingsController
Added by Saskia Schreiber over 6 years ago. Updated almost 6 years ago.
100%
Updated by Saskia Schreiber over 6 years ago
- Status changed from New to In Progress
Updated by Saskia Schreiber over 6 years ago
- Subject changed from Use ServerRequestInterface in UserSettingsController to Refactor UserSettingsController to use a separate registry for user settings
The class UserSettingsController is currently a mixture of an ajax entry point and a registry for user related settings.
It provides an additional public method process() which is used inside the TreeController (to check for expanded pages).
This should be avoided as controllers should only provide proper actions as entry points.
Therefore there should be another class providing the settings for both UserSettingsController and TreeController.
Updated by Gerrit Code Review over 6 years ago
- Status changed from In Progress 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/56239
Updated by Mathias Brodala over 6 years ago
- Subject changed from Refactor UserSettingsController to use a separate registry for user settings to Decouple TreeController from UserSettingsController
Updated by Mathias Brodala over 6 years ago
- Blocks Task #84369: Use ServerRequestInterface in UserSettingsController added
Updated by Gerrit Code Review over 6 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/56239
Updated by Gerrit Code Review over 6 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/56239
Updated by Gerrit Code Review over 6 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/56239
Updated by Gerrit Code Review over 6 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/56239
Updated by Gerrit Code Review over 6 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/56239
Updated by Gerrit Code Review over 6 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/56239
Updated by Saskia Schreiber over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 6c2fe86ecd4141a5cfe066c3b8120028030e1f81.
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed