Skip to content

[stable34] perf: Don't fetch all the trash items when deleting a single item #62180

Merged
AndyScherzinger merged 6 commits into
stable34from
carl/stable34/trashitem-delete-optimization
Jul 15, 2026
Merged

[stable34] perf: Don't fetch all the trash items when deleting a single item #62180
AndyScherzinger merged 6 commits into
stable34from
carl/stable34/trashitem-delete-optimization

Conversation

@CarlSchwan

@CarlSchwan CarlSchwan commented Jul 15, 2026 •
edited by cristianscheid
Loading

Copy link
Copy Markdown
Member

Manual backport of #61906

Summary

TODO

  • ...

Checklist

AI (if applicable)

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

@CarlSchwan
refactor: Extract buildFileInfo in a seperate method
06437f3 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
perf: Don't fetch all the trash items when deleting a single item
a8bb264 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
refactor: Extract and improve ITrashItem mapping to INode
d177e46 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
refactor: Remove some dead code from the trashbin Helper
2c539db 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
refactor(trashbin): Replace mapTrashItems by mapTrashItem
b25551d 
Only take one argument and instead call this with array map

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan added this to the Nextcloud 34.0.2 milestone Jul 15, 2026
@CarlSchwan CarlSchwan self-assigned this Jul 15, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner July 15, 2026 13:10
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Jul 15, 2026
@CarlSchwan
CarlSchwan requested review from icewind1991 and removed request for a team July 15, 2026 13:10
@github-actions github-actions Bot changed the title Carl/stable34/trashitem delete optimization [stable34] Carl/stable34/trashitem delete optimization Jul 15, 2026
cristianscheid
cristianscheid reviewed Jul 15, 2026
View reviewed changes
Comment thread apps/files_trashbin/lib/Trash/ITrashManager.php Outdated
cristianscheid
cristianscheid reviewed Jul 15, 2026
View reviewed changes
Comment thread apps/files_trashbin/lib/Trash/ITrashManager.php Outdated
@CarlSchwan CarlSchwan changed the title [stable34] Carl/stable34/trashitem delete optimization [stable34] perf: Don't fetch all the trash items when deleting a single item Jul 15, 2026
@CarlSchwan
fix: Adjust API @SInCE version
5d4b58f 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwan force-pushed the carl/stable34/trashitem-delete-optimization branch from edc55e3 to 5d4b58f Compare July 15, 2026 14:02
@AndyScherzinger
AndyScherzinger merged commit 78001ed into stable34 Jul 15, 2026
177 of 187 checks passed
@AndyScherzinger
AndyScherzinger deleted the carl/stable34/trashitem-delete-optimization branch July 15, 2026 22:27
@nextcloud-bot nextcloud-bot mentioned this pull request Jul 16, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes
@cristianscheid cristianscheid cristianscheid approved these changes
@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend
@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend
@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend

Assignees

@CarlSchwan CarlSchwan

Labels

3. to review Waiting for reviews performance 🚀

Projects

None yet

Milestone

Nextcloud 34.0.2

Development

Successfully merging this pull request may close these issues.

3 participants

@CarlSchwan @AndyScherzinger @cristianscheid