Auth: Replace usages for User::setAuthSource with User::addAuthSourceByAuthentication for more flexible handling of auth sources

Author: AngelFQC

src/CoreBundle/Entity/User.php

@@ -940,18 +940,6 @@ public function setClasses(Collection $classes): self
         return $this;
     }
 
-    public function getAuthSource(): ?string
-    {
-        return $this->authSource;
-    }
-
-    public function setAuthSource(string $authSource): self
-    {
-        $this->authSource = $authSource;
-
-        return $this;
-    }
-
     public function getEmail(): string
     {
         return $this->email;

src/CoreBundle/Migrations/Schema/V200/Version20230720142900.php

@@ -9,6 +9,7 @@
 use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
 use Chamilo\CoreBundle\Repository\Node\UserRepository;
+use Chamilo\CoreBundle\ServiceHelper\AccessUrlHelper;
 use Doctrine\DBAL\Schema\Schema;
 
 use const PASSWORD_DEFAULT;
@@ -48,7 +49,10 @@ public function up(Schema $schema): void
             ->setLastname('Fallback')
             ->setFirstname('User')
             ->setOfficialCode('FALLBACK')
-            ->setAuthSource('platform')
+            ->addAuthSourceByAuthentication(
+                'platform',
+                $this->container->get(AccessUrlHelper::class)->getCurrent()
+            )
             ->setPhone('0000000000')
             ->setLocale('en')
             ->setActive(User::SOFT_DELETED)

src/CoreBundle/Security/Authenticator/OAuth2/FacebookAuthenticator.php

@@ -67,7 +67,10 @@ protected function userLoader(AccessToken $accessToken): User
             ->setUsername($this->changeToValidChamiloLogin($resourceOwner->getEmail()))
             ->setPlainPassword('facebook')
             ->setStatus(STUDENT)
-            ->setAuthSource('facebook')
+            ->addAuthSourceByAuthentication(
+                'facebook',
+                $this->urlHelper->getCurrent()
+            )
             ->setRoleFromStatus(STUDENT)
         ;
 

src/CoreBundle/Security/Authenticator/OAuth2/GenericAuthenticator.php

@@ -170,7 +170,10 @@ public function saveUserInfo(User $user, array $resourceOwnerData, array $provid
                     $user->getEmail()
                 )
             )
-            ->setAuthSource('oauth2')
+            ->addAuthSourceByAuthentication(
+                'oauth2',
+                $this->urlHelper->getCurrent()
+            )
             ->setStatus($status)
             ->setRoleFromStatus($status)
         ;

src/CoreBundle/Security/Authenticator/OAuth2/KeycloakAuthenticator.php

@@ -44,7 +44,10 @@ protected function userLoader(AccessToken $accessToken): User
             ->setUsername($username)
             ->setPlainPassword('keycloak')
             ->setStatus(STUDENT)
-            ->setAuthSource('keycloak')
+            ->addAuthSourceByAuthentication(
+                'keycloak',
+                $this->urlHelper->getCurrent()
+            )
             ->setRoleFromStatus(STUDENT)
         ;
 

src/CoreBundle/ServiceHelper/AzureAuthenticatorHelper.php

@@ -67,7 +67,10 @@ public function registerUser(array $azureUserInfo, string $azureUidKey = 'object
             ->setUsername($username)
             ->setPlainPassword('azure')
             ->setStatus(STUDENT)
-            ->setAuthSource($authSource)
+            ->addAuthSourceByAuthentication(
+                $authSource,
+                $this->urlHelper->getCurrent()
+            )
             ->setPhone($phone)
             ->setActive($active)
             ->setRoleFromStatus(STUDENT)

tests/CoreBundle/Repository/Node/UserRepositoryTest.php

@@ -228,7 +228,10 @@ public function testCreateUserSkipResourceNode(): void
             ->setExpired(false)
             ->setTeach('teach')
             ->setApiToken('tok')
-            ->setAuthSource('auth')
+            ->addAuthSourceByAuthentication(
+                'auth',
+                $this->getAccessUrl()
+            )
             ->setProductions('prod')
             ->setCompetences('comp')
             ->setDiplomas('diploma')