Mahara

ldap sync users error when there is no email address for a user

Bug #1903592 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Unassigned
Mahara 21.04.0

Bug Description

When syncing a user it does a check to make sure that there is no email collision/clash

But this bit of code will fail if the checking doesn't have an email to check against, eg tries to check 'null'

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/11424

Robert Lyon (robertl-9)
Changed in mahara:
status: New → In Progress
importance: Undecided → High
Kristina Hoeppner (kris-hoeppner)
Changed in mahara:
importance: High → Medium
milestone: none → 21.04.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/11424
Committed: https://git.mahara.org/mahara/mahara/commit/8bcb937f864265d41ef7715b1da22e785d293925
Submitter: Lisa Seeto (<email address hidden>)
Branch: master

commit 8bcb937f864265d41ef7715b1da22e785d293925
Author: Robert Lyon <email address hidden>
Date: Tue Nov 10 10:00:35 2020 +1300

Bug 1903592: Only try to check the email if supplied

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

Lisa Seeto (lisaseeto)
Changed in mahara:
status: In Progress → Fix Committed
Robert Lyon (robertl-9)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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