Merge branch '7.4' into 8.0

* 7.4:
  [DoctrineBridge] Make `EntityUserProvider`s pass attributes to their loader
  [WebProfilerBundle][HttpKernel] Display runner class in the profiler toolbar

Author: nicolas-grekas

Security/User/EntityUserProvider.php

@@ -18,6 +18,7 @@
 use Doctrine\Persistence\Proxy;
 use Symfony\Component\Security\Core\Exception\UnsupportedUserException;
 use Symfony\Component\Security\Core\Exception\UserNotFoundException;
+use Symfony\Component\Security\Core\User\AttributesBasedUserProviderInterface;
 use Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface;
 use Symfony\Component\Security\Core\User\PasswordUpgraderInterface;
 use Symfony\Component\Security\Core\User\UserInterface;
@@ -33,9 +34,9 @@
  *
  * @template TUser of UserInterface
  *
- * @template-implements UserProviderInterface<TUser>
+ * @template-implements AttributesBasedUserProviderInterface<TUser>
  */
-class EntityUserProvider implements UserProviderInterface, PasswordUpgraderInterface
+class EntityUserProvider implements AttributesBasedUserProviderInterface, PasswordUpgraderInterface
 {
     private string $class;
 
@@ -47,7 +48,7 @@ public function __construct(
     ) {
     }
 
-    public function loadUserByIdentifier(string $identifier): UserInterface
+    public function loadUserByIdentifier(string $identifier, ?array $attributes = null): UserInterface
     {
         $repository = $this->getRepository();
         if (null !== $this->property) {
@@ -57,7 +58,11 @@ public function loadUserByIdentifier(string $identifier): UserInterface
                 throw new \InvalidArgumentException(\sprintf('You must either make the "%s" entity Doctrine Repository ("%s") implement "Symfony\Bridge\Doctrine\Security\User\UserLoaderInterface" or set the "property" option in the corresponding entity provider configuration.', $this->classOrAlias, get_debug_type($repository)));
             }
 
-            $user = $repository->loadUserByIdentifier($identifier);
+            if (null === $attributes) {
+                $user = $repository->loadUserByIdentifier($identifier);
+            } else {
+                $user = $repository->loadUserByIdentifier($identifier, $attributes);
+            }
         }
 
         if (null === $user) {

Tests/Security/User/EntityUserProviderTest.php

@@ -171,6 +171,24 @@ public function testLoadUserByIdentifierShouldLoadUserWhenProperInterfaceProvide
         $provider->loadUserByIdentifier('name');
     }
 
+    public function testLoadUserByIdentifierShouldPassAttributesToTheUserLoader()
+    {
+        $repository = $this->createMock(UserLoaderRepository::class);
+        $repository->expects($this->once())
+            ->method('loadUserByIdentifier')
+            ->with('name', ['foo' => 'bar'])
+            ->willReturn(
+                $this->createMock(UserInterface::class)
+            );
+
+        $provider = new EntityUserProvider(
+            $this->getManager($this->getObjectManager($repository)),
+            'Symfony\Bridge\Doctrine\Tests\Fixtures\User'
+        );
+
+        $provider->loadUserByIdentifier('name', ['foo' => 'bar']);
+    }
+
     public function testLoadUserByIdentifierShouldDeclineInvalidInterface()
     {
         $repository = $this->createMock(ObjectRepository::class);