Deprecate EuLoginUser::getUser() method.

src/DependencyInjection/EuLoginExtension.php

@@ -16,7 +16,6 @@ class EuLoginExtension extends Extension
 {
     /**
      * @param array<string> $configs
-     * @param ContainerBuilder $container
      */
     public function load(array $configs, ContainerBuilder $container): void
     {

src/Security/Core/User/EuLoginUser.php

@@ -21,8 +21,6 @@ final class EuLoginUser implements EuLoginUserInterface
 
     /**
      * EuLoginUser constructor.
-     *
-     * @param \EcPhp\CasBundle\Security\Core\User\CasUserInterface $user
      */
     public function __construct(CasUserInterface $user)
     {
@@ -264,14 +262,21 @@ public function getUid(): ?string
      */
     public function getUser(): string
     {
-        return $this->user->getUser();
+        trigger_deprecation(
+            'ecphp/eu-login-bundle',
+            '2.2.3',
+            'The method "%s::getUser()" is deprecated, use %s::getUsername() instead.',
+            EuLoginUser::class
+        );
+
+        return $this->user->getUsername();
     }
 
     /**
      * {@inheritdoc}
      */
     public function getUsername()
     {
-        return $this->getUser();
+        return $this->user->getUsername();
     }
 }

src/Security/Core/User/EuLoginUserInterface.php

@@ -11,103 +11,52 @@
  */
 interface EuLoginUserInterface extends CasUserInterface
 {
-    /**
-     * @return string|null
-     */
     public function getAssuranceLevel(): ?string;
 
     /**
      * @return string[]
      */
     public function getAuthenticationFactors(): array;
 
-    /**
-     * @return string|null
-     */
     public function getDepartmentNumber(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getDomain(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getDomainUsername(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getEmail(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getEmployeeNumber(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getEmployeeType(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getFirstName(): ?string;
 
     /**
      * @return mixed[]
      */
     public function getGroups(): array;
 
-    /**
-     * @return string|null
-     */
     public function getLastName(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getLocale(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getLoginDate(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getOrgId(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getSso(): ?string;
 
     /**
      * @return string[]
      */
     public function getStrengths(): array;
 
-    /**
-     * @return string|null
-     */
     public function getTelephoneNumber(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getTeleworkingPriority(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getTicketType(): ?string;
 
-    /**
-     * @return string|null
-     */
     public function getUid(): ?string;
 }

src/Security/Core/User/EuLoginUserProviderInterface.php

@@ -14,8 +14,6 @@
 interface EuLoginUserProviderInterface extends CasUserProviderInterface
 {
     /**
-     * @param ResponseInterface $response
-     *
      * @return CasUserInterface|EuLoginUserInterface
      */
     public function loadUserByResponse(ResponseInterface $response): CasUserInterface;