fix: Fire UserChangedEvent only after change happenned

This is not a before event, so the user should already have been changed when it fires. Should fix issues with dav address book and newly mapped saml users. Signed-off-by: Côme Chilliet <[email protected]>
nextcloud · Aug 16, 2024 · 8355cb0 · 8355cb0
1 parent 60d8302
commit 8355cb0
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/UserBackend.php b/lib/UserBackend.php
@@ -657,8 +657,8 @@ public function updateAttributes($uid,
 			$currentDisplayname = $this->getDisplayName($uid);
 			if ($newDisplayname !== null
 				&& $currentDisplayname !== $newDisplayname) {
-				$this->eventDispatcher->dispatchTyped(new UserChangedEvent($user, 'displayName', $newDisplayname, $currentDisplayname));
 				$this->setDisplayName($uid, $newDisplayname);
+				$this->eventDispatcher->dispatchTyped(new UserChangedEvent($user, 'displayName', $newDisplayname, $currentDisplayname));
 			}
 
 			if ($newQuota !== null) {