Mahara

  1. Bug #1488697
  2. Comment #7

Comment 7 for bug 1488697

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

Reviewed: https://reviews.mahara.org/5222
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/9898e9677a8c04b4b5ac44c201b08fc7185003df
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.10_STABLE

commit 9898e9677a8c04b4b5ac44c201b08fc7185003df
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
(cherry picked from commit 91cc873f0c42a237d5b9d6187b0526d194c01cc7)