src/CoreBundle/Security/Authenticator/OAuth2/AbstractAuthenticator.php
The class AbstractAuthenticator has a coupling between objects value of 17. Consider to reduce the number of dependencies under 13. Open
Open
abstract class AbstractAuthenticator extends OAuth2Authenticator implements AuthenticationEntryPointInterface
{
protected string $providerName = '';
protected OAuth2ClientInterface $client;- Read upRead up
- Exclude checks
CouplingBetweenObjects
Since: 1.1.0
A class with too many dependencies has negative impacts on several quality aspects of a class. This includes quality criteria like stability, maintainability and understandability
Example
class Foo {
/**
* @var \foo\bar\X
*/
private $x = null;
/**
* @var \foo\bar\Y
*/
private $y = null;
/**
* @var \foo\bar\Z
*/
private $z = null;
public function setFoo(\Foo $foo) {}
public function setBar(\Bar $bar) {}
public function setBaz(\Baz $baz) {}
/**
* @return \SplObjectStorage
* @throws \OutOfRangeException
* @throws \InvalidArgumentException
* @throws \ErrorException
*/
public function process(\Iterator $it) {}
// ...
}Source https://phpmd.org/rules/design.html#couplingbetweenobjects
Avoid unused parameters such as '$authException'. Open
Open
public function start(Request $request, ?AuthenticationException $authException = null): Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$request'. Open
Open
public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$request'. Open
Open
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$token'. Open
Open
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$request'. Open
Open
public function start(Request $request, ?AuthenticationException $authException = null): Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$request'. Open
Open
public function authenticate(Request $request): Passport- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Avoid unused parameters such as '$firewallName'. Open
Open
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
Missing class doc comment Open
Open
abstract class AbstractAuthenticator extends OAuth2Authenticator implements AuthenticationEntryPointInterface- Exclude checks
Add a single space around assignment operators Open
Open
declare(strict_types=1);- Exclude checks
Missing function doc comment Open
Open
public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response- Exclude checks
Missing function doc comment Open
Open
abstract public function supports(Request $request): ?bool;- Exclude checks
Missing function doc comment Open
Open
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response- Exclude checks
Missing function doc comment Open
Open
public function start(Request $request, ?AuthenticationException $authException = null): Response- Exclude checks
Missing function doc comment Open
Open
public function __construct(- Exclude checks
Missing function doc comment Open
Open
public function authenticate(Request $request): Passport- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly AccessUrlHelper $urlHelper,- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly EntityManagerInterface $entityManager,- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly RouterInterface $router,- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly ClientRegistry $clientRegistry,- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly UserRepository $userRepository,- Exclude checks
Line indented incorrectly; expected 4 spaces, found 8 Open
Open
protected readonly AuthenticationConfigHelper $authenticationConfigHelper,- Exclude checks