Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable30] fix(CoreQueryBuilder): Use correct member entry for circle as initiator #1765

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Nov 21, 2024

Backport of #1758

@backportbot backportbot bot requested review from ArtificialOwl and mejo- November 21, 2024 11:02
@backportbot backportbot bot added this to the Nextcloud 30.0.3 milestone Nov 21, 2024
@mejo- mejo- force-pushed the backport/1758/stable30 branch 2 times, most recently from 8267fb6 to c46fb8a Compare November 25, 2024 10:27
Before, the member entry that matched the `singleId` of the user and
`level = 9` was used as initiator, which in practice means the member
entry for the internal user circle of the user most of the time.

Instead, we want to use the member entry that matches `singleId` of
the user and `circleId` of the circle in question.

This fixes the wrong `level` being set for `initiator` when calling
`circleProbe()` with `DataProbe::INITIATOR`.

Fixes: #1757

Signed-off-by: Jonas <[email protected]>
Required for testing (where only one php thread is available), but as
well for clients that require the event to be excecuted immediately.

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- force-pushed the backport/1758/stable30 branch from c46fb8a to 23e90f6 Compare November 25, 2024 10:37
@mejo- mejo- marked this pull request as ready for review November 25, 2024 10:41
@mejo- mejo- merged commit 5a70f6c into stable30 Nov 25, 2024
29 checks passed
@mejo- mejo- deleted the backport/1758/stable30 branch November 25, 2024 10:42
@skjnldsv skjnldsv mentioned this pull request Nov 27, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant