Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable29] fix: try to find non-recursive share source #47278

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Aug 16, 2024

Backport of PR #46881

instead of always picking the first one, try to find one that won't blow up

Signed-off-by: Robin Appelman <[email protected]>
$this->cache = null;
break;
}
if (!$this->nonMaskedStorage) {

Check notice

Code scanning / Psalm

DocblockTypeContradiction Note

Operand of type false is always falsy
$this->cache = null;
break;
}
if (!$this->nonMaskedStorage) {

Check notice

Code scanning / Psalm

DocblockTypeContradiction Note

Docblock-defined type OCP\Files\Storage\IStorage for $this->nonMaskedStorage is never falsy
@sorbaugh sorbaugh merged commit b8a864a into stable29 Aug 19, 2024
176 of 177 checks passed
@sorbaugh sorbaugh deleted the backport/46881/stable29 branch August 19, 2024 14:44
@skjnldsv skjnldsv mentioned this pull request Aug 20, 2024
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants