Ldap sync_groups throws error if no admin members

Bug #1804959 reported by Robert Lyon on 2018-11-24
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Medium
Unassigned

Bug Description

When creating the list of members for the group sync/update we use the site admin to be the admin of the group

But if the site admin is already a normal member of the group then things fail

Robert Lyon (robertl-9) on 2018-11-24
Changed in mahara:
milestone: none → 19.04.0
importance: Undecided → Medium
status: New → In Progress

Reviewed: https://reviews.mahara.org/9333
Committed: https://git.mahara.org/mahara/mahara/commit/0a42ea6561923b08ff3bcfb2d1936e2a59ff8e26
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 0a42ea6561923b08ff3bcfb2d1936e2a59ff8e26
Author: Robert Lyon <email address hidden>
Date: Sun Nov 25 08:41:06 2018 +1300

Bug 1804959: Make sure there is at least 1 admin

In members for ldap sync_group

behatnotneeded

Change-Id: I15a69e4ba4b90d95a9e4dbbb61c04235b4dcac62
Signed-off-by: Robert Lyon <email address hidden>

Changed in mahara:
status: In Progress → Fix Committed
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers