Kodus\Mail\SMTP\SMTPMailService

Complexity

Total Complexity

12

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Test Coverage

Coverage

92.68%

Importance

Changes	6
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	getRecipients()	0	17	4
A	getClient()	0	9	2
A	send()	0	9	1
A	disconnect()	0	6	1
A	getSender()	0	11	2

<?php

namespace Kodus\Mail\SMTP;

use Kodus\Mail\MailService;
use Kodus\Mail\Message;
use Kodus\Mail\MIMEWriter;
use Kodus\Mail\SMTP\Authenticator\NoAuthenticator;

/**
 * This Mail Service implementation delivers Messages directly to an SMTP server.
 *
 * It will block the script while connecting and delivering the Message.
 */
class SMTPMailService implements MailService
{
    /**
     * @var SMTPConnector
     */
    private $connector;

    /**
     * @var SMTPAuthenticator
     */
    protected $authenticator;

    /**
     * @var string
     */
    protected $client_domain;

    /**
     * @var SMTPClient|null
     */
    private $client;

    /**
     * @param SMTPConnector          $connector     provides the SMTP connection
     * @param SMTPAuthenticator|null $authenticator performs SMTP authentication (or NULL to use `NoAuthenticator`)
     * @param string                 $client_domain client domain-name (provided in handshakes when connecting)
     */
    public function __construct(SMTPConnector $connector, ?SMTPAuthenticator $authenticator, string $client_domain)
    {
        $this->connector = $connector;
        $this->authenticator = $authenticator ?: new NoAuthenticator();
        $this->client_domain = $client_domain;
    }

    public function send(Message $message): void
    {
        $this->getClient()->sendMail(
            $this->getSender($message),
            $this->getRecipients($message),
            function ($socket) use ($message) {
                $writer = new MIMEWriter($socket);

                $writer->writeMessage($message);
            }
        );
    }

    /**
     * Internally disconnect the SMTP Client.
     *
     * Long-running services may wish to disconnect the SMTP client after sending a batch
     * of Messages, to avoid timeouts.
     */
    public function disconnect(): void
    {
        // NOTE: this will cause the SMTP Client instance will fall out of scope, which
        //       will trigger it's destructor, which will send QUIT and close the socket.

        $this->client = null;
    }

    /**
     * Connect and authenticate SMTP Client (if not already connected)
     *
     * @return SMTPClient
     */
    protected function getClient(): SMTPClient
    {
        if (! isset($this->client)) {
            $this->client = $this->connector->connect($this->client_domain);

            $this->authenticator->authenticate($this->client);
        }

        return $this->client;

The expression return $this->client could return the type null which is incompatible with the type-hinted return Kodus\Mail\SMTP\SMTPClient. Consider adding an additional type-check to rule them out.

    }

    /**
     * Determine the sender e-mail address from the "Sender" or first "From" field of the Message
     *
     * @param Message $message
     *
     * @return string sender e-mail address
     */
    private function getSender(Message $message): string
    {
        $sender = $message->getSender();

        if ($sender) {
            return $sender->getEmail();
        }

        $from = $message->getFrom();

        return $from[0]->getEmail();
    }

    /**
     * Extract recipient e-mail addresses from the "To", "CC" and "BCC" fields of the Message
     *
     * @param Message $message
     *
     * @return string[] list of recipient e-mail addresses
     */
    private function getRecipients(Message $message): array
    {
        $recipients = [];

        foreach ($message->getTo() as $recipient) {
            $recipients[] = $recipient->getEmail();
        }

        foreach ($message->getCC() as $recipient) {
            $recipients[] = $recipient->getEmail();
        }

        foreach ($message->getBCC() as $recipient) {
            $recipients[] = $recipient->getEmail();
        }

        return $recipients;
    }
}