Comment 12 for bug 1606101

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

Reviewed: https://reviews.mahara.org/6766
Committed: https://git.mahara.org/mahara/mahara/commit/dbaebc656a35fbed870ed5b3c9162ccb5bcb9464
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.10_STABLE

commit dbaebc656a35fbed870ed5b3c9162ccb5bcb9464
Author: Ghada El-Zoghbi <email address hidden>
Date: Mon Jul 25 17:14:01 2016 +1000

Bug 1606101: usr.suspendedcusr must be non-zero

It turns out a lot of existing code checks the boolean
value of usr.suspendedcusr to determine if a user should
be treated as suspended or not. The LDAP sync cron (and,
indeed, any code suspending users via a cron task) was
setting usr.suspendedcusr to 0, which is boolean false,
so these users would be treated as not suspended.

We are going to update all usr.suspendedcusr = 0
to a valid site admin ID.

Change-Id: Iecfbfd8a4cdd98d5d07149bb40c64308262ea234
behatnotneeded: Test to come later