mahara-contributors team mailing list archive

[Mahara Bot <1705547@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/7892
Committed: https://git.mahara.org/mahara/mahara/commit/b163548fd10848afeb3ffeff23ecc3cbf508e806
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit b163548fd10848afeb3ffeff23ecc3cbf508e806
Author: Gilles-Philippe Leblanc <gilles-philippe.leblanc@xxxxxxxxxxxx>
Date:   Thu Jul 20 22:12:19 2017 -0400

Bug 1705547: Fix for error with auth_ldap_sync_cron

behatnotneeded
Change-Id: Ib51cd670a4cebee9756b896ec83ef26c9a5f5d85

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1705547

Title:
  Error with auth_ldap_sync_cron

Status in Mahara:
  Fix Committed
Status in Mahara 17.04 series:
  Fix Committed
Status in Mahara 17.10 series:
  Fix Committed

Bug description:
  We have this error in our log :
  [WAR] eb (auth/ldap/lib.php:1627) Undefined property: stdClass::$active

  The cause is the the db record do not contain "active":
  $auths = get_records_array('auth_instance', 'authname', 'ldap', 'id', 'id');

  It can be fixed by adding it :
  $auths = get_records_array('auth_instance', 'authname', 'ldap', 'id', 'id,active');

  I think its a regression caused by :
  Bug 1640308: Adding an auth_instance "active" column

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1705547/+subscriptions