Mahara

  1. Bug #1488697
  2. Comment #6

Comment 6 for bug 1488697

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

Reviewed: https://reviews.mahara.org/5209
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/9ff91d4824b5c55b7be6ba920288290b905ac68f
Submitter: Son Nguyen (<email address hidden>)
Branch: 15.04_STABLE

commit 9ff91d4824b5c55b7be6ba920288290b905ac68f
Author: Aaron Wells <email address hidden>
Date: Wed Aug 26 13:41:49 2015 +1200

Whitelist the $user properties sent to email_user() (Bug 1488697)

The old code passes every value from the form into email_user(), which
has wound up causing problems because email_user() thinks $user->id
refers to the user's ID, but in our case id refers to the usr_registration
record. (There is no user created yet at this point)

behatnotneeded: No infrastructure to test email yet

Change-Id: I0d862c2d1b2fdba5d5a1dc0068ce594207ecace9