Skip to content

feat: Make it possible to add a storage wrapper using public APIs#61493

Open
CarlSchwan wants to merge 2 commits into
masterfrom
carl/addStorageWrapper
Open

feat: Make it possible to add a storage wrapper using public APIs#61493
CarlSchwan wants to merge 2 commits into
masterfrom
carl/addStorageWrapper

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Summary

Before that every app is using an internal call to Filesystem::addStorageWrapper. The new API enforce that this is called at the right moment.

Required for nextcloud/files_lock#1125

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Jun 22, 2026
@CarlSchwan CarlSchwan self-assigned this Jun 22, 2026
@CarlSchwan CarlSchwan requested a review from a team as a code owner June 22, 2026 10:43
@CarlSchwan CarlSchwan requested review from Altahrim, ArtificialOwl, come-nc and leftybournes and removed request for a team June 22, 2026 10:43
Before that every app is using an internal call to
Filesystem::addStorageWrapper. The new API enforce that this is called
at the right moment.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the carl/addStorageWrapper branch from 64e7a6c to d6cf529 Compare June 22, 2026 10:44
Comment thread lib/public/Files/Events/BeforeFileSystemSetupEvent.php
@CarlSchwan CarlSchwan force-pushed the carl/addStorageWrapper branch from b26da95 to df9d3c9 Compare June 22, 2026 11:45
Comment thread lib/private/Files/SetupManager.php Outdated
@CarlSchwan CarlSchwan force-pushed the carl/addStorageWrapper branch 3 times, most recently from ec69522 to 84052f8 Compare June 22, 2026 13:55
Remove call to deprecated Filesystem api and allow to reuse instead of
the mounts points between addStorageFactory.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the carl/addStorageWrapper branch from 84052f8 to 415aeeb Compare June 22, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants