I upgraded from 1.3 to 2.0-Alpha2.0 to fix duplicate emails being sent out. (and upgraded from 1.1 to 1.3 to solve other issues...) But now I don't appear to be able to send test emails. I keep getting the error message "Invalid email address my@emailaddress.com". I have the same email that was in there before and it is valid. I've been using this moduel for a few years and don't understand how I can get this error despite the email address being valid!

Any ideas?

Comments

RyanR’s picture

I looked through the Simplenews code and discovered the problem to be the email verification module. For some reason it wouldn't verify my company email even though I've been using it for 3 years...

RyanR’s picture

Status: Active » Closed (works as designed)
garyg’s picture

Just ran into the same issue. Although I have other sites that the upgrade went fine on.
What did you do to solve the problem?

garyg’s picture

Status: Closed (works as designed) » Active

bump to active

garyg’s picture

Status: Active » Closed (works as designed)

Got it!!
remove the email verification module.

rmcom’s picture

Title: "Invalid email address" » "Invalid email address": error in logic and compatibility with email_verify module
Version: 6.x-2.0-alpha2 » 6.x-2.0-alpha3
Status: Closed (works as designed) » Patch (to be ported)
StatusFileSize
new928 bytes

simplenews is mis-interpreting how email_verify works.

email_verify_check returns nothing if everything is ok; it returns a string describing the error when something goes wrong.

The provided patch updates simpenews to correctly interpret the result returned by email verify.

Credit: Mr3T

simon georges’s picture

Version: 6.x-2.0-alpha3 » 6.x-2.x-dev
Status: Patch (to be ported) » Needs review

Status: Needs review » Needs work

The last submitted patch, simplenews-6x20alpha3-email_verify_fix-1291774-6.patch, failed testing.

rmcom’s picture

The patch failed testing because it was made for v6x20alpha3 (not v6x20alpha4 which was the version when the test was done).

I will produce a patch for the current version (v6x20alpha5) and post it here asap for re-test.