Merge branch '5.4' into 6.0

* 5.4:
  Fix CI by removing ext-mongodb
  [ErrorHandler] fix parsing `@param` with dollars in the description
  [SecurityBundle] Fix wrong array key
  [SecurityBundle] Fix listing listeners in profiler when authenticator manager is disabled
  Fix FlattenException::setPrevious argument typing
  bug #43941 [FrameworkBundle] Comment design decision
  [DependencyInjection] fix auto-refresh when inline_factories is enabled

Author: nicolas-grekas

DebugClassLoader.php

@@ -1183,11 +1183,8 @@ private function parsePhpDoc(\Reflector $reflector): array
             if (false === $i = strpos($param, '$')) {
                 continue;
             }
-            if (false !== $j = strpos($param, ' $') ?: strpos($param, '&$')) {
-                $i = 1 + $j;
-            }
 
-            $type = 0 === $i ? '' : rtrim(substr($param, 0, $i - 1));
+            $type = 0 === $i ? '' : rtrim(substr($param, 0, $i), ' &');
             $param = substr($param, 1 + $i, (strpos($param, ' ', $i) ?: (1 + $i + \strlen($param))) - $i - 1);
 
             $tags['param'][$param] = $type;

Exception/FlattenException.php

@@ -233,7 +233,7 @@ public function getPrevious(): ?self
     /**
      * @return $this
      */
-    public function setPrevious(self $previous): static
+    public function setPrevious(?self $previous): static
     {
         $this->previous = $previous;
 

Tests/DebugClassLoaderTest.php

@@ -295,7 +295,6 @@ class_exists(Fixtures\SubClassWithAnnotatedParameters::class, true);
             'The "Symfony\Component\ErrorHandler\Tests\Fixtures\SubClassWithAnnotatedParameters::iAmHere()" method will require a new "callable $callback" argument in the next major version of its interface "Symfony\Component\ErrorHandler\Tests\Fixtures\InterfaceWithAnnotatedParameters", not defining it is deprecated.',
             'The "Symfony\Component\ErrorHandler\Tests\Fixtures\SubClassWithAnnotatedParameters::iAmHere()" method will require a new "string $param" argument in the next major version of its interface "Symfony\Component\ErrorHandler\Tests\Fixtures\InterfaceWithAnnotatedParameters", not defining it is deprecated.',
             'The "Symfony\Component\ErrorHandler\Tests\Fixtures\SubClassWithAnnotatedParameters::iAmHere()" method will require a new "callable $anotherOne" argument in the next major version of its interface "Symfony\Component\ErrorHandler\Tests\Fixtures\InterfaceWithAnnotatedParameters", not defining it is deprecated.',
-            'The "Symfony\Component\ErrorHandler\Tests\Fixtures\SubClassWithAnnotatedParameters::iAmHere()" method will require a new "Type$WithDollarIsStillAType $ccc" argument in the next major version of its interface "Symfony\Component\ErrorHandler\Tests\Fixtures\InterfaceWithAnnotatedParameters", not defining it is deprecated.',
             'The "Symfony\Component\ErrorHandler\Tests\Fixtures\SubClassWithAnnotatedParameters::isSymfony()" method will require a new "true $yes" argument in the next major version of its parent class "Symfony\Component\ErrorHandler\Tests\Fixtures\ClassWithAnnotatedParameters", not defining it is deprecated.',
         ], $deprecations);
     }

Tests/Fixtures/InterfaceWithAnnotatedParameters.php

@@ -13,14 +13,13 @@ interface InterfaceWithAnnotatedParameters
     public function whereAmI();
 
     /**
-     * @param       $noType
+     * @param       $noType with $dollar after
      * @param callable(\Throwable|null $reason, mixed $value) $callback and a comment
      * about this great param
      * @param string                                          $param (comment with $dollar)
      * @param $defined
      * @param  callable  ($a,  $b)  $anotherOne
      * @param callable (mixed $a, $b) $definedCallable
-     * @param Type$WithDollarIsStillAType $ccc
      * @param \JustAType
      */
     public function iAmHere();