upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-05-28 04:32:30 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 25

fixing Psalm messages

Browse source 
Signed-off-by: Marc Hefter <marchefter@gmail.com>
This commit is contained in:
Marc Hefter 2023-02-27 09:43:42 +01:00
parent f812b87576
commit c6408587ed
No known key found for this signature in database
GPG key ID: AC45BA6A849F8610
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
apps/user_ldap/lib/User/User.php
View file

@ -583,11 +583,6 @@ class User {
*
* @param array $profileValues associaive array of property keys and values from LDAP
* @param string|null $profileScope the scope of visibility to set
* @var string $property the array key (property name from AccountManager class)
* @var string $valueFromLDAP the value as read from LDAP
* @var string $propertyValue
* @var string $value
* @var string $currentValue
*/
private function updateProfile(array $profileValues, ?string $profileScope=null): void {
// check if user profile was refreshed before
@ -604,16 +599,21 @@ class User {
return;
}
// loop through the properties and handle them
/** @var string $property the array key (property name from AccountManager class) */
/** @var string $valueFromLDAP the value as read from LDAP */
foreach($profileValues as $property => $valueFromLDAP) {
$this->logger->debug('user profile data ('.$property.') from LDAP '.$this->dn.' ='.((string)$valueFromLDAP), ['app' => 'user_ldap']);
$this->logger->debug('user profile data ('.$property.') from LDAP '.$this->dn, ['app' => 'user_ldap']);
// check and update profile properties
/** @var string $propertyValue */
$propertyValue = [$valueFromLDAP];
if ($propertyValue && isset($propertyValue[0])) {
if (isset($propertyValue[0])) {
/** @var string $value */
$value = $propertyValue[0];
try {
/** @var string $currentValue */
$currentValue = (string)$user->getProfilePropertyValue($property);
if ($currentValue !== $value) {
$user->setProfileProperty($property,$value,$scope,null);
$user->setProfileProperty($property,$value,$profileScope,null);
$this->logger->debug('property updated: '.$property.'='.$value.' for user '.$this->getUsername().'', ['app' => 'user_ldap']);
}
} catch (PropertyDoesNotExistException $e) {