chetcuti / gjclasses

src/GJClasses/MailSmtp.php

<?php
namespace GJClasses;

class MailSmtp
{
    public function send($from_name, $from_address, $reply_name, $reply_address, $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($from_address, $from_name);
        $mail->addReplyTo($reply_address, $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?

    }
}