Merge branch '4.4' into 5.4

* 4.4:
  [Cache] Fix connecting to Redis via a socket file
  [DependencyInjection][FrameworkBundle] Fix using PHP 8.1 enum as parameters
  [PropertyAccessor] Add missing TypeError catch
  [FrameworkBundle] Fix log channel of TagAwareAdapter
  [HttpClient] Fix Content-Length header when possible
  [DependencyInjection] Don't dump polyfilled classes in preload script

Author: nicolas-grekas

Console/Descriptor/Descriptor.php

@@ -173,6 +173,20 @@ protected function formatValue($value): string
      */
     protected function formatParameter($value): string
     {
+        if ($value instanceof \UnitEnum) {
+            return var_export($value, true);
+        }
+
+        // Recursively search for enum values, so we can replace it
+        // before json_encode (which will not display anything for \UnitEnum otherwise)
+        if (\is_array($value)) {
+            array_walk_recursive($value, static function (&$value) {
+                if ($value instanceof \UnitEnum) {
+                    $value = var_export($value, true);
+                }
+            });
+        }
+
         if (\is_bool($value) || \is_array($value) || (null === $value)) {
             $jsonString = json_encode($value);
 

Console/Descriptor/JsonDescriptor.php

@@ -184,6 +184,14 @@ private function writeData(array $data, array $options)
     {
         $flags = $options['json_encoding'] ?? 0;
 
+        // Recursively search for enum values, so we can replace it
+        // before json_encode (which will not display anything for \UnitEnum otherwise)
+        array_walk_recursive($data, static function (&$value) {
+            if ($value instanceof \UnitEnum) {
+                $value = var_export($value, true);
+            }
+        });
+
         $this->write(json_encode($data, $flags | \JSON_PRETTY_PRINT)."\n");
     }
 

Console/Descriptor/TextDescriptor.php

@@ -348,6 +348,8 @@ protected function describeContainerDefinition(Definition $definition, array $op
                     $argumentsInformation[] = sprintf('Service locator (%d element(s))', \count($argument->getValues()));
                 } elseif ($argument instanceof Definition) {
                     $argumentsInformation[] = 'Inlined Service';
+                } elseif ($argument instanceof \UnitEnum) {
+                    $argumentsInformation[] = var_export($argument, true);
                 } elseif ($argument instanceof AbstractArgument) {
                     $argumentsInformation[] = sprintf('Abstract argument (%s)', $argument->getText());
                 } else {

Console/Descriptor/XmlDescriptor.php

@@ -419,6 +419,9 @@ private function getArgumentNodes(array $arguments, \DOMDocument $dom): array
                 foreach ($this->getArgumentNodes($argument, $dom) as $childArgumentXML) {
                     $argumentXML->appendChild($childArgumentXML);
                 }
+            } elseif ($argument instanceof \UnitEnum) {
+                $argumentXML->setAttribute('type', 'constant');
+                $argumentXML->appendChild(new \DOMText(var_export($argument, true)));
             } else {
                 $argumentXML->appendChild(new \DOMText($argument));
             }

Controller/AbstractController.php

@@ -75,7 +75,7 @@ public function setContainer(ContainerInterface $container): ?ContainerInterface
     /**
      * Gets a container parameter by its name.
      *
-     * @return array|bool|float|int|string|null
+     * @return array|bool|float|int|string|\UnitEnum|null
      */
     protected function getParameter(string $name)
     {

DependencyInjection/FrameworkExtension.php

@@ -2256,7 +2256,8 @@ private function registerCacheConfiguration(array $config, ContainerBuilder $con
                 if (method_exists(TagAwareAdapter::class, 'setLogger')) {
                     $container
                         ->getDefinition($name)
-                        ->addMethodCall('setLogger', [new Reference('logger', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)]);
+                        ->addMethodCall('setLogger', [new Reference('logger', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)])
+                        ->addTag('monolog.logger', ['channel' => 'cache']);
                 }
 
                 $pool['name'] = $tagAwareId = $name;

Test/TestContainer.php

@@ -64,7 +64,7 @@ public function getParameterBag(): ParameterBagInterface
     /**
      * {@inheritdoc}
      *
-     * @return array|bool|float|int|string|null
+     * @return array|bool|float|int|string|\UnitEnum|null
      */
     public function getParameter(string $name)
     {

Tests/Console/Descriptor/AbstractDescriptorTest.php

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\FrameworkBundle\Tests\Console\Descriptor;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\FooUnitEnum;
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Output\BufferedOutput;
 use Symfony\Component\Console\Style\SymfonyStyle;
@@ -121,6 +122,10 @@ public function getDescribeContainerDefinitionWithArgumentsShownTestData()
             $definitionsWithArgs[str_replace('definition_', 'definition_arguments_', $key)] = $definition;
         }
 
+        if (\PHP_VERSION_ID >= 80100) {
+            $definitionsWithArgs['definition_arguments_with_enum'] = (new Definition('definition_with_enum'))->setArgument(0, FooUnitEnum::FOO);
+        }
+
         return $this->getDescriptionTestData($definitionsWithArgs);
     }
 
@@ -261,7 +266,7 @@ private function assertDescription($expectedDescription, $describedObject, array
         }
     }
 
-    private function getDescriptionTestData(array $objects)
+    private function getDescriptionTestData(iterable $objects)
     {
         $data = [];
         foreach ($objects as $name => $object) {

Tests/Console/Descriptor/ObjectsProvider.php

@@ -11,6 +11,8 @@
 
 namespace Symfony\Bundle\FrameworkBundle\Tests\Console\Descriptor;
 
+use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\FooUnitEnum;
+use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Suit;
 use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\Argument\AbstractArgument;
 use Symfony\Component\DependencyInjection\Argument\IteratorArgument;
@@ -62,14 +64,26 @@ public static function getRoutes()
 
     public static function getContainerParameters()
     {
-        return [
-            'parameters_1' => new ParameterBag([
-                'integer' => 12,
-                'string' => 'Hello world!',
-                'boolean' => true,
-                'array' => [12, 'Hello world!', true],
-            ]),
-        ];
+        yield 'parameters_1' => new ParameterBag([
+            'integer' => 12,
+            'string' => 'Hello world!',
+            'boolean' => true,
+            'array' => [12, 'Hello world!', true],
+        ]);
+
+        if (\PHP_VERSION_ID < 80100) {
+            return;
+        }
+
+        yield 'parameters_enums' => new ParameterBag([
+            'unit_enum' => FooUnitEnum::BAR,
+            'backed_enum' => Suit::Hearts,
+            'array_of_enums' => Suit::cases(),
+            'map' => [
+                'mixed' => [Suit::Hearts, FooUnitEnum::BAR],
+                'single' => FooUnitEnum::BAR,
+            ],
+        ]);
     }
 
     public static function getContainerParameter()

Tests/DependencyInjection/FrameworkExtensionTest.php

@@ -1645,6 +1645,7 @@ public function testCachePoolServices()
             $this->assertEquals([
                 ['setLogger', [new Reference('logger', ContainerInterface::IGNORE_ON_INVALID_REFERENCE)]],
             ], $tagAwareDefinition->getMethodCalls());
+            $this->assertSame([['channel' => 'cache']], $tagAwareDefinition->getTag('monolog.logger'));
         }
     }