Comment 3 for bug 888840

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

Reviewed: https://reviews.mahara.org/977
Committed: http://gitorious.org/mahara/mahara/commit/e4d9c24f9ea135cdb165c039b66bb98a7db7648f
Submitter: Francois Marier (<email address hidden>)
Branch: master

commit e4d9c24f9ea135cdb165c039b66bb98a7db7648f
Author: Richard Mansfield <email address hidden>
Date: Mon Jan 9 13:46:36 2012 +1300

    Fix warning when trying to sanitise missing ldap fields (bug #888840)

    During "update user info on login", we were attempting to sanitise all
    ldap attributes in the config, even when they are not set (commit
    46189cc1f8). This generates php warnings, which have become more
    obvious now that Display Name and Student ID have been added to the
    list of attributes (see commit 35d2f822da).

    Change-Id: I015855b956f0fcfa71db087ad1dfebffffd09396
    Signed-off-by: Richard Mansfield <email address hidden>