Hey, thank you very much for creating this PR and sry for the late answer!

rebased, fixed code style issues and changed output to be a bit more concise

I think this will also potentially:
Close #30074

Looks like this is ready to go?

Looking forward to this going live, but would be nice if it had the option if restoring a specific file based on path or id.

@Grunthos I think it would be fairly easy to add that in an inelegant way -- restoreDeletedFiles already handles the individual files it restores by $filename, so you could accept an optional input list of filenames to restore, then check for membership. I am sure there is a faster and better way to accomplish this, though.

@szaimen as far as I am concerned yes :)

@cahogan I was just thinking of something based on internal object IDs from the database; in many cases where accidental deletions have occurred they are pretty easy to find. If a path was specifiable, then a wildcard recursive path would be nice ;-).

/rebase

seems our bot likely cannot rebase on forks

I've retriggered CI to get an up to date result

I rebased manually. Lets see what CI says

I think the failures are unrelated.
cc @PVince81 if we can merge this

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@cahogan can you update the documentation as well ? https://docs.nextcloud.com/server/stable/admin_manual/configuration_server/occ_command.html?highlight=occ#trashbin-label

the repo is here: https://github.com/nextcloud/documentation/tree/master/admin_manual/configuration_server

@cahogan ping, can you add the docs as per my previous comment ?
let me know if you can't so someone else can take care.
thanks!

seems the documentation is not yet updated: https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/occ_command.html#trashbin

nextcloud/documentation#9521

Hi all, my apologies -- I didn't receive notifications from this thread. Thank you very much for adding documentation -- is there any additional info that I could help with at this point, or is the PR complete?

@cahogan it is complete now. thanks!