Skip to content

Commit 6a6a32e

Browse files
committed
Fix coding standards
1 parent 888068a commit 6a6a32e

38 files changed

+11
-311
lines changed

.php-cs-fixer.dist.php

+4
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,10 @@
2323
'php_unit_construct' => true,
2424
'php_unit_strict' => true,
2525
'phpdoc_no_empty_return' => false,
26+
'no_superfluous_phpdoc_tags' => [
27+
'remove_inheritdoc' => true,
28+
'allow_mixed' => true,
29+
],
2630
])
2731
->setUsingCache(true)
2832
->setRiskyAllowed(true)

Command/ActivateUserCommand.php

-9
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ public function __construct(UserManipulator $userManipulator)
4141
$this->userManipulator = $userManipulator;
4242
}
4343

44-
/**
45-
* {@inheritdoc}
46-
*/
4744
protected function configure()
4845
{
4946
$this
@@ -61,9 +58,6 @@ protected function configure()
6158
);
6259
}
6360

64-
/**
65-
* {@inheritdoc}
66-
*/
6761
protected function execute(InputInterface $input, OutputInterface $output): int
6862
{
6963
$username = $input->getArgument('username');
@@ -75,9 +69,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
7569
return 0;
7670
}
7771

78-
/**
79-
* {@inheritdoc}
80-
*/
8172
protected function interact(InputInterface $input, OutputInterface $output)
8273
{
8374
if (!$input->getArgument('username')) {

Command/ChangePasswordCommand.php

-9
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,6 @@ public function __construct(UserManipulator $userManipulator)
3939
$this->userManipulator = $userManipulator;
4040
}
4141

42-
/**
43-
* {@inheritdoc}
44-
*/
4542
protected function configure()
4643
{
4744
$this
@@ -67,9 +64,6 @@ protected function configure()
6764
);
6865
}
6966

70-
/**
71-
* {@inheritdoc}
72-
*/
7367
protected function execute(InputInterface $input, OutputInterface $output): int
7468
{
7569
$username = $input->getArgument('username');
@@ -82,9 +76,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
8276
return 0;
8377
}
8478

85-
/**
86-
* {@inheritdoc}
87-
*/
8879
protected function interact(InputInterface $input, OutputInterface $output)
8980
{
9081
$questions = [];

Command/CreateUserCommand.php

-9
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,6 @@ public function __construct(UserManipulator $userManipulator)
4444
$this->userManipulator = $userManipulator;
4545
}
4646

47-
/**
48-
* {@inheritdoc}
49-
*/
5047
protected function configure()
5148
{
5249
$this
@@ -83,9 +80,6 @@ protected function configure()
8380
);
8481
}
8582

86-
/**
87-
* {@inheritdoc}
88-
*/
8983
protected function execute(InputInterface $input, OutputInterface $output): int
9084
{
9185
$username = $input->getArgument('username');
@@ -101,9 +95,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
10195
return 0;
10296
}
10397

104-
/**
105-
* {@inheritdoc}
106-
*/
10798
protected function interact(InputInterface $input, OutputInterface $output)
10899
{
109100
$questions = [];

Command/DeactivateUserCommand.php

-9
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ public function __construct(UserManipulator $userManipulator)
4141
$this->userManipulator = $userManipulator;
4242
}
4343

44-
/**
45-
* {@inheritdoc}
46-
*/
4744
protected function configure()
4845
{
4946
$this
@@ -61,9 +58,6 @@ protected function configure()
6158
);
6259
}
6360

64-
/**
65-
* {@inheritdoc}
66-
*/
6761
protected function execute(InputInterface $input, OutputInterface $output): int
6862
{
6963
$username = $input->getArgument('username');
@@ -75,9 +69,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
7569
return 0;
7670
}
7771

