[vanxa]

Summary

Handle folder deletion to go through TrashService.
Currently, folder deletion is a simple recursive deletion of the provided directory, and does not go through the TrashService functionality. A brief explanation to what needs to be achieved is provided in ResourceShareService:

   * <p><b>Folders are still hard-deleted here</b>, matching {@code FolderController}'s own delete
   * endpoint for the owner's own folders: this codebase has no folder-level trash mechanism at all
   * today ({@link TrashService#moveToTrash} only accepts a single regular file). Routing folder
   * deletes through trash is real follow-up work — it needs a trash design that can hold a whole
   * subtree, restore it, and account for its size against retention/quota — not a one-line change
   * here, so it is intentionally out of scope for this fix rather than silently expanding it.
   */

This PR attempts to implement this functionality.

AC:

• TrashEntries can be nested within each other (i.e. parent-child relationship)
• TrashService handles deletion of an entire folder
• Size calculation is expanded to account for an entire folder that is moved to trash
• When restoring a folder, quota and size calculations are adapted accordingly
• Listing trash items can handle trashed folders as well
• New code is covered with tests

Linked issue

Closes #155

How to test

Notes / Risk

[github-actions]

Agent review

The only change is an empty if (Files.isDirectory(source)) stub in moveToTrash. It has no effect, directories still throw ResourceNotFoundException, and there are no tests. See the inline comment.

Generated by Pull Request Review for #193 · copilot · auto · 10.1 AIC · ⌖ 0.509 AIC · ⊞ 7.1K

[github-actions]

This block is empty, so it does nothing. A directory path (e.g. moveToTrash(root, uid, "docs")) skips the block and then fails the !Files.isRegularFile(source) check just below with ResourceNotFoundException. Behaviour is unchanged. If directory trashing is meant to be implemented here, it needs real logic (move the tree, record trash metadata, restore support) plus tests. Otherwise drop this dead stub. Please don't merge a placeholder that appears to handle directories but doesn't.

[DenizAltunkapan]

@vanxa please check this