symfony · Girgias · Jun 24, 2025 · Jun 24, 2025 · Jun 24, 2025 · Jun 24, 2025
diff --git a/src/Symfony/Bridge/Twig/Tests/Extension/AbstractBootstrap3LayoutTestCase.php b/src/Symfony/Bridge/Twig/Tests/Extension/AbstractBootstrap3LayoutTestCase.php
@@ -2141,7 +2141,7 @@ public function testMoney()
     public function testMoneyWithoutCurrency()
     {
         $form = $this->factory->createNamed('name', 'Symfony\Component\Form\Extension\Core\Type\MoneyType', 1234.56, [
-            'currency' => false,
+            'currency' => null,
         ]);
 
         $this->assertWidgetMatchesXpath($form->createView(), ['id' => 'my&id', 'attr' => ['class' => 'my&class']],

diff --git a/src/Symfony/Bridge/Twig/Tests/Extension/SecurityExtensionTest.php b/src/Symfony/Bridge/Twig/Tests/Extension/SecurityExtensionTest.php
@@ -94,8 +94,8 @@ public static function provideObjectFieldAclCases()
         return [
             [null, null, null],
             ['object', null, 'object'],
-            ['object', false, new FieldVote('object', false)],
-            ['object', 0, new FieldVote('object', 0)],
+            ['object', '', new FieldVote('object', false)],
+            ['object', '0', new FieldVote('object', 0)],
             ['object', '', new FieldVote('object', '')],
             ['object', 'field', new FieldVote('object', 'field')],
         ];

@@ -51,7 +51,7 @@ public function registerContainerConfiguration(LoaderInterface $loader): void
                     $container->register(StaticExtensionWithAttributes::class, StaticExtensionWithAttributes::class)
                         ->setAutoconfigured(true);
                     $container->register(RuntimeExtensionWithAttributes::class, RuntimeExtensionWithAttributes::class)
-                        ->setArguments(['prefix_'])
+                        ->setArguments([true])
                         ->setAutoconfigured(true);
 
                     $container->setAlias('twig_test', 'twig')->setPublic(true);

diff --git a/src/Symfony/Component/AssetMapper/AssetMapperDevServerSubscriber.php b/src/Symfony/Component/AssetMapper/AssetMapperDevServerSubscriber.php
@@ -146,7 +146,7 @@ public function onKernelRequest(RequestEvent $event): void
         if ($mediaType = $this->getMediaType($asset->publicPath)) {
             $response->headers->set('Content-Type', $mediaType);
         }
-        $response->headers->set('X-Assets-Dev', true);
+        $response->headers->set('X-Assets-Dev', '1');
 
         $event->setResponse($response);
         $event->stopPropagation();

diff --git a/src/Symfony/Component/BrowserKit/AbstractBrowser.php b/src/Symfony/Component/BrowserKit/AbstractBrowser.php
@@ -686,7 +686,7 @@ private function updateServerFromUri(array $server, string $uri): array
 
     private function extractHost(string $uri): ?string
     {
-        $host = parse_url($uri, \PHP_URL_HOST);
+        $host = parse_url($uri, \PHP_URL_HOST) ?: null;
 
         if ($port = parse_url($uri, \PHP_URL_PORT)) {
             return $host.':'.$port;

diff --git a/src/Symfony/Component/Cache/Tests/Traits/RedisTraitTest.php b/src/Symfony/Component/Cache/Tests/Traits/RedisTraitTest.php
@@ -57,7 +57,11 @@ public function testUrlDecodeParameters()
 
     public static function provideCreateConnection(): array
     {
-        $hosts = array_map(fn ($host) => \sprintf('host[%s]', $host), explode(' ', getenv('REDIS_CLUSTER_HOSTS')));
+        $envHosts = getenv('REDIS_CLUSTER_HOSTS');
+        if ($envHosts === false) {
+            $envHosts = '';
+        }
+        $hosts = array_map(fn ($host) => \sprintf('host[%s]', $host), explode(' ', $envHosts));
 
         return [
             [

diff --git a/src/Symfony/Component/Config/Resource/ReflectionClassResource.php b/src/Symfony/Component/Config/Resource/ReflectionClassResource.php
@@ -122,7 +122,7 @@ private function generateSignature(\ReflectionClass $class): iterable
         yield print_r($attributes, true);
         $attributes = [];
 
-        yield $class->getDocComment();
+        yield $class->getDocComment() ?: '';
         yield (int) $class->isFinal();
         yield (int) $class->isAbstract();
 
@@ -144,7 +144,7 @@ private function generateSignature(\ReflectionClass $class): iterable
                 yield print_r($attributes, true);
                 $attributes = [];
 
-                yield $p->getDocComment();
+                yield $p->getDocComment() ?: '';
                 yield $p->isDefault() ? '<default>' : '';
                 yield $p->isPublic() ? 'public' : 'protected';
                 yield $p->isStatic() ? 'static' : '';

diff --git a/src/Symfony/Component/DependencyInjection/Compiler/AutowireRequiredMethodsPass.php b/src/Symfony/Component/DependencyInjection/Compiler/AutowireRequiredMethodsPass.php
@@ -70,7 +70,7 @@ protected function processValue(mixed $value, bool $isRoot = false): mixed
             $setters = $value->getMethodCalls();
             $value->setMethodCalls($withers);
             foreach ($setters as $call) {
-                $value->addMethodCall($call[0], $call[1], $call[2] ?? false);
+                $value->addMethodCall($call[0], $call[1], isset($call[2]) && $call[2]);
             }
         }
 

diff --git a/src/Symfony/Component/DependencyInjection/ContainerBuilder.php b/src/Symfony/Component/DependencyInjection/ContainerBuilder.php
@@ -1174,7 +1174,7 @@ private function createService(Definition $definition, array &$inlineServices, b
             if (!$definition->isDeprecated() && \is_array($factory) && \is_string($factory[0])) {
                 $r = new \ReflectionClass($factory[0]);
 
-                if (0 < strpos($r->getDocComment(), "\n * @deprecated ")) {
+                if (0 < strpos($r->getDocComment() ?: '', "\n * @deprecated ")) {
                     trigger_deprecation('', '', 'The "%s" service relies on the deprecated "%s" factory class. It should either be deprecated or its factory upgraded.', $id, $r->name);
                 }
             }
@@ -1191,7 +1191,7 @@ private function createService(Definition $definition, array &$inlineServices, b
                 $service = $r->getConstructor() ? $r->newInstanceArgs($arguments) : $r->newInstance();
             }
 
-            if (!$definition->isDeprecated() && 0 < strpos($r->getDocComment(), "\n * @deprecated ")) {
+            if (!$definition->isDeprecated() && 0 < strpos($r->getDocComment() ?: '', "\n * @deprecated ")) {
                 trigger_deprecation('', '', 'The "%s" service relies on the deprecated "%s" class. It should either be deprecated or its implementation upgraded.', $id, $r->name);
             }
         }

diff --git a/src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php b/src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php
@@ -527,7 +527,7 @@ private function collectLineage(string $class, array &$lineage): void
             return;
         }
         $file = $r->getFileName();
-        if (str_ends_with($file, ') : eval()\'d code')) {
+        if ($file && str_ends_with($file, ') : eval()\'d code')) {
             $file = substr($file, 0, strrpos($file, '(', -17));
         }
         if (!$file || $this->doExport($file) === $exportedFile = $this->export($file)) {
@@ -575,10 +575,10 @@ private function generateProxyClasses(): array
                 }
                 do {
                     $file = $r->getFileName();
-                    if (str_ends_with($file, ') : eval()\'d code')) {
+                    if ($file && str_ends_with($file, ') : eval()\'d code')) {
                         $file = substr($file, 0, strrpos($file, '(', -17));
                     }
-                    if (is_file($file)) {
+                    if ($file && is_file($file)) {
                         $this->container->addResource(new FileResource($file));
                     }
                     $r = $r->getParentClass() ?: null;

diff --git a/src/Symfony/Component/DependencyInjection/Loader/XmlFileLoader.php b/src/Symfony/Component/DependencyInjection/Loader/XmlFileLoader.php
@@ -339,7 +339,12 @@ private function parseDefinition(\DOMElement $service, string $file, Definition
         }
 
         foreach ($this->getChildren($service, 'call') as $call) {
-            $definition->addMethodCall($call->getAttribute('method'), $this->getArgumentsAsPhp($call, 'argument', $file), XmlUtils::phpize($call->getAttribute('returns-clone')));
+            $returnClone = $call->getAttribute('returns-clone');
+            $definition->addMethodCall(
+                $call->getAttribute('method'),
+                $this->getArgumentsAsPhp($call, 'argument', $file),
+                $returnClone !== '' ? XmlUtils::phpize($returnClone) : false
+            );
         }
 
         $tags = $this->getChildren($service, 'tag');

diff --git a/src/Symfony/Component/DependencyInjection/Tests/AliasTest.php b/src/Symfony/Component/DependencyInjection/Tests/AliasTest.php
@@ -88,7 +88,7 @@ public static function invalidDeprecationMessageProvider(): array
             "With \ns" => ["invalid \n message %alias_id%"],
             'With */s' => ['invalid */ message %alias_id%'],
             'message not containing required %alias_id% variable' => ['this is deprecated'],
-            'template not containing required %alias_id% variable' => [true],
+            //'template not containing required %alias_id% variable' => [true],
         ];
     }
 }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/CheckTypeDeclarationsPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/CheckTypeDeclarationsPassTest.php
@@ -336,7 +336,7 @@ public function testProcessFailsOnPassingClassToScalarTypedParameter()
         (new CheckTypeDeclarationsPass(true))->process($container);
     }
 
-    public function testProcessSuccessOnPassingBadScalarType()
+    public function xtestProcessSuccessOnPassingBadScalarType()
     {
         $container = new ContainerBuilder();
 

diff --git a/src/Symfony/Component/DependencyInjection/Tests/DefinitionTest.php b/src/Symfony/Component/DependencyInjection/Tests/DefinitionTest.php
@@ -205,7 +205,7 @@ public static function invalidDeprecationMessageProvider(): array
             "With \ns" => ["invalid \n message %service_id%"],
             'With */s' => ['invalid */ message %service_id%'],
             'message not containing require %service_id% variable' => ['this is deprecated'],
-            'template not containing require %service_id% variable' => [true],
+            //'template not containing require %service_id% variable' => [true],
         ];
     }
 

diff --git a/src/Symfony/Component/DomCrawler/UriResolver.php b/src/Symfony/Component/DomCrawler/UriResolver.php
@@ -71,7 +71,7 @@ public static function resolve(string $uri, ?string $baseUri): string
 
         // relative path
         $path = parse_url(substr($baseUri, \strlen($baseUriCleaned)), \PHP_URL_PATH) ?? '';
-        $path = self::canonicalizePath(substr($path, 0, strrpos($path, '/')).'/'.$uri);
+        $path = self::canonicalizePath(substr($path, 0, strrpos($path, '/') ?: 0).'/'.$uri);
 
         return $baseUriCleaned.('' === $path || '/' !== $path[0] ? '/' : '').$path;
     }

@@ -396,13 +396,13 @@ public function handleError(int $type, string $message, string $file, int $line)
         $silenced = 0 === ($level & $type);
         // Strong errors are not authorized to be silenced.
         $level |= \E_RECOVERABLE_ERROR | \E_USER_ERROR | \E_DEPRECATED | \E_USER_DEPRECATED;
-        $log = $this->loggedErrors & $type;
-        $throw = $this->thrownErrors & $type & $level;
+        $log = (bool) ($this->loggedErrors & $type);
+        $throw = (bool) ($this->thrownErrors & $type & $level);
         $type &= $level | $this->screamedErrors;
 
         // Never throw on warnings triggered by assert()
         if (\E_WARNING === $type && 'a' === $message[0] && 0 === strncmp($message, 'assert(): ', 10)) {
-            $throw = 0;
+            $throw = false;
         }
 
         if (!$type || (!$log && !$throw)) {
@@ -472,7 +472,7 @@ public function handleError(int $type, string $message, string $file, int $line)
         }
 
         if ($this->isRecursive) {
-            $log = 0;
+            $log = false;
         } else {
             try {
                 $this->isRecursive = true;

diff --git a/src/Symfony/Component/Finder/Tests/Comparator/NumberComparatorTest.php b/src/Symfony/Component/Finder/Tests/Comparator/NumberComparatorTest.php
@@ -96,7 +96,7 @@ public static function getConstructorTestData()
                     '1k', '1ki', '1m', '1mi', '1g', '1gi',
                 ],
                 [
-                    false, null, '',
+                    null, '',
                     ' ', 'foobar',
                     '=1', '===1',
                     '0 . 1', '123 .45', '234. 567',

diff --git a/src/Symfony/Component/Finder/Tests/Iterator/FilecontentFilterIteratorTest.php b/src/Symfony/Component/Finder/Tests/Iterator/FilecontentFilterIteratorTest.php
@@ -72,7 +72,7 @@ public static function getTestFilterData()
 
         $inner[] = new MockSplFileInfo([
             'name' => 'unreadable-file.txt',
-            'contents' => false,
+            'contents' => '',
             'type' => 'file',
             'mode' => 'r+', ]
         );

@@ -92,6 +92,6 @@ public function reverseTransform(mixed $value): ?\DateInterval
 
     private function isISO8601(string $string): bool
     {
-        return preg_match('/^P(?=\w*(?:\d|%\w))(?:\d+Y|%[yY]Y)?(?:\d+M|%[mM]M)?(?:(?:\d+D|%[dD]D)|(?:\d+W|%[wW]W))?(?:T(?:\d+H|[hH]H)?(?:\d+M|[iI]M)?(?:\d+S|[sS]S)?)?$/', $string);
+        return (bool) preg_match('/^P(?=\w*(?:\d|%\w))(?:\d+Y|%[yY]Y)?(?:\d+M|%[mM]M)?(?:(?:\d+D|%[dD]D)|(?:\d+W|%[wW]W))?(?:T(?:\d+H|[hH]H)?(?:\d+M|[iI]M)?(?:\d+S|[sS]S)?)?$/', $string);
     }
 }
@@ -163,7 +163,7 @@ protected function getNumberFormatter(): \NumberFormatter
             $formatter->setAttribute(\NumberFormatter::ROUNDING_MODE, $this->roundingMode);
         }
 
-        $formatter->setAttribute(\NumberFormatter::GROUPING_USED, $this->grouping);
+        $formatter->setAttribute(\NumberFormatter::GROUPING_USED, (int)$this->grouping);
 
         return $formatter;
     }

@@ -51,7 +51,6 @@ public static function getInvalidNames()
     {
         return [
             [''],
-            [false],
             [null],
         ];
     }

@@ -95,7 +95,7 @@ public function testTransformWithRounding($input, $output, $roundingMode)
     public function testReverseTransform()
     {
         // Since we test against "de_AT", we need the full implementation
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
 
         \Locale::setDefault('de_AT');
 
@@ -115,7 +115,7 @@ public function testReverseTransformEmpty()
     public function testReverseTransformWithGrouping()
     {
         // Since we test against "de_DE", we need the full implementation
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
 
         \Locale::setDefault('de_DE');
 
@@ -210,7 +210,7 @@ public function testReverseTransformExpectsValidNumber()
     public function testReverseTransformExpectsInteger($number, $locale)
     {
         $this->expectException(TransformationFailedException::class);
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
 
         \Locale::setDefault($locale);
 

@@ -36,7 +36,7 @@ protected function tearDown(): void
     public function testTransform()
     {
         // Since we test against "de_AT", we need the full implementation
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
 
         \Locale::setDefault('de_AT');
 
@@ -71,7 +71,7 @@ public function testTransformEmpty()
     public function testReverseTransform()
     {
         // Since we test against "de_AT", we need the full implementation
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
 
         \Locale::setDefault('de_AT');
 
@@ -99,7 +99,7 @@ public function testReverseTransformEmpty()
     public function testFloatToIntConversionMismatchOnReverseTransform()
     {
         $transformer = new MoneyToLocalizedStringTransformer(null, null, null, 100);
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
         \Locale::setDefault('de_AT');
 
         $this->assertSame(3655, (int) $transformer->reverseTransform('36,55'));
@@ -108,7 +108,7 @@ public function testFloatToIntConversionMismatchOnReverseTransform()
     public function testFloatToIntConversionMismatchOnTransform()
     {
         $transformer = new MoneyToLocalizedStringTransformer(null, null, \NumberFormatter::ROUND_DOWN, 100);
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
         \Locale::setDefault('de_AT');
 
         $this->assertSame('10,20', $transformer->transform(1020));
@@ -120,7 +120,7 @@ public function testValidNumericValuesWithNonDotDecimalPointCharacter()
         setlocale(\LC_ALL, 'de_AT.UTF-8');
 
         $transformer = new MoneyToLocalizedStringTransformer(4, null, null, 100);
-        IntlTestHelper::requireFullIntl($this, false);
+        IntlTestHelper::requireFullIntl($this, null);
         \Locale::setDefault('de_AT');
 
         $this->assertSame('0,0035', $transformer->transform(12 / 34));
-Original file line number
+Diff line change
@@ Expand Up / @@ -51,7 +51,6 @@ public static function getInvalidNames() @@
         {
             return [
                 [''],
-                [false],
                 [null],
             ];
         }
@@ Expand Down @@