fix: restore FolderService userId in throwIfFileNotFound finally block#7538
Merged
vitormattos merged 9 commits intomainfrom Apr 22, 2026
Merged
fix: restore FolderService userId in throwIfFileNotFound finally block#7538vitormattos merged 9 commits intomainfrom
vitormattos merged 9 commits intomainfrom
Conversation
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
- add FolderSettings.path typing and regenerate OpenAPI artifacts - return controlled 404 when nodeId is invalid or source file was deleted - generalize WebDAV Behat step and streamline validate scenarios - add unit tests for throwIfFileNotFound behavior and userId restoration Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
vitormattos
force-pushed
the
fix/7494-invalid-node-external-id-docs
branch
from
9e6ef1c to
edb7291
Compare
- Apply PHP CS Fixer rules to AbstractIdentifyMethod and EmailTest - Regenerate OpenAPI artifacts Signed-off-by: Vitor Mattos <[email protected]>
Member
Author
|
/backport to stable33 |
Member
Author
|
/backport to stable32 |
Signed-off-by: Vitor Mattos <[email protected]>
vitormattos
force-pushed
the
fix/7494-invalid-node-external-id-docs
branch
from
b6686bf to
c010d1d
Compare
github-project-automation
Bot
moved this from 0. Needs triage
to 4. to release
in Roadmap
This was referenced Apr 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue
Fixes #7494 - Invalid node external ID causing file validation failures
Changes
throwIfFileNotFoundto guarantee userId restoration after temporary override for cross-storage file validationTest Coverage
Impact