[TASK] Better description for parameter in AuthenticationService::getGroups() 79/40579/8
authorFrank Nägler <typo3@naegler.net>
Tue, 23 Jun 2015 21:04:13 +0000 (23:04 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Wed, 24 Jun 2015 18:55:09 +0000 (20:55 +0200)
This patch add a note why this parameter should not be removed.

Resolves: #67715
Releases: master
Change-Id: Iaefc4e3d73e1df2e34a2b9d0f60a0b9d49a158d0
Reviewed-on: http://review.typo3.org/40579
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/sv/Classes/AuthenticationService.php

index 43577b8..119397a 100644 (file)
@@ -121,6 +121,11 @@ class AuthenticationService extends AbstractAuthenticationService {
         * @return mixed Groups array, keys = uid which must be unique
         */
        public function getGroups($user, $knownGroups) {
+               /*
+                * Attention: $knownGroups is not used within this method, but other services can use it.
+                * This parameter should not be removed!
+                * The FrontendUserAuthentication call getGroups and handover the previous detected groups.
+                */
                $groupDataArr = array();
                if ($this->mode === 'getGroupsFE') {
                        $groups = array();