[stable33] perf(sharing): Use new user field from VerifyMountEvent

[icewind1991]

• Backport of perf(sharing): Use new user field from VerifyMountEvent #17046 to stable33

---

• See [stable33] feat: improve VerifyMountPointEvent event server#59546 for the server part.
• Server part is included in [stable33] authoritative share - revival server#59177

Since not all 33 versions will have the updated event, I've gated it behind method_exists calls, falling back to the existing logic.
A bit hacky, but the performance gains are to significant to leave behind.

[nickvergessen]

Since not all 33 versions will have the updated event, I've gated it behind method_exists calls, falling back to the existing logic.

👍

[nickvergessen]

Looks good, pushed a commit to run CI against the server PR

[nickvergessen]

OCA\Talk\Config::getAttachmentFolder(): Argument #1 ($userId) must be of type string, OC\User\LazyUser given, called in /home/runner/work/spreed/spreed/apps/spreed/lib/Share/Listener.php on line 79

🙈

[nickvergessen]

Seems it now needs some more adjustments about the placeholders

But its not changed on main: https://github.com/nextcloud/spreed/blob/772c540f6ddf202d2e2e1bff8725d83fc236ce6d/tests/integration/features/sharing-4/transfer-ownership.feature

[icewind1991]

Same error happens with nextcloud/server#57760 so it's an effect of the other share handling changes.

[nickvergessen]

• Right, so that is test: Run tests against nextcloud/server#57760 share-update-check-single #16987 for sharing-1 and sharing-4
• And test: Adjust to server change, the Talk/ folder is now staying also w… #16890 for sharing-2