Minor: Format code - refs BT#21881

AngelFQC · Aug 26, 2024 · 1afd985 · 1afd985
1 parent df1bd5d
commit 1afd985
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/src/CoreBundle/Repository/ExtraFieldValuesRepository.php b/src/CoreBundle/Repository/ExtraFieldValuesRepository.php
@@ -145,6 +145,7 @@ public function findLegalAcceptByItemId($itemId)
 
     /**
      * @return ExtraFieldValues|array<ExtraFieldValues>|null
+     *
      * @throws NonUniqueResultException
      */
     public function findByVariableAndValue(

diff --git a/src/CoreBundle/Security/Authenticator/OAuth2/FacebookAuthenticator.php b/src/CoreBundle/Security/Authenticator/OAuth2/FacebookAuthenticator.php
@@ -11,7 +11,6 @@
 use Chamilo\CoreBundle\ServiceHelper\AuthenticationConfigHelper;
 use Cocur\Slugify\SlugifyInterface;
 use KnpU\OAuth2ClientBundle\Client\ClientRegistry;
-use KnpU\OAuth2ClientBundle\Client\OAuth2ClientInterface;
 use League\OAuth2\Client\Provider\FacebookUser;
 use League\OAuth2\Client\Token\AccessToken;
 use Symfony\Component\HttpFoundation\Request;
@@ -52,7 +51,7 @@ protected function userLoader(AccessToken $accessToken): User
         $user
             ->setFirstname($resourceOwner->getFirstName())
             ->setLastname($resourceOwner->getLastName())
-            //->setLocale($resourceOwner->getLocale())
+            // ->setLocale($resourceOwner->getLocale())
             ->setEmail($resourceOwner->getEmail())
             ->setUsername($this->changeToValidChamiloLogin($resourceOwner->getEmail()))
             ->setPlainPassword('facebook')

diff --git a/src/CoreBundle/Security/Authenticator/OAuth2/GenericAuthenticator.php b/src/CoreBundle/Security/Authenticator/OAuth2/GenericAuthenticator.php
@@ -90,7 +90,7 @@ protected function userLoader(AccessToken $accessToken): User
                 $user = (new User())
                     ->setFirstname('OAuth2 User default firstname')
                     ->setLastname('OAuth2 User default firstname')
-                    ->setEmail('oauth2user_'.$resourceOwnerId.'@'.(gethostname() or 'localhost'))
+                    ->setEmail('oauth2user_'.$resourceOwnerId.'@'.(gethostname() ?: 'localhost'))
                     ->setUsername($username)
                     ->setPlainPassword($username)
                     ->setStatus(STUDENT)
@@ -120,7 +120,7 @@ protected function userLoader(AccessToken $accessToken): User
     }
 
     /**
-     * Set user information from the resource owner's data or the user itself
+     * Set user information from the resource owner's data or the user itself.
      */
     public function saveUserInfo(User $user, array $resourceOwnerData, array $providerParams): void
     {

diff --git a/src/CoreBundle/Security/Authenticator/OAuth2/KeycloakAuthenticator.php b/src/CoreBundle/Security/Authenticator/OAuth2/KeycloakAuthenticator.php
@@ -7,7 +7,6 @@
 namespace Chamilo\CoreBundle\Security\Authenticator\OAuth2;
 
 use Chamilo\CoreBundle\Entity\User;
-use KnpU\OAuth2ClientBundle\Client\OAuth2ClientInterface;
 use League\OAuth2\Client\Token\AccessToken;
 use Stevenmaguire\OAuth2\Client\Provider\KeycloakResourceOwner;
 use Symfony\Component\HttpFoundation\Request;
-Original file line number
+Diff line change
@@ Expand Up / @@ -145,6 +145,7 @@ public function findLegalAcceptByItemId($itemId) @@
         /**
          * @return ExtraFieldValues|array<ExtraFieldValues>|null
+         *
          * @throws NonUniqueResultException
          */
         public function findByVariableAndValue(
@@ Expand Down @@