Fixes #24497

@blizzz Sorry for my noob question, but is there anything I should or could do to proceed this PR? The codesniffer check fails, but not on a line that I have manipulated. If from now on, it's all out of my hands, that's fine with me ;)

Fixes #24497

@blizzz ping?

@blizzz just wanted to confirm that these patches still work on version 21.0.1.

@blizzz Patches still work with newest version (22.2.3)

@blizzz Patches still apply cleanly with version 22.2.5. System is running smoothly since the PR was created.

@jospoortvliet Last ping from my end. First time collaborator to an open source project, and very disappointing to see absolutely noone cares.

@zenlord any update here? :)

@szaimen @blizzz @come-nc Finally got around resolving the comments made by @come-nc . Instead of doing a rebase, I have simply created a new clone, applied the original commits as a patch, made the code changes and then force-pushed the result to my fork. I hope this is acceptable, because going through all the merges required from rebasing against Nextcloud 20-something got dull very quickly :/

@come-nc Looks like this one is better: single commit and DCO is not complaining.

25 is feature freezed so this will go into 26, waiting for 25 to branch out before merging into master.

Thank you for this contribution @zenlord !

(actually needs 1 more review so I put back "to review" label)

/rebase

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