Adding support to remove data from deleted users, fixes #32

[stalker314314]

If user is deleted from Nextcloud, all files from that users are also deleted. However, we don't have callback for that, and we end up with bunch of images/faces/persons in database which do not exist anymore, either in filecache or physically, on disk. Since those stuff can pile up, best is to clean them from time to time, and we use another task for that.

Tests also added.

[matiasdelellis]

You have to use hooks for it..

https://docs.nextcloud.com/server/14/developer_manual/app/hooks.html#available-hooks

[stalker314314]

You have to use hooks for it..

Not sure I understand, can you clarify? I already explained why hooks are not an option in associated issue #32. Do you mean that or some other hooks?

[matiasdelellis]

You have to use hooks for it..

Not sure I understand, can you clarify? I already explained why hooks are not an option in associated issue #32. Do you mean that or sođe other hooks?

Ohh.. Now remember it.. 😞
During the day I check it... but I think we should report it to NC.. ;)

[stalker314314]

Filed issue nextcloud/server#14004, but I would not hold my breath on this:)

Friendly ping for you to take a look:)

[stalker314314]

OK, completely changed this pull request! Removed everything and replaced with hook. No tests, though (test for reset already exist). BTW, you can easily try it out with simple ./occ user:add and ./occ user:delete commands.

P.S. I was lucky to extract reset functionality into separate commit (c4d1b2f), so stuff here were easy to refactor.

[matiasdelellis]

Beautiful.. 😅

[matiasdelellis]

Just to be sure, when you were talking about business logic, you referring to using a single command that executes all the tasks?

[stalker314314]

Just to be sure, when you were talking about business logic, you referring to using a single command that executes all the tasks?

Yeah, in the sense "all of background processing". I think of it as all classes under Tasks. But, ignore my comments about "grouping business logic together", business logic is in Watcher.php too...