Skip to content

feat(files_sharing): Support multiples users in UserShareAccessUpdatedEvent - #57564

Merged
provokateurin merged 1 commit into
masterfrom
feat/files_sharing/share-access-updated-event-multiple-users
Jan 16, 2026
Merged

feat(files_sharing): Support multiples users in UserShareAccessUpdatedEvent#57564
provokateurin merged 1 commit into
masterfrom
feat/files_sharing/share-access-updated-event-multiple-users

Conversation

@provokateurin

Copy link
Copy Markdown
Member

Allows the listener handle cases where multiple users are affected more efficiently (e.g. Talk and Teams).

*/
public function __construct(
private readonly IUser $user,
private readonly array $users,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
privatereadonlyarray $users,
privatereadonlyIUser ...$users,

Another option to keep keep a bit more compatibility and ensure we only have IUser

…dEvent
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurinforce-pushed the feat/files_sharing/share-access-updated-event-multiple-users branch from dbb91d8 to 653d2a0CompareJanuary 15, 2026 10:07
@nextcloud-botnextcloud-bot mentioned this pull request Jan 15, 2026
@provokateurin
provokateurin merged commit fb1dc43 into masterJan 16, 2026
228 of 241 checks passed
@provokateurin
provokateurin deleted the feat/files_sharing/share-access-updated-event-multiple-users branch January 16, 2026 13:31
@nextcloud-botnextcloud-bot mentioned this pull request Jan 20, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@provokateurin@nickvergessen@icewind1991@Altahrim@CarlSchwan