Omnimail\SendinBlue

Complexity

Total Complexity

39

Size/Duplication

Total Lines	146
Duplicated Lines	39.73 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	4
Bugs	1	Features	3

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
C	send()	10	41	11
C	mapAttachments()	24	24	10
C	mapInlineImages()	24	24	10
B	mapEmails()	0	11	5
A	mapEmail()	0	10	2

<?php

namespace Omnimail;

use Omnimail\Exception\Exception;
use Omnimail\Exception\InvalidRequestException;
use Psr\Log\LoggerInterface;
use Sendinblue\Mailin;

class SendinBlue implements EmailSenderInterface
{
    private $accessKey;
    private $logger;

    /**
     * @param string $accessKey
     * @param LoggerInterface|null $logger
     */
    public function __construct($accessKey, LoggerInterface $logger = null)
    {
        $this->accessKey = $accessKey;
        $this->logger = $logger;
    }

    public function send(EmailInterface $email)
    {
        $mailin = new Mailin('https://api.sendinblue.com/v2.0', $this->accessKey);

        $data = [
            'to' => $this->mapEmails($email->getTos()),
            'cc' => $this->mapEmails($email->getCcs()),
            'bcc' => $this->mapEmails($email->getBccs()),
            'from' => $this->mapEmail($email->getFrom()),
            'replyto' => $this->mapEmails($email->getReplyTos()),
            'subject' => $email->getSubject(),
            'text' => $email->getTextBody(),
            'html' => $email->getHtmlBody(),
            'attachment' => $this->mapAttachments($email->getAttachments()),
            'inline_image' => $this->mapInlineImages($email->getAttachments())
        ];

        $response = $mailin->send_email($data);
        if ($response && $response['code'] && $response['code'] === 'success') {
            if ($this->logger) {
                $this->logger->info("Email sent: '{$email->getSubject()}'", $email);

$email is of type object<Omnimail\EmailInterface>, but the function expects a array.

            }
        } else {
            if (!$response || !$response['code']) {
                throw new Exception('Unknown exception');
            } else {
                switch ($response['code']) {
                    case 'failure':

This code seems to be duplicated across your project.

                        if ($this->logger) {
                            $this->logger->info("Email error: '{$response['message']}'", $email);

$email is of type object<Omnimail\EmailInterface>, but the function expects a array.

                        }
                        throw new InvalidRequestException($response['message']);
                    case 'error':

This code seems to be duplicated across your project.

                        if ($this->logger) {
                            $this->logger->info("Email error: '{$response['message']}'", $email);

$email is of type object<Omnimail\EmailInterface>, but the function expects a array.

                        }
                        throw new InvalidRequestException($response['message']);
                }
            }
        }
    }

    /**
     * @param array|null $attachments
     * @return array|null
     */
    private function mapAttachments(array $attachments = null)

This method seems to be duplicated in your project.

    {
        if (null === $attachments || !is_array($attachments) || !count($attachments)) {
            return null;
        }
        $finalAttachments = [];
        /** @var AttachmentInterface $attachment */
        foreach ($attachments as $attachment) {
            if ($attachment->getContentId()) {
                continue;
            }
            
            $content = null;
            if (!$attachment->getPath() && $attachment->getContent()) {
                $content = base64_encode($attachment->getContent());
            } elseif ($attachment->getPath()) {
                $content = base64_encode(file_get_contents($attachment->getPath()));
            }
            if ($content) {

The expression $content of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $finalAttachments[$attachment->getName()] = $content;
            }
        }
        return $finalAttachments;
    }

    /**
     * @param array|null $attachments
     * @return array|null
     */
    private function mapInlineImages(array $attachments = null)

This method seems to be duplicated in your project.

    {
        if (null === $attachments || !is_array($attachments) || !count($attachments)) {
            return null;
        }
        $finalAttachments = [];
        /** @var AttachmentInterface $attachment */
        foreach ($attachments as $attachment) {
            if (!$attachment->getContentId()) {
                continue;
            }

            $content = null;
            if (!$attachment->getPath() && $attachment->getContent()) {
                $content = base64_encode($attachment->getContent());
            } elseif ($attachment->getPath()) {
                $content = base64_encode(file_get_contents($attachment->getPath()));
            }
            if ($content) {

The expression $content of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $finalAttachments[$attachment->getContentId()] = $content;
            }
        }
        return $finalAttachments;
    }

    /**
     * @param array|null $emails
     * @return array|null
     */
    private function mapEmails(array $emails = null)
    {
        if (null === $emails || !is_array($emails) || !count($emails)) {
            return null;
        }
        $finalEmails = [];
        foreach ($emails as $email) {
            $finalEmails = array_merge($finalEmails, $this->mapEmail($email));
        }
        return $finalEmails;
    }

    /**
     * @param array $email
     * @return array
     */
    private function mapEmail(array $email)
    {
        $finalEmail = [];
        if ($email['name']) {
            $finalEmail[$email['email']] = $email['name'];
        } else {
            $finalEmail[$email['email']] = '';
        }
        return $finalEmail;
    }
}