Merge pull request #39437 from nextcloud/backport/39309/stable26

blizzz · web-flow · commit 105d0663f9c2 · 2023-08-02T21:16:31.000+02:00
[stable26] fix(profile): fix getUID on nullable user variable
diff --git a/lib/private/Profile/ProfileManager.php b/lib/private/Profile/ProfileManager.php
@@ -169,7 +169,7 @@ private function registerAction(ILinkAction $action, IUser $targetUser, ?IUser $
 				return;
 			}
 			if (!$this->appManager->isEnabledForUser($action->getAppId(), $visitingUser)) {
-				$this->logger->notice('App: ' . $action->getAppId() . ' cannot register actions as it is not enabled for the visiting user: ' . $visitingUser->getUID());
+				$this->logger->notice('App: ' . $action->getAppId() . ' cannot register actions as it is not enabled for the visiting user: ' . ($visitingUser ? $visitingUser->getUID() : '(user not connected)'));
 				return;
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ private function registerAction(ILinkAction $action, IUser $targetUser, ?IUser $`
`169`	`169`	`return;`
`170`	`170`	`}`
`171`	`171`	`if (!$this->appManager->isEnabledForUser($action->getAppId(), $visitingUser)) {`
`172`		`- $this->logger->notice('App: ' . $action->getAppId() . ' cannot register actions as it is not enabled for the visiting user: ' . $visitingUser->getUID());`
	`172`	`+ $this->logger->notice('App: ' . $action->getAppId() . ' cannot register actions as it is not enabled for the visiting user: ' . ($visitingUser ? $visitingUser->getUID() : '(user not connected)'));`
`173`	`173`	`return;`
`174`	`174`	`}`
`175`	`175`	`}`