Task #92426
closedCleanup EXT:redirects
100%
Description
The system extension redirects received two new features recently. The code should now be revisited and streamlined.
List of things to improve:
- use singular for array command options
- clarify option description
- remove same purpose configuration class and extend demand instead
- remove proxy method in service class
- use demand as argument instead of property
- filter order field against allow list
- use dependency injection
- cleanup superfluous annotations
Updated by Gerrit Code Review about 4 years ago
- Status changed from New to Under Review
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/c/Packages/TYPO3.CMS/+/65916
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/65916
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/65916
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/65916
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/65916
Updated by Helmut Hummel about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 2ae98bbef04fb6f38b585db647c08a01e06be4e7.
Updated by Anja Leichsenring 10 months ago
- Related to Bug #102833: missing label in RedirectsCleanupCommand option statuscode added