diff --git a/lib/Mailer/Methods/PHPMail.php b/lib/Mailer/Methods/PHPMail.php index b3cf01df2a..a8ce7b0af8 100644 --- a/lib/Mailer/Methods/PHPMail.php +++ b/lib/Mailer/Methods/PHPMail.php @@ -28,7 +28,7 @@ class PHPMail { return Mailer::formatMailerSendErrorResult($e->getMessage()); } if($result === 1) { - Mailer::formatMailerSendSuccessResult(); + return Mailer::formatMailerSendSuccessResult(); } else { $result = sprintf(__('%s has returned an unknown error.', 'mailpoet'), Mailer::METHOD_PHPMAIL); $result .= sprintf(' %s: %s', __('Unprocessed subscriber', 'mailpoet'), htmlspecialchars($subscriber)); diff --git a/tests/unit/Mailer/Methods/PHPMailTest.php b/tests/unit/Mailer/Methods/PHPMailTest.php index a7936f178e..93d381daec 100644 --- a/tests/unit/Mailer/Methods/PHPMailTest.php +++ b/tests/unit/Mailer/Methods/PHPMailTest.php @@ -77,7 +77,7 @@ class PHPMailTest extends MailPoetTest { ->equals(array('test@test.com' => 'First Last')); } - function testItCanSend() {return; + function testItCanSend() { if(getenv('WP_TEST_MAILER_ENABLE_SENDING') !== 'true') return; $result = $this->mailer->send( $this->newsletter,