Skip to content

Commit ffbd444

Browse files
committed
minor symfony#59095 chore: fix CS (keradus)
This PR was squashed before being merged into the 7.2 branch. Discussion ---------- chore: fix CS | Q | A | ------------- | --- | Branch? | 7.2 | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Issues | Fix CS <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead --> | License | MIT Commits ------- a35a5b6 chore: fix CS
2 parents f24ac9e + a35a5b6 commit ffbd444

File tree

11 files changed

+14
-8
lines changed

11 files changed

+14
-8
lines changed

src/Symfony/Bridge/Doctrine/Tests/Security/RememberMe/DoctrineTokenProviderPostgresTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010

1111
/**
1212
* @requires extension pdo_pgsql
13+
*
1314
* @group integration
1415
*/
1516
class DoctrineTokenProviderPostgresTest extends DoctrineTokenProviderTest

src/Symfony/Bundle/FrameworkBundle/Command/AboutCommand.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
8686
['Timezone', date_default_timezone_get().' (<comment>'.(new \DateTimeImmutable())->format(\DateTimeInterface::W3C).'</>)'],
8787
['OPcache', \extension_loaded('Zend OPcache') ? (filter_var(\ini_get('opcache.enable'), FILTER_VALIDATE_BOOLEAN) ? 'Enabled' : 'Not enabled') : 'Not installed'],
8888
['APCu', \extension_loaded('apcu') ? (filter_var(\ini_get('apc.enabled'), FILTER_VALIDATE_BOOLEAN) ? 'Enabled' : 'Not enabled') : 'Not installed'],
89-
['Xdebug', \extension_loaded('xdebug') ? ($xdebugMode && $xdebugMode !== 'off' ? 'Enabled (' . $xdebugMode . ')' : 'Not enabled') : 'Not installed'],
89+
['Xdebug', \extension_loaded('xdebug') ? ($xdebugMode && 'off' !== $xdebugMode ? 'Enabled (' . $xdebugMode . ')' : 'Not enabled') : 'Not installed'],
9090
];
9191

9292
$io->table([], $rows);

src/Symfony/Component/Cache/Traits/Relay/MoveTrait.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ public function lmove($srckey, $dstkey, $srcpos, $dstpos): mixed
3333
*/
3434
trait MoveTrait
3535
{
36-
public function blmove($srckey, $dstkey, $srcpos, $dstpos, $timeout): \Relay\Relay|false|null|string
36+
public function blmove($srckey, $dstkey, $srcpos, $dstpos, $timeout): \Relay\Relay|false|string|null
3737
{
3838
return $this->initializeLazyObject()->blmove(...\func_get_args());
3939
}
4040

41-
public function lmove($srckey, $dstkey, $srcpos, $dstpos): \Relay\Relay|false|null|string
41+
public function lmove($srckey, $dstkey, $srcpos, $dstpos): \Relay\Relay|false|string|null
4242
{
4343
return $this->initializeLazyObject()->lmove(...\func_get_args());
4444
}

src/Symfony/Component/HttpClient/Response/CurlResponse.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -316,7 +316,7 @@ private static function perform(ClientState $multi, ?array &$responses = null):
316316
}
317317