78-
/**
79-
* {@inheritdoc}
80-
*/
8172
protected function interact(InputInterface $input, OutputInterface $output)
8273
{
8374
if (!$input->getArgument('username')) {

Command/DemoteUserCommand.php

-6
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,6 @@ class DemoteUserCommand extends RoleCommand
2929
// BC with Symfony <5.3
3030
protected static $defaultName = 'fos:user:demote';
3131

32-
/**
33-
* {@inheritdoc}
34-
*/
3532
protected function configure()
3633
{
3734
parent::configure();
@@ -49,9 +46,6 @@ protected function configure()
4946
);
5047
}
5148

52-
/**
53-
* {@inheritdoc}
54-
*/
5549
protected function executeRoleCommand(UserManipulator $manipulator, OutputInterface $output, $username, $super, $role)
5650
{
5751
if ($super) {

Command/PromoteUserCommand.php

-6
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,6 @@ class PromoteUserCommand extends RoleCommand
3131
// BC with Symfony <5.3
3232
protected static $defaultName = 'fos:user:promote';
3333

34-
/**
35-
* {@inheritdoc}
36-
*/
3734
protected function configure()
3835
{
3936
parent::configure();
@@ -51,9 +48,6 @@ protected function configure()
5148
);
5249
}
5350

54-
/**
55-
* {@inheritdoc}
56-
*/
5751
protected function executeRoleCommand(UserManipulator $manipulator, OutputInterface $output, $username, $super, $role)
5852
{
5953
if ($super) {

Command/RoleCommand.php

-9
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,6 @@ public function __construct(UserManipulator $userManipulator)
3535
$this->userManipulator = $userManipulator;
3636
}
3737

38-
/**
39-
* {@inheritdoc}
40-
*/
4138
protected function configure()
4239
{
4340
$this
@@ -48,9 +45,6 @@ protected function configure()
4845
]);
4946
}
5047

51-
/**
52-
* {@inheritdoc}
53-
*/
5448
protected function execute(InputInterface $input, OutputInterface $output): int
5549
{
5650
$username = $input->getArgument('username');
@@ -80,9 +74,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
8074
*/
8175
abstract protected function executeRoleCommand(UserManipulator $manipulator, OutputInterface $output, $username, $super, $role);
8276

83-
/**
84-
* {@inheritdoc}
85-
*/
8677
protected function interact(InputInterface $input, OutputInterface $output)
8778
{
8879
$questions = [];

Controller/RegistrationController.php

-1
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@ private function getTargetUrlFromSession(SessionInterface $session): ?string
183183
return null;
184184
}
185185

186-
187186
$key = sprintf('_security.%s.target_path', $firewallName);
188187

189188
if ($session->has($key)) {

DependencyInjection/Compiler/CheckForSessionPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@
2626
*/
2727
class CheckForSessionPass implements CompilerPassInterface
2828
{
29-
/**
30-
* {@inheritdoc}
31-
*/
3229
public function process(ContainerBuilder $container)
3330
{
3431
if ($container->hasParameter('fos_user.session_needed') && !$container->has('session.storage.factory') && !$container->has('session')) {

DependencyInjection/Compiler/CheckForSwiftMailerPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@
2626
*/
2727
class CheckForSwiftMailerPass implements CompilerPassInterface
2828
{
29-
/**
30-
* {@inheritdoc}
31-
*/
3229
public function process(ContainerBuilder $container)
3330
{
3431
// if the mailer isn't needed, then no error needed

DependencyInjection/Compiler/ConfigurePasswordHasherPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,6 @@
2121
*/
2222
final class ConfigurePasswordHasherPass implements CompilerPassInterface
2323
{
24-
/**
25-
* {@inheritdoc}
26-
*/
2724
public function process(ContainerBuilder $container)
2825
{
2926
if ($container->has('security.password_hasher_factory')) {

DependencyInjection/Compiler/InjectRememberMeServicesPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@
2626
*/
2727
class InjectRememberMeServicesPass implements CompilerPassInterface
2828
{
29-
/**
30-
* {@inheritdoc}
31-
*/
3229
public function process(ContainerBuilder $container)
3330
{
3431
$firewallName = $container->getParameter('fos_user.firewall_name');

DependencyInjection/Compiler/InjectUserCheckerPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@
2626
*/
2727
class InjectUserCheckerPass implements CompilerPassInterface
2828
{
29-
/**
30-
* {@inheritdoc}
31-
*/
3229
public function process(ContainerBuilder $container)
3330
{
3431
$firewallName = $container->getParameter('fos_user.firewall_name');

DependencyInjection/Compiler/ValidationPass.php

-3
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@
2525
*/
2626
class ValidationPass implements CompilerPassInterface
2727
{
28-
/**
29-
* {@inheritdoc}
30-
*/
3128
public function process(ContainerBuilder $container)
3229
{
3330
if (!$container->hasParameter('fos_user.storage')) {

DependencyInjection/FOSUserExtension.php

-3
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,6 @@ class FOSUserExtension extends Extension
4949
private $mailerNeeded = false;
5050
private $sessionNeeded = false;
5151

52-
/**
53-
* {@inheritdoc}
54-
*/
5552
public function load(array $configs, ContainerBuilder $container)
5653
{
5754
$processor = new Processor();

Doctrine/CouchDB/UserListener.php

-3
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,6 @@ public function __construct(PasswordUpdaterInterface $passwordUpdater, Canonical
3434
$this->canonicalFieldsUpdater = $canonicalFieldsUpdater;
3535
}
3636

37-
/**
38-
* {@inheritdoc}
39-
*/
4037
public function getSubscribedEvents(): array
4138
{
4239
return [

Doctrine/UserListener.php

-3
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ public function __construct(PasswordUpdaterInterface $passwordUpdater, Canonical
4141
$this->canonicalFieldsUpdater = $canonicalFieldsUpdater;
4242
}
4343

44-
/**
45-
* {@inheritdoc}
46-
*/
4744
public function getSubscribedEvents(): array
4845
{
4946
return [

Doctrine/UserManager.php

-18
Original file line numberDiff line numberDiff line change
@@ -43,18 +43,12 @@ public function __construct(PasswordUpdaterInterface $passwordUpdater, Canonical
4343
$this->class = $class;
4444
}
4545

46-
/**
47-
* {@inheritdoc}
48-
*/
4946
public function deleteUser(UserInterface $user)
5047
{
5148
$this->objectManager->remove($user);
5249
$this->objectManager->flush();
5350
}
5451

55-
/**
56-
* {@inheritdoc}
57-
*/
5852
public function getClass()
5953
{
6054
if (false !== strpos($this->class, ':')) {
@@ -65,33 +59,21 @@ public function getClass()
6559
return $this->class;
6660
}
6761

68-
/**
69-
* {@inheritdoc}
70-
*/
7162
public function findUserBy(array $criteria)
7263
{
7364
return $this->getRepository()->findOneBy($criteria);
7465
}
7566

76-
/**
77-
* {@inheritdoc}
78-
*/
7967
public function findUsers()
8068
{
8169
return $this->getRepository()->findAll();
8270
}
8371

84-
/**
85-
* {@inheritdoc}
86-
*/
8772
public function reloadUser(UserInterface $user)
8873
{
8974
$this->objectManager->refresh($user);
9075
}
9176

92-
/**
93-
* {@inheritdoc}
94-
*/
9577
public function updateUser(UserInterface $user, $andFlush = true)
9678
{
9779
$this->updateCanonicalFields($user);

EventListener/AuthenticationListener.php

-3
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,6 @@ public function __construct(LoginManagerInterface $loginManager, $firewallName)
4848
$this->firewallName = $firewallName;
4949
}
5050

51-
/**
52-
* {@inheritdoc}
53-
*/
5451
public static function getSubscribedEvents(): array
5552
{
5653
return [

EventListener/FlashListener.php

-3
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,6 @@ public function __construct(RequestStack $requestStack, TranslatorInterface $tra
5454
$this->requestStack = $requestStack;
5555
}
5656

57-
/**
58-
* {@inheritdoc}
59-
*/
6057
public static function getSubscribedEvents(): array
6158
{
6259
return [

EventListener/ResettingListener.php

-3
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,6 @@ public function __construct(UrlGeneratorInterface $router, $tokenTtl)
4646
$this->tokenTtl = $tokenTtl;
4747
}
4848

49-
/**
50-
* {@inheritdoc}
51-
*/
5249
public static function getSubscribedEvents(): array
5350
{
5451
return [

Form/Factory/FormFactory.php

-3
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,6 @@ public function __construct(FormFactoryInterface $formFactory, $name, $type, arr
5050
$this->validationGroups = $validationGroups;
5151
}
5252

53-
/**
54-
* {@inheritdoc}
55-
*/
5653
public function createForm(array $options = [])
5754
{
5855
$options = array_merge(['validation_groups' => $this->validationGroups], $options);

0 commit comments

Comments
 (0)