Merge branch '7.2' into 7.3

* 7.2:
  Fix php.net links
  fix version number in deprecation
  [DoctrineBridge] Restore compatibility with Doctrine ODM by validating $class object type
  chore: Increase minimum version of type-info component
  [FrameworkBundle] Add functional tests for the `ContainerLintCommand` command
  Fix precision loss when rounding large integers in `NumberToLocalizedStringTransformer`

Author: nicolas-grekas

src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php

@@ -11,7 +11,6 @@
 
 namespace Symfony\Bridge\Doctrine\Validator\Constraints;
 
-use Doctrine\ORM\Mapping\ClassMetadata as OrmClassMetadata;
 use Doctrine\ORM\Mapping\MappingException as ORMMappingException;
 use Doctrine\Persistence\ManagerRegistry;
 use Doctrine\Persistence\Mapping\ClassMetadata;
@@ -294,7 +293,7 @@ private function getFieldValues(mixed $object, ClassMetadata $class, array $fiel
                 throw new ConstraintDefinitionException(\sprintf('The field "%s" is not a property of class "%s".', $fieldName, $objectClass));
             }
 
-            if ($isValueEntity && $object instanceof ($class->getName()) && property_exists(OrmClassMetadata::class, 'propertyAccessors')) {
+            if ($isValueEntity && $object instanceof ($class->getName()) && property_exists($class, 'propertyAccessors')) {
                 $fieldValues[$entityFieldName] = $class->propertyAccessors[$fieldName]->getValue($object);
             } elseif ($isValueEntity && $object instanceof ($class->getName())) {
                 $fieldValues[$entityFieldName] = $class->reflFields[$fieldName]->getValue($object);

src/Symfony/Bridge/Doctrine/composer.json

@@ -39,7 +39,7 @@
         "symfony/security-core": "^6.4|^7.0",
         "symfony/stopwatch": "^6.4|^7.0",
         "symfony/translation": "^6.4|^7.0",
-        "symfony/type-info": "^7.1",
+        "symfony/type-info": "^7.1.8",
         "symfony/uid": "^6.4|^7.0",
         "symfony/validator": "^6.4|^7.0",
         "symfony/var-dumper": "^6.4|^7.0",

src/Symfony/Bridge/Twig/Node/FormThemeNode.php

@@ -28,7 +28,7 @@ final class FormThemeNode extends Node
     public function __construct(Node $form, Node $resources, int $lineno, $only = false)
     {
         if (null === $only || \is_string($only)) {
-            trigger_deprecation('symfony/twig-bridge', '3.12', 'Passing a tag to %s() is deprecated.', __METHOD__);
+            trigger_deprecation('symfony/twig-bridge', '7.2', 'Passing a tag to %s() is deprecated.', __METHOD__);
             $only = \func_num_args() > 4 ? func_get_arg(4) : true;
         } elseif (!\is_bool($only)) {
             throw new \TypeError(\sprintf('Argument 4 passed to "%s()" must be a boolean, "%s" given.', __METHOD__, get_debug_type($only)));

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/ContainerLintCommandTest.php

@@ -0,0 +1,55 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Bundle\FrameworkBundle\Tests\Functional;
+
+use Symfony\Bundle\FrameworkBundle\Console\Application;
+use Symfony\Component\Console\Tester\CommandTester;
+
+/**
+ * @group functional
+ */
+class ContainerLintCommandTest extends AbstractWebTestCase
+{
+    private Application $application;
+
+    /**
+     * @dataProvider containerLintProvider
+     */
+    public function testLintContainer(string $configFile, string $expectedOutput)
+    {
+        $kernel = static::createKernel([
+            'test_case' => 'ContainerDebug',
+            'root_config' => $configFile,
+            'debug' => true,
+        ]);
+        $this->application = new Application($kernel);
+
+        $tester = $this->createCommandTester();
+        $exitCode = $tester->execute([]);
+
+        $this->assertSame(0, $exitCode);
+        $this->assertStringContainsString($expectedOutput, $tester->getDisplay());
+    }
+
+    public static function containerLintProvider(): array
+    {
+        return [
+            'default container' => ['config.yml', 'The container was linted successfully'],
+            'missing dump file' => ['no_dump.yml', 'The container was linted successfully'],
+        ];
+    }
+
+    private function createCommandTester(): CommandTester
+    {
+        return new CommandTester($this->application->get('lint:container'));
+    }
+}

src/Symfony/Bundle/FrameworkBundle/composer.json

@@ -65,7 +65,7 @@
         "symfony/string": "^6.4|^7.0",
         "symfony/translation": "^7.3",
         "symfony/twig-bundle": "^6.4|^7.0",
-        "symfony/type-info": "^7.1",
+        "symfony/type-info": "^7.1.8",
         "symfony/validator": "^6.4|^7.0",
         "symfony/workflow": "^7.3",
         "symfony/yaml": "^6.4|^7.0",

src/Symfony/Component/ErrorHandler/Tests/ErrorRenderer/HtmlErrorRendererTest.php

@@ -100,7 +100,7 @@ public static function provideFileLinkFormats(): iterable
 
     public function testRendersStackWithoutBinaryStrings()
     {
-        // make sure method arguments are available in stack traces (see https://www.php.net/manual/en/ini.core.php)
+        // make sure method arguments are available in stack traces (see https://php.net/ini.core)
         ini_set('zend.exception_ignore_args', false);
 
         $binaryData = file_get_contents(__DIR__.'/../Fixtures/pixel.png');

src/Symfony/Component/Filesystem/Filesystem.php

@@ -225,7 +225,7 @@ public function chmod(string|iterable $files, int $mode, int $umask = 0000, bool
      *
      * This method always throws on Windows, as the underlying PHP function is not supported.
      *
-     * @see https://www.php.net/chown
+     * @see https://php.net/chown
      *
      * @param string|int $user      A user name or number
      * @param bool       $recursive Whether change the owner recursively or not
@@ -255,7 +255,7 @@ public function chown(string|iterable $files, string|int $user, bool $recursive
      *
      * This method always throws on Windows, as the underlying PHP function is not supported.
      *
-     * @see https://www.php.net/chgrp
+     * @see https://php.net/chgrp
      *
      * @param string|int $group     A group name or number
      * @param bool       $recursive Whether change the group recursively or not

src/Symfony/Component/Form/Extension/Core/DataTransformer/NumberToLocalizedStringTransformer.php

@@ -185,6 +185,10 @@ protected function castParsedValue(int|float $value): int|float
      */
     private function round(int|float $number): int|float
     {
+        if (\is_int($number)) {
+            return $number;
+        }
+
         if (null !== $this->scale) {
             // shift number to maintain the correct scale during rounding
             $roundingCoef = 10 ** $this->scale;

src/Symfony/Component/Form/Tests/Extension/Core/DataTransformer/NumberToLocalizedStringTransformerTest.php

@@ -726,4 +726,33 @@ public static function eNotationProvider(): array
             [1232.0, '1.232e3'],
         ];
     }
+
+    public function testReverseTransformDoesNotCauseIntegerPrecisionLoss()
+    {
+        $transformer = new NumberToLocalizedStringTransformer();
+
+        // Test a large integer that causes actual precision loss when cast to float
+        $largeInt = \PHP_INT_MAX - 1; // This value loses precision when cast to float
+        $result = $transformer->reverseTransform((string) $largeInt);
+
+        $this->assertSame($largeInt, $result);
+        $this->assertIsInt($result);
+    }
+
+    public function testRoundMethodKeepsIntegersAsIntegers()
+    {
+        $transformer = new NumberToLocalizedStringTransformer(2); // scale=2 triggers rounding
+        
+        // Use reflection to test the private round() method directly
+        $reflection = new \ReflectionClass($transformer);
+        $roundMethod = $reflection->getMethod('round');
+        $roundMethod->setAccessible(true);
+        
+        $int = \PHP_INT_MAX - 1;
+        $result = $roundMethod->invoke($transformer, $int);
+        
+        // With the fix, integers should stay as integers, not be converted to floats
+        $this->assertSame($int, $result);
+        $this->assertIsInt($result);
+    }
 }

src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php

@@ -123,7 +123,7 @@ public function start(): bool
          * ---------- Part 1
          *
          * The part `[a-zA-Z0-9,-]` is related to the PHP ini directive `session.sid_bits_per_character` defined as 6.
-         * See https://www.php.net/manual/en/session.configuration.php#ini.session.sid-bits-per-character.
+         * See https://php.net/session.configuration#ini.session.sid-bits-per-character
          * Allowed values are integers such as:
          * - 4 for range `a-f0-9`
          * - 5 for range `a-v0-9` (@deprecated since Symfony 7.2, it will default to 4 and the option will be ignored in Symfony 8.0)
@@ -132,7 +132,7 @@ public function start(): bool
          * ---------- Part 2
          *
          * The part `{22,250}` is related to the PHP ini directive `session.sid_length`.
-         * See https://www.php.net/manual/en/session.configuration.php#ini.session.sid-length.
+         * See https://php.net/session.configuration#ini.session.sid-length
          * Allowed values are integers between 22 and 256, but we use 250 for the max.
          *
          * Where does the 250 come from?