GJClasses\Smtp

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	send()	0	38	3

<?php
namespace GJClasses;

class Smtp
{
    public function send($recipients, $subject, $message_html, $message_text)
    {
        $mail = new \PHPMailer();

        // $mail->SMTPDebug = 3;
        $mail->isSMTP();
        $mail->isHTML(true);
        $mail->CharSet = EMAIL_ENCODING_TYPE;
        $mail->SMTPSecure = SMTP_PROTOCOL;
        $mail->Host = SMTP_MAIL_SERVER;
        $mail->Port = SMTP_PORT;
        $mail->SMTPAuth = SMTP_AUTHENTICATION_ON;
        $mail->Username = SMTP_USERNAME;
        $mail->Password = SMTP_PASSWORD;
        $mail->setFrom(SMTP_FROM_ADDRESS, SMTP_FROM_NAME);
        $mail->addReplyTo(SMTP_REPLY_ADDRESS, SMTP_REPLY_NAME);

        foreach ($recipients AS $recipient) {

            $mail->Subject = $subject;
            $mail->Body = $message_html;
            $mail->AltBody = $message_text;
            $mail->clearAddresses();
            $mail->addAddress($recipient);

            if ($mail->send()) {

                $result_message = 'Email Send Succeeded';

            } else {

                $result_message = 'Email Send Failed';

            }

        }

        return $result_message;

The variable $result_message seems to be defined by a foreach iteration on line 23. Are you sure the iterator is never empty, otherwise this variable is not defined?

    }
}