Bug #43949
closed
Cleanups for the Authentication Classes
0%
Description
For the classes:
\TYPO3\CMS\Core\Authentication\AbstractUserAuthentication
\TYPO3\CMS\Core\Authentication\AbstractUserAuthenticationFrontendUserAuthtenication (did you see the typo in the class name? :/ )
\TYPO3\CMS\Core\Authentication\BackendUserAuthentication
\TYPO3\CMS\Backend\FrontendBackendUserAuthentication
do the following cleanups:
- provide proper phpDoc comments for all properties and methods
- remove a lot of "@todo Define visibility" annotations for properties and methods that already have a visibility definition
- some minor cleanups
Updated by Gerrit Code Review over 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17148
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17148
Updated by Gerrit Code Review over 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17148
Updated by Gerrit Code Review over 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17148
Updated by Wouter Wolters about 11 years ago
- Status changed from Under Review to Closed
Abandoned in Gerrit
Author: "@todo Define visilibility" shouldn't be removed and patch can't merge anymore and was too big anyway