aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFerdinand Thiessen <opensource@fthiessen.de>2024-09-11 11:26:59 +0200
committerJohn Molakvoæ <skjnldsv@users.noreply.github.com>2025-01-28 11:22:08 +0100
commit00286eefa819110ec88c05a5e115690f4f7a7cba (patch)
tree0855b08af9a6c435db8dfa017f6dc8893c95ea4b
parent5650a59cfc7530450424ee04803d75778955b5d6 (diff)
downloadnextcloud-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.php7
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;
}