318318
$multi->handlesActivity[$id][] = null;
319-
$multi->handlesActivity[$id][] = \in_array($result, [\CURLE_OK, \CURLE_TOO_MANY_REDIRECTS], true) || '_0' === $waitFor || curl_getinfo($ch, \CURLINFO_SIZE_DOWNLOAD) === curl_getinfo($ch, \CURLINFO_CONTENT_LENGTH_DOWNLOAD) || (curl_error($ch) === 'OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 0' && -1.0 === curl_getinfo($ch, \CURLINFO_CONTENT_LENGTH_DOWNLOAD) && \in_array('close', array_map('strtolower', $responses[$id]->headers['connection']), true)) ? null : new TransportException(ucfirst(curl_error($ch) ?: curl_strerror($result)).\sprintf(' for "%s".', curl_getinfo($ch, \CURLINFO_EFFECTIVE_URL)));
319+
$multi->handlesActivity[$id][] = \in_array($result, [\CURLE_OK, \CURLE_TOO_MANY_REDIRECTS], true) || '_0' === $waitFor || curl_getinfo($ch, \CURLINFO_SIZE_DOWNLOAD) === curl_getinfo($ch, \CURLINFO_CONTENT_LENGTH_DOWNLOAD) || ('OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 0' === curl_error($ch) && -1.0 === curl_getinfo($ch, \CURLINFO_CONTENT_LENGTH_DOWNLOAD) && \in_array('close', array_map('strtolower', $responses[$id]->headers['connection']), true)) ? null : new TransportException(ucfirst(curl_error($ch) ?: curl_strerror($result)).\sprintf(' for "%s".', curl_getinfo($ch, \CURLINFO_EFFECTIVE_URL)));
320320
}
321321
} finally {
322322
$multi->performing = false;

src/Symfony/Component/HttpClient/Tests/CurlHttpClientTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717

1818
/**
1919
* @requires extension curl
20+
*
2021
* @group dns-sensitive
2122
*/
2223
class CurlHttpClientTest extends HttpClientTestCase

src/Symfony/Component/HttpClient/Tests/NoPrivateNetworkHttpClientTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ public static function getExcludeHostData(): iterable
6868

6969
/**
7070
* @dataProvider getExcludeIpData
71+
*
7172
* @group dns-sensitive
7273
*/
7374
public function testExcludeByIp(string $ipAddr, $subnets, bool $mustThrow)
@@ -105,6 +106,7 @@ public function testExcludeByIp(string $ipAddr, $subnets, bool $mustThrow)
105106

106107
/**
107108
* @dataProvider getExcludeHostData
109+
*
108110
* @group dns-sensitive
109111
*/
110112
public function testExcludeByHost(string $ipAddr, $subnets, bool $mustThrow)

src/Symfony/Component/HttpKernel/DataCollector/ConfigDataCollector.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public function collect(Request $request, Response $response, ?\Throwable $excep
5757
'php_intl_locale' => class_exists(\Locale::class, false) && \Locale::getDefault() ? \Locale::getDefault() : 'n/a',
5858
'php_timezone' => date_default_timezone_get(),
5959
'xdebug_enabled' => \extension_loaded('xdebug'),
60-
'xdebug_status' => \extension_loaded('xdebug') ? ($xdebugMode && $xdebugMode !== 'off' ? 'Enabled (' . $xdebugMode . ')' : 'Not enabled') : 'Not installed',
60+
'xdebug_status' => \extension_loaded('xdebug') ? ($xdebugMode && 'off' !== $xdebugMode ? 'Enabled (' . $xdebugMode . ')' : 'Not enabled') : 'Not installed',
6161
'apcu_enabled' => \extension_loaded('apcu') && filter_var(\ini_get('apc.enabled'), \FILTER_VALIDATE_BOOL),
6262
'apcu_status' => \extension_loaded('apcu') ? (filter_var(\ini_get('apc.enabled'), FILTER_VALIDATE_BOOLEAN) ? 'Enabled' : 'Not enabled') : 'Not installed',
6363
'zend_opcache_enabled' => \extension_loaded('Zend OPcache') && filter_var(\ini_get('opcache.enable'), \FILTER_VALIDATE_BOOL),

src/Symfony/Component/Messenger/Attribute/AsMessage.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public function __construct(
2323
/**
2424
* Name of the transports to which the message should be routed.
2525
*/
26-
public null|string|array $transport = null,
26+
public string|array|null $transport = null,
2727
) {
2828
}
2929
}

src/Symfony/Component/PropertyInfo/Tests/Extractor/ReflectionExtractorTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -718,6 +718,7 @@ public function testVirtualProperties()
718718

719719
/**
720720
* @dataProvider provideAsymmetricVisibilityMutator
721+
*
721722
* @requires PHP 8.4
722723
*/
723724
public function testAsymmetricVisibilityMutator(string $property, string $readVisibility, string $writeVisibility)
@@ -743,6 +744,7 @@ public static function provideAsymmetricVisibilityMutator(): iterable
743744

744745
/**
745746
* @dataProvider provideVirtualPropertiesMutator
747+
*
746748
* @requires PHP 8.4
747749
*/
748750
public function testVirtualPropertiesMutator(string $property, string $readVisibility, string $writeVisibility)

src/Symfony/Component/TypeInfo/Type/UnionType.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public function __construct(Type ...$types)
4545
}
4646

4747
if ($type instanceof BuiltinType) {
48-
if ($type->getTypeIdentifier() === TypeIdentifier::NULL && !is_a(static::class, NullableType::class, allow_string: true)) {
48+
if (TypeIdentifier::NULL === $type->getTypeIdentifier() && !is_a(static::class, NullableType::class, allow_string: true)) {
4949
throw new InvalidArgumentException(\sprintf('Cannot create union with "null", please use "%s" instead.', NullableType::class));
5050
}
5151

0 commit comments

Comments
 (0)