Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 21, 2024

Backport of #43727

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from artonge, skjnldsv and susnux February 21, 2024 17:39
@backportbot backportbot bot added 3. to review Waiting for reviews feature: versions javascript php Pull requests that update Php code labels Feb 21, 2024
@backportbot backportbot bot added this to the Nextcloud 27.1.7 milestone Feb 21, 2024
@artonge
Copy link
Contributor

artonge commented Feb 21, 2024

/compile amend /

}

if ($sourceFile->getOwner()?->getUID() !== $currentUserId) {
$nodes = $this->rootFolder->getUserFolder($currentUserId)->getById($sourceFile->getId());

Check notice

Code scanning / Psalm

PossiblyNullArgument

Argument 1 of OCP\Files\Folder::getById cannot be null, possibly null value provided
@artonge artonge marked this pull request as ready for review February 21, 2024 20:46
@artonge artonge force-pushed the backport/43727/stable27 branch from 4f07984 to 4c710b8 Compare February 21, 2024 20:46
@artonge artonge force-pushed the backport/43727/stable27 branch from 4c710b8 to 0820481 Compare February 21, 2024 20:48
@solracsf solracsf mentioned this pull request Feb 21, 2024
@skjnldsv skjnldsv merged commit 245a439 into stable27 Feb 22, 2024
@skjnldsv skjnldsv deleted the backport/43727/stable27 branch February 22, 2024 09:45
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 22, 2024
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 feature: versions javascript php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants