Skip to content

Commit 4b8bcd2

Browse files
authored
fix(TemplateLayout): Refactor user ID handling
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
1 parent 7a0093c commit 4b8bcd2

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

lib/private/TemplateLayout.php

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -119,17 +119,18 @@ public function getPageTemplate(string $renderAs, string $appId): ITemplate {
119119
}
120120

121121
$user = Server::get(IUserSession::class)->getUser();
122+
$userId = $user?->getUID();
122123

123-
if ($user === null) {
124+
if (empty($userId)) {
124125
$page->assign('user_uid', false);
125126
$page->assign('user_displayname', false);
126127
$page->assign('userAvatarSet', false);
127128
$page->assign('userStatus', false);
128129
} else {
129-
$page->assign('user_uid', $user->getUID());
130+
$page->assign('user_uid', $userId);
130131
$page->assign('user_displayname', $user->getDisplayName());
131132
$page->assign('userAvatarSet', true);
132-
$page->assign('userAvatarVersion', $this->config->getUserValue($user->getUID(), 'avatar', 'version', 0));
133+
$page->assign('userAvatarVersion', $this->config->getUserValue($userId, 'avatar', 'version', 0));
133134
}
134135
break;
135136
case TemplateResponse::RENDER_AS_ERROR:
@@ -145,12 +146,15 @@ public function getPageTemplate(string $renderAs, string $appId): ITemplate {
145146

146147
$userDisplayName = false;
147148
$user = Server::get(IUserSession::class)->getUser();
148-
if ($user) {
149-
$userDisplayName = $user->getDisplayName();
150-
}
149+
$userId = $user?->getUID();
151150

152-
$page->assign('user_displayname', $userDisplayName);
153-
$page->assign('user_uid', \OC_User::getUser());
151+
if (empty($userId)) {
152+
$page->assign('user_displayname', false);
153+
$page->assign('user_uid', false);
154+
} else {
155+
$page->assign('user_displayname', $user->getDisplayName());
156+
$page->assign('user_uid', $userId);
157+
}
154158
break;
155159
case TemplateResponse::RENDER_AS_PUBLIC:
156160
$page = $this->templateManager->getTemplate('core', 'layout.public');

0 commit comments

Comments
 (0)