O2System\Email\Protocols\MailProtocol::sending()

Complexity

Conditions	10
Paths	64

Size

Total Lines	49
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php

    protected function sending(Message $message)

    {

        $phpMailer = new PHPMailer();

        $phpMailer->isMail();


        // Set from

        if (false !== ($from = $message->getFrom())) {

The condition false !== $from = $message->getFrom() is always true.

            $phpMailer->setFrom($from->getEmail(), $from->getName());

It seems like $from->getEmail() can also be of type false; however, parameter $address of PHPMailer\PHPMailer\PHPMailer::setFrom() does only seem to accept string, maybe add an additional type check?

It seems like $from->getName() can also be of type false; however, parameter $name of PHPMailer\PHPMailer\PHPMailer::setFrom() does only seem to accept string, maybe add an additional type check?

        }


        // Set recipient

        if (false !== ($to = $message->getTo())) {

            foreach ($to as $address) {

                if ($address instanceof Address) {

                    $phpMailer->addAddress($address->getEmail(), $address->getName());

It seems like $address->getEmail() can also be of type false; however, parameter $address of PHPMailer\PHPMailer\PHPMailer::addAddress() does only seem to accept string, maybe add an additional type check?

It seems like $address->getName() can also be of type false; however, parameter $name of PHPMailer\PHPMailer\PHPMailer::addAddress() does only seem to accept string, maybe add an additional type check?

                }

            }

        }


        // Set reply-to

        if (false !== ($replyTo = $message->getReplyTo())) {

The condition false !== $replyTo = $message->getReplyTo() is always true.

            $phpMailer->addReplyTo($replyTo->getEmail(), $replyTo->getName());

It seems like $replyTo->getEmail() can also be of type false; however, parameter $address of PHPMailer\PHPMailer\PHPMailer::addReplyTo() does only seem to accept string, maybe add an additional type check?

It seems like $replyTo->getName() can also be of type false; however, parameter $name of PHPMailer\PHPMailer\PHPMailer::addReplyTo() does only seem to accept string, maybe add an additional type check?

        }


        // Set content-type

        if ($message->getContentType() === 'html') {

            $phpMailer->isHTML(true);

        }


        // Set subject, body & alt-body

        $phpMailer->Subject = $message->getSubject();

        $phpMailer->Body = $message->getBody();

        $phpMailer->AltBody = $message->getAltBody();


        if (false !== ($attachments = $message->getAttachments())) {

            foreach ($attachments as $filename => $attachment) {

                $phpMailer->addAttachment($attachment, $filename);

            }

        }


        if ( ! $phpMailer->send()) {

            $this->addErrors([

                $phpMailer->ErrorInfo,

            ]);


            return false;

        }


        return true;

    }

}