diff options
author | Ferdinand Thiessen <opensource@fthiessen.de> | 2024-09-11 11:26:59 +0200 |
---|---|---|
committer | John Molakvoæ <skjnldsv@users.noreply.github.com> | 2025-01-28 11:22:08 +0100 |
commit | 00286eefa819110ec88c05a5e115690f4f7a7cba (patch) | |
tree | 0855b08af9a6c435db8dfa017f6dc8893c95ea4b | |
parent | 5650a59cfc7530450424ee04803d75778955b5d6 (diff) | |
download | nextcloud-server-backport/47889/stable31.tar.gz nextcloud-server-backport/47889/stable31.zip |
fix: `user:settings` command when user is not availablebackport/47889/stable31
If `ignore-missing-user` all sub commands work, except listing all settings
for a user like `occ user:settings --ignore-missing-user user core`.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
-rw-r--r-- | core/Command/User/Setting.php | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/core/Command/User/Setting.php b/core/Command/User/Setting.php index e2e65f7d5f9..16e851d8252 100644 --- a/core/Command/User/Setting.php +++ b/core/Command/User/Setting.php @@ -219,7 +219,7 @@ class Setting extends Base { } } - protected function getUserSettings($uid, $app) { + protected function getUserSettings(string $uid, string $app): array { $settings = $this->config->getAllUserValues($uid); if ($app !== '') { if (isset($settings[$app])) { @@ -230,7 +230,10 @@ class Setting extends Base { } $user = $this->userManager->get($uid); - $settings['settings']['display_name'] = $user->getDisplayName(); + if ($user !== null) { + // Only add the display name if the user exists + $settings['settings']['display_name'] = $user->getDisplayName(); + } return $settings; } |