fix(dav): Migrate from hooks to user events

[come-nc]

Part of #32128

Summary

Migrate from hooks to user events in dav application.
Also avoid issues if several users are deleted in the same request/process.

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[come-nc]

This error in oracle tests seems random:

There was 1 failure:

1) OCA\DAV\Tests\unit\CardDAV\CardDavBackendTest::testUpdateProperties
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-'UID'
+'FN'

/home/runner/actions-runner/_work/server/server/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php:537

I suppose depending on the run the order of UID and FN in the card array is different?
I’m not sure how it could be related to the changes in this PR, but I do not think I’ve seen the same failure on other PRs, has anyone encounter that before?

[tcitworld]

I suppose depending on the run the order of UID and FN in the card array is different?
I’m not sure how it could be related to the changes in this PR, but I do not think I’ve seen the same failure on other PRs, has anyone encounter that before?

Not sure what's causing this either, but as an easy fix I'd simply try to order the select from cards_properties above by name to see if that helps.

[come-nc]

I suppose depending on the run the order of UID and FN in the card array is different?
I’m not sure how it could be related to the changes in this PR, but I do not think I’ve seen the same failure on other PRs, has anyone encounter that before?

Not sure what's causing this either, but as an easy fix I'd simply try to order the select from cards_properties above by name to see if that helps.

Did that, it works, thanks.

[provokateurin]

Squash would be nice