fixes retrieving multiple values of extra fields

mvhirsch · mvhirsch · commit 765d8933a3a5 · 2023-02-01T14:24:34.000+01:00
diff --git a/src/Symfony/Component/Ldap/Security/LdapUserProvider.php b/src/Symfony/Component/Ldap/Security/LdapUserProvider.php
@@ -187,7 +187,7 @@ private function getAttributeValue(Entry $entry, string $attribute)
         }
 
         $values = $entry->getAttribute($attribute);
-        if (!in_array($attribute, [$this->uidKey, $this->passwordAttribute])) {
+        if (!\in_array($attribute, [$this->uidKey, $this->passwordAttribute])) {
             return $values;
         }
 
diff --git a/src/Symfony/Component/Ldap/Tests/Security/LdapUserProviderTest.php b/src/Symfony/Component/Ldap/Tests/Security/LdapUserProviderTest.php
@@ -345,7 +345,7 @@ public function testLoadUserByIdentifierIsSuccessfulWithMultipleExtraAttributes(
         $ldap = $this->createMock(LdapInterface::class);
         $memberOf = [
             'cn=foo,ou=MyBusiness,dc=symfony,dc=com',
-            'cn=bar,ou=MyBusiness,dc=symfony,dc=com'
+            'cn=bar,ou=MyBusiness,dc=symfony,dc=com',
         ];
         $result
             ->expects($this->once())

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ private function getAttributeValue(Entry $entry, string $attribute)`
`187`	`187`	`}`
`188`	`188`
`189`	`189`	`$values = $entry->getAttribute($attribute);`
`190`		`- if (!in_array($attribute, [$this->uidKey, $this->passwordAttribute])) {`
	`190`	`+ if (!\in_array($attribute, [$this->uidKey, $this->passwordAttribute])) {`
`191`	`191`	`return $values;`
`192`	`192`	`}`
`193`	`193`