Skip to content

Commit 76e8ac9

Browse files
committed
Fix CS
1 parent 0b16a9a commit 76e8ac9

File tree

14 files changed

+16
-39
lines changed

14 files changed

+16
-39
lines changed

src/Blacklist/ArrayProvider.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,6 @@ class ArrayProvider implements BlacklistProviderInterface
2222
{
2323
private $blacklist = [];
2424

25-
/**
26-
* @param array $blacklist
27-
*/
2825
public function __construct(array $blacklist)
2926
{
3027
$this->blacklist = $blacklist;

src/Blacklist/ChainProvider.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@ public function __construct(array $providers = [])
3838
/**
3939
* Adds a new blacklist provider.
4040
*
41-
* @param BlacklistProviderInterface $provider
42-
*
4341
* @throws \RuntimeException
4442
*
4543
* @return self

src/Blacklist/LazyChainProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ final class LazyChainProvider implements BlacklistProviderInterface
2626
/**
2727
* Constructor.
2828
*
29-
* @param ContainerInterface $container
30-
* @param string[] $providers
29+
* @param string[] $providers
3130
*/
3231
public function __construct(ContainerInterface $container, array $providers)
3332
{

src/Blacklist/PdoProvider.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ abstract protected function initDb();
147147
/**
148148
* @param object $db
149149
* @param string $query
150-
* @param array $args
151150
*
152151
* @return mixed
153152
*/
@@ -166,7 +165,6 @@ protected function fetch($db, $query, array $args = [])
166165
/**
167166
* @param object $db
168167
* @param string $query
169-
* @param array $args
170168
*
171169
* @throws \RuntimeException
172170
*/

src/Command/BlacklistCommand.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,7 @@ protected function initialize(InputInterface $input, OutputInterface $output)
4747
$this->blacklistProvider = $this->providers->get($input->getOption('provider'));
4848

4949
if (!$this->blacklistProvider instanceof UpdatableBlacklistProviderInterface) {
50-
throw new \RuntimeException(
51-
sprintf('Blacklist provider "%s" is not updatable.', $input->getOption('provider'))
52-
);
50+
throw new \RuntimeException(sprintf('Blacklist provider "%s" is not updatable.', $input->getOption('provider')));
5351
}
5452
}
5553
}

src/P0wnedPassword/Request/Client.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212
namespace Rollerworks\Component\PasswordStrength\P0wnedPassword\Request;
1313

1414
use GuzzleHttp\Psr7\Request;
15-
use Http\Client\Exception\HttpException;
1615
use Http\Client\Exception as HttpException2;
16+
use Http\Client\Exception\HttpException;
1717
use Http\Client\HttpClient;
1818
use Psr\Log\LoggerInterface;
1919

src/Validator/Constraints/BlacklistValidator.php

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,8 @@ class BlacklistValidator extends ConstraintValidator
3131
private $providersLoader;
3232

3333
/**
34-
* @param BlacklistProviderInterface $defaultProvider
35-
* @param ContainerInterface $providersLoader Service-container for loading
36-
* blacklist providers
34+
* @param ContainerInterface $providersLoader Service-container for loading
35+
* blacklist providers
3736
*/
3837
public function __construct(BlacklistProviderInterface $defaultProvider, ContainerInterface $providersLoader = null)
3938
{
@@ -59,12 +58,7 @@ public function validate($password, Constraint $constraint)
5958
$provider = $this->defaultProvider;
6059
} else {
6160
if (null === $this->providersLoader || !$this->providersLoader->has($constraint->provider)) {
62-
throw new RuntimeException(
63-
sprintf(
64-
'Unable to use blacklist provider "%s", eg. no blacklists were configured or this provider is not supported.',
65-
$constraint->provider
66-
)
67-
);
61+
throw new RuntimeException(sprintf('Unable to use blacklist provider "%s", eg. no blacklists were configured or this provider is not supported.', $constraint->provider));
6862
}
6963

7064
$provider = $this->providersLoader->get($constraint->provider);

src/Validator/Constraints/P0wnedPasswordValidator.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,6 @@ class P0wnedPasswordValidator extends ConstraintValidator
2323

2424
/**
2525
* P0wnedPasswordValidator constructor.
26-
*
27-
* @param Client $client
2826
*/
2927
public function __construct(Client $client)
3028
{

src/Validator/Constraints/PasswordStrengthValidator.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@
1414
use Symfony\Component\Translation\Loader\XliffFileLoader;
1515
use Symfony\Component\Translation\Translator;
1616
use Symfony\Component\Translation\TranslatorInterface as LegacyTranslatorInterface;
17-
use Symfony\Contracts\Translation\TranslatorInterface;
1817
use Symfony\Component\Validator\Constraint;
1918
use Symfony\Component\Validator\ConstraintValidator;
2019
use Symfony\Component\Validator\Exception\UnexpectedTypeException;
20+
use Symfony\Contracts\Translation\TranslatorInterface;
2121

2222
/**
2323
* Password strength Validation.
@@ -61,7 +61,7 @@ public function __construct($translator = null)
6161
if (null === $translator) {
6262
$translator = new Translator('en');
6363
$translator->addLoader('xlf', new XliffFileLoader());
64-
$translator->addResource('xlf', dirname(__DIR__, 2) . '/Resources/translations/validators.en.xlf', 'en', 'validators');
64+
$translator->addResource('xlf', dirname(__DIR__, 2).'/Resources/translations/validators.en.xlf', 'en', 'validators');
6565
}
6666

6767
$this->translator = $translator;
@@ -112,8 +112,8 @@ public function validate($password, Constraint $constraint)
112112
if ($passwordStrength < $constraint->minStrength) {
113113
$parameters = [
114114
'{{ length }}' => $constraint->minLength,
115-
'{{ min_strength }}' => $this->translator->trans(/** @Ignore */'rollerworks_password.strength_level.'.self::$levelToLabel[$constraint->minStrength], [], 'validators'),
116-
'{{ current_strength }}' => $this->translator->trans(/** @Ignore */'rollerworks_password.strength_level.'.self::$levelToLabel[$passwordStrength], [], 'validators'),
115+
'{{ min_strength }}' => $this->translator->trans(/* @Ignore */'rollerworks_password.strength_level.'.self::$levelToLabel[$constraint->minStrength], [], 'validators'),
116+
'{{ current_strength }}' => $this->translator->trans(/* @Ignore */'rollerworks_password.strength_level.'.self::$levelToLabel[$passwordStrength], [], 'validators'),
117117
'{{ strength_tips }}' => implode(', ', array_map([$this, 'translateTips'], $tips)),
118118
];
119119

@@ -128,7 +128,7 @@ public function validate($password, Constraint $constraint)
128128
*/
129129
public function translateTips($tip)
130130
{
131-
return $this->translator->trans(/** @Ignore */'rollerworks_password.tip.'.$tip, [], 'validators');
131+
return $this->translator->trans(/* @Ignore */'rollerworks_password.tip.'.$tip, [], 'validators');
132132
}
133133

134134
private function calculateStrength($password, &$tips)

tests/BlackListMockProviderTrait.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@ protected function createMockedProvider($blacklisted)
3333
}
3434

3535
/**
36-
* @param array $loaders
37-
*
3836
* @return ContainerInterface|object
3937
*/
4038
protected function createLoadersContainer(array $loaders)

0 commit comments

Comments
 (0)