09-29-2016, 02:51 PM
The change you made should be fine. We will make sure we get them rolled into the next 4.x release.
The part you removed the rawurlencode() from is the "to" field of the email. For PHPMailer to parse it correctly, it needs to have the @ in there correctly rather than url encoded. The email template also used urlencode() which with the changes you've implemented just needs to be changed to rawurlencode().
The fix I posted worked on my copy. Not sure why it wouldn't have worked on yours...that's the more peculiar part.
The part you removed the rawurlencode() from is the "to" field of the email. For PHPMailer to parse it correctly, it needs to have the @ in there correctly rather than url encoded. The email template also used urlencode() which with the changes you've implemented just needs to be changed to rawurlencode().
The fix I posted worked on my copy. Not sure why it wouldn't have worked on yours...that's the more peculiar part.