Comment 2 for bug 1488697
Revision history for this message
| Mahara Bot (dev-mahara) wrote A change has been merged | #2 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Reviewed: https:/
Committed: https:/
Submitter: Son Nguyen (<email address hidden>)
Branch: master
commit 91cc873f0c42a23
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: I0d862c2d1b2fdb