Skip to content

[stable26] Improve path resolution in files_version hooks - #42937

Merged
artonge merged 1 commit into
stable26from
artonge/backport/stable26/42891
Jan 25, 2024
Merged

[stable26] Improve path resolution in files_version hooks#42937
artonge merged 1 commit into
stable26from
artonge/backport/stable26/42891

Conversation

@artonge

Copy link
Copy Markdown
Collaborator

Manual backport of #42891

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artongeartonge self-assigned this Jan 18, 2024
@artongeartonge added 3. to review Waiting for reviews bug labels Jan 18, 2024
@artongeartonge added this to the Nextcloud 26.0.11 milestone Jan 18, 2024
@artongeartonge added the php Pull requests that update Php code label Jan 18, 2024
}
}

$owner = $node->getOwner()?->getUid();

Check notice

Code scanning / Psalm

DocblockTypeContradiction

OCP\IUser does not contain null
}
}

$owner = $node->getOwner()?->getUid();

Check notice

Code scanning / Psalm

RedundantConditionGivenDocblockType

Docblock-defined type OCP\IUser for $__tmp_nullsafe__11722 is never null
@blizzz

Copy link
Copy Markdown
Member

fixed the milestone

@artonge
artonge requested a review from a teamJanuary 25, 2024 16:35
@artonge
artonge enabled auto-merge January 25, 2024 17:26
@artonge
artonge merged commit 156fc2b into stable26Jan 25, 2024
@artonge
artonge deleted the artonge/backport/stable26/42891 branch January 25, 2024 21:40
@skjnldsvskjnldsv mentioned this pull request Feb 14, 2024
8 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugphpPull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@artonge@blizzz@ArtificialOwl@skjnldsv@github-advanced-security