[Ldap] fix refreshUser() ignoring extra_fields

arkste · chalasr · commit cb8f12996c68 · 2020-06-10T10:36:58.000+02:00
diff --git a/src/Symfony/Component/Ldap/Security/LdapUserProvider.php b/src/Symfony/Component/Ldap/Security/LdapUserProvider.php
@@ -108,7 +108,7 @@ public function refreshUser(UserInterface $user)
             throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', \get_class($user)));
         }
 
-        return new LdapUser($user->getEntry(), $user->getUsername(), $user->getPassword(), $user->getRoles());
+        return new LdapUser($user->getEntry(), $user->getUsername(), $user->getPassword(), $user->getRoles(), $user->getExtraFields());
     }
 
     /**
diff --git a/src/Symfony/Component/Ldap/Tests/Security/LdapUserProviderTest.php b/src/Symfony/Component/Ldap/Tests/Security/LdapUserProviderTest.php
@@ -330,4 +330,14 @@ public function testLoadUserByUsernameIsSuccessfulWithPasswordAttribute()
         $provider = new LdapUserProvider($ldap, 'ou=MyBusiness,dc=symfony,dc=com', null, null, [], 'sAMAccountName', '({uid_key}={username})', 'userpassword', ['email']);
         $this->assertInstanceOf(LdapUser::class, $provider->loadUserByUsername('foo'));
     }
+
+    public function testRefreshUserShouldReturnUserWithSameProperties()
+    {
+        $ldap = $this->createMock(LdapInterface::class);
+        $provider = new LdapUserProvider($ldap, 'ou=MyBusiness,dc=symfony,dc=com', null, null, [], 'sAMAccountName', '({uid_key}={username})', 'userpassword', ['email']);
+
+        $user = new LdapUser(new Entry('foo'), 'foo', 'bar', ['ROLE_DUMMY'], ['email' => 'foo@symfony.com']);
+
+        $this->assertEquals($user, $provider->refreshUser($user));
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ public function refreshUser(UserInterface $user)`
`108`	`108`	`throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', \get_class($user)));`
`109`	`109`	`}`
`110`	`110`
`111`		`- return new LdapUser($user->getEntry(), $user->getUsername(), $user->getPassword(), $user->getRoles());`
	`111`	`+ return new LdapUser($user->getEntry(), $user->getUsername(), $user->getPassword(), $user->getRoles(), $user->getExtraFields());`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`/**`