Skip to content

Fix: External Storage Support#55

Closed
zango-me wants to merge 1 commit intonextcloud:mainfrom
zango-me:main
Closed

Fix: External Storage Support#55
zango-me wants to merge 1 commit intonextcloud:mainfrom
zango-me:main

Conversation

@zango-me
Copy link
Copy Markdown

Added support to load files by id from storage mounted by the "external storage" app

Added support to load files by id from storage mounted by the "external storage" app

Signed-off-by: Zango <ace@acewolf.de>
@Rikdekker
Copy link
Copy Markdown
Collaborator

Thanks for the contribution @zango-me! 🙏

We've incorporated your external storage support into v1.1.1 with a few adjustments:

  • Removed the global rootFolder->getById() lookup upfront — instead, external storage is handled as Case 3 in the existing structured flow (after personal folders and group folders)
  • This avoids redundant lookups and keeps the method's logic consistent
  • Modernized strpos() to str_starts_with() (PHP 8.2+ is required)
  • Kept your findUserWithStorage() helper as-is — clean approach

The release is live: v1.1.1 Changelog

@Rikdekker Rikdekker closed this Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants