This SMTP module does not play nice with Ubercart 6.2.2. Ubercart uses the format

"From Name"

for email addresses, and the SMTP module causes an error message to be printed at the top of the page when returning from Paypal as described here: http://www.ubercart.org/forum/bug_reports/14256/get_invalid_address_writ...

CommentFileSizeAuthor
#3 smtp_replyto_invalid_email.patch759 bytesjeffam

Comments

shaundychko’s picture

shaundychko
Location Vancouver, Canada
commented

sorry, I must remember the code tags.

Ubercart's email format is:

"From Name" <from@example.com>

franz’s picture

franz
any
Location Montréal
commented
Status: Active » Closed (duplicate)

Please test the patch on:

#279836-13: "Name" <email@example.com> FROM name fails

I think it solves the issue.

jeffam’s picture

jeffam
Primary language English
commented
Status: Closed (duplicate) » Needs review
StatusFileSize
new smtp_replyto_invalid_email.patch759 bytes

I'm not so sure that the patch referred to in #2 solves this issue. My error was with the reply-to header, and the attached patch fixed it for me.

franz’s picture

franz
any
Location Montréal
commented

Can someone test this? The patch seems fine to me.

artatac’s picture

artatac commented

Tested and the issue has now gone away - thanks Please roll into next update please

franz’s picture

franz
any
Location Montréal
commented
Version: 6.x-1.0-beta4 » 6.x-1.x-dev
Status: Needs review » Postponed (maintainer needs more info)

I know this is long gone, but the patch cannot be applied to current dev version.

@artatac, did your test include a patch?

artatac’s picture

artatac commented

I patched against beta 4 if that helps

franz’s picture

franz
any
Location Montréal
commented

The patch from #3?

sketchysteven’s picture

sketchysteven commented

What's the latest on this? Did the patch make it in beta 5?

wundo’s picture

wundo commented
Status: Postponed (maintainer needs more info) » Fixed

this is committed to 6.x-1.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.