Skip to content

Conversation

@PVince81
Copy link
Member

Backport of #34599 to stable25

skjnldsv and others added 5 commits October 14, 2022 16:19
@PVince81 PVince81 added the 3. to review Waiting for reviews label Oct 14, 2022
@PVince81 PVince81 added this to the Nextcloud 25 milestone Oct 14, 2022
@PVince81 PVince81 self-assigned this Oct 14, 2022
$this->defaultTheme = $defaultTheme;
$this->config = $config;
if ($userSession->getUser() !== null) {
$this->userId = $userSession->getUser()->getUID();

Check notice

Code scanning / Psalm

PossiblyNullReference

Cannot call method getUID on possibly null value
@blizzz blizzz mentioned this pull request Oct 14, 2022
1 task
ILogger $logger,
ITempManager $tempManager
) {
ITempManager $tempManager) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

less readable imo, but does not matter

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@blizzz blizzz added bug 4. to release Ready to be released and/or waiting for tests to finish regression and removed 3. to review Waiting for reviews labels Oct 14, 2022
@PVince81 PVince81 merged commit e487d82 into stable25 Oct 14, 2022
@PVince81 PVince81 deleted the stable25-fix-background-appdata-scrope branch October 14, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants