Skip to content

fix(files_trashbin): create folder if missing when expiring trash#61442

Open
invario wants to merge 1 commit into
nextcloud:masterfrom
invario:fix_files_trashbin_exception
Open

fix(files_trashbin): create folder if missing when expiring trash#61442
invario wants to merge 1 commit into
nextcloud:masterfrom
invario:fix_files_trashbin_exception

Conversation

@invario

@invario invario commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

If the background job to expire trash was executed before a user had ever trashed a file, the files_trashbin folder would be missing and an exception was thrown. Change the method used from get to getOrCreateFolder instead.

Checklist

@invario invario requested a review from a team as a code owner June 19, 2026 13:03
@invario invario requested review from Altahrim, ArtificialOwl, CarlSchwan and icewind1991 and removed request for a team June 19, 2026 13:03
@invario invario changed the title fix(files_trashbin): create folder if missing when expiring fix(files_trashbin): create folder if missing when expiring trash Jun 19, 2026
@phonon112358

Copy link
Copy Markdown

The PR also solves the issue for NC 34

Comment thread apps/files_trashbin/lib/BackgroundJob/ExpireTrash.php Outdated
Signed-off-by: invario <67800603+invario@users.noreply.github.com>
@invario invario force-pushed the fix_files_trashbin_exception branch from fa93943 to 359aeda Compare June 19, 2026 17:01
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.

[Bug]: files_trashbin logged NotFoundException /USERNAME/files_trashbin Error while expiring trashbin for user USERNAME

3 participants