diff --git a/lib/UserBackend.php b/lib/UserBackend.php index 4110bcddc..ac07114ec 100644 --- a/lib/UserBackend.php +++ b/lib/UserBackend.php @@ -541,7 +541,7 @@ public function updateAttributes(string $uid): void { $currentDisplayname = $this->getDisplayName($uid); if ($newDisplayname !== null && $currentDisplayname !== $newDisplayname) { - $this->setDisplayName($uid, $newDisplayname); + $user->setDisplayName($newDisplayname); $this->logger->debug('Display name updated', ['app' => 'user_saml', 'user' => $user->getUID()]); $this->eventDispatcher->dispatchTyped(new UserChangedEvent($user, 'displayName', $newDisplayname, $currentDisplayname)); $this->logger->debug('Display name update event dispatched', ['app' => 'user_saml', 'user' => $user->getUID()]); diff --git a/tests/unit/UserBackendTest.php b/tests/unit/UserBackendTest.php index 3ab06a72e..e91a37c8f 100644 --- a/tests/unit/UserBackendTest.php +++ b/tests/unit/UserBackendTest.php @@ -289,10 +289,9 @@ public function testUpdateAttributes(): void { ->method('getDisplayName') ->with('ExistingUser') ->willReturn(''); - $this->userBackend - ->expects($this->once()) + $user->expects($this->once()) ->method('setDisplayName') - ->with('ExistingUser', 'New Displayname'); + ->with('New Displayname'); $this->groupManager ->expects($this->once()) ->method('handleIncomingGroups') @@ -343,10 +342,9 @@ public function testUpdateAttributesQuotaDefaultFallback(): void { ->method('getDisplayName') ->with('ExistingUser') ->willReturn(''); - $this->userBackend - ->expects($this->once()) + $user->expects($this->once()) ->method('setDisplayName') - ->with('ExistingUser', 'New Displayname'); + ->with('New Displayname'); $this->eventDispatcher->expects($this->once()) ->method('dispatchTyped') ->with(new UserChangedEvent($user, 'displayName', 'New Displayname', ''));