Skip to content

[stable29] fix(provisioning_api): group admins can't add all users to groups - #66407

Open
backportbot[bot] wants to merge 1 commit into
stable29from
backport/66185/stable29
Open

backportbot[bot] wants to merge 1 commit into
stable29from
backport/66185/stable29

Conversation

@backportbot

@backportbot backportbot Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Backport of #66185

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot added 3. to review Waiting for reviews php Pull requests that update Php code labels Nov 6, 2026
@Altahrim
Altahrim force-pushed the backport/66185/stable29 branch from 2c2b056 to a542e81 Compare November 6, 2026 15:29
Group admins can't add users they are not supposed to see to their
groups

Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
@Altahrim
Altahrim force-pushed the backport/66185/stable29 branch from a542e81 to 6cfc948 Compare November 6, 2026 15:36
@Altahrim
Altahrim marked this pull request as ready for review November 6, 2026 15:36
$subAdminManager = $this->groupManager->getSubAdmin();
if (!$this->groupManager->isAdmin($loggedInUser->getUID()) && !$subAdminManager->isSubAdminOfGroup($loggedInUser, $group)) {
if (
!$this->groupManager->isAdmin($loggedInUser->getUID())

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants