ByJG\Mail\Wrapper\PHPMailerWrapper

Complexity

Total Complexity

13

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

98.11%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getMailer()	0	5	1
A	send()	0	21	3
C	prepareMailer()	0	66	9

<?php

namespace ByJG\Mail\Wrapper;

use ByJG\Convert\FromUTF8;
use ByJG\Mail\Envelope;
use ByJG\Mail\Exception\MailApiException;
use ByJG\Mail\Override\PHPMailerOverride;
use ByJG\Mail\Util;

class PHPMailerWrapper extends BaseWrapper
{
    /**
     * @return \ByJG\Mail\Override\PHPMailerOverride
     */
    public function getMailer()
    {
        // the true param means it will throw exceptions on errors, which we need to catch
        return new PHPMailerOverride(true);
    }

    /**
     * @param Envelope $envelope
     * @return PHPMailerOverride
     * @throws \phpmailerException
     */
    protected function prepareMailer(Envelope $envelope)
    {
        $mail = $this->getMailer();
        $mail->Subject = FromUTF8::toIso88591Email($envelope->getSubject());
        $mail->CharSet = "utf-8";
        $mail->Body = $envelope->getBody();
        if ($envelope->isHtml()) {
            $mail->msgHTML($envelope->getBody());
            $mail->AltBody = $envelope->getBodyText();
        }

        $mail->isSMTP(); // telling the class to use SMTP

        if ($this->uri->getScheme() != "smtp") {
            $mail->SMTPSecure = $this->uri->getScheme(); // ssl ou tls!
        }

        $replyTo = Util::decomposeEmail($envelope->getReplyTo());
        $mail->addReplyTo($replyTo["email"], $replyTo["name"]);

        // Define From email
        $from = Util::decomposeEmail($envelope->getFrom());
        $mail->setFrom($from["email"], $from["name"]);

        // Add Recipients
        foreach ((array)$envelope->getTo() as $toItem) {
            $to = Util::decomposeEmail($toItem);
            $mail->addAddress($to["email"], $to["name"]);
        }

        // Add Carbon Copy
        foreach ((array)$envelope->getCC() as $ccItem) {
            $cc = Util::decomposeEmail($ccItem);
            $mail->addCC($cc["email"], $cc["name"]);
        }

        // Add Blind Carbon Copy
        foreach ((array)$envelope->getBCC() as $bccItem) {
            $bcc = Util::decomposeEmail($bccItem);
            $mail->addCustomHeader("Bcc: " . $bcc["email"]);
        }

        // Attachments
        foreach ((array)$envelope->getAttachments() as $name => $value) {
            switch ($value['disposition']) {
                case 'attachment':
                    $mail->addAttachment(
                        $value['content'],
                        $name,
                        'base64',
                        $value['content-type'],
                        'attachment'
                    );
                    break;

                case 'inline':
                    $mail->addEmbeddedImage($value['content'], $name, $name, 'base64', $value['content-type']);
                    break;

                default:
                    throw new \InvalidArgumentException('Invalid attachment type');
            }
        }

        return $mail;
    }

    /**
     * @param Envelope $envelope
     * @return bool
     * @throws \ByJG\Mail\Exception\MailApiException
     * @throws \Exception
     * @throws \phpmailerException
     */
    public function send(Envelope $envelope)
    {
        $this->validate($envelope);

        $mail = $this->prepareMailer($envelope);

        $mail->Host = $this->uri->getHost();
        $mail->Port = $this->uri->getPort();

It seems like $this->uri->getPort() can also be of type string. However, the property $Port is declared as type integer. Maybe add an additional type check?

        if (!empty($this->uri->getUsername())) {
            $mail->SMTPAuth = true;
            $mail->Username = $this->uri->getUsername(); // SMTP account username
            $mail->Password = $this->uri->getPassword();        // SMTP account password
        }

        if (!$mail->send()) {
            throw new MailApiException($mail->ErrorInfo);
        }

        return true;
    }
}