Ldap sync_groups throws error if no admin members

Bug #1804959 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
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)
Changed in mahara:
milestone: none → 19.04.0
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/9333

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.