sweelix\sendgrid\Mailer::sendMessage()

Complexity

Conditions	21
Paths	16893

Size

Total Lines	71
Code Lines	50

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	379.5109

Importance

Changes

0

<?php
    public function sendMessage($message)
    {
        try {
            if (($this->token === null) && ($this->user === null && $this->password === null)) {
                throw new InvalidConfigException('Token or login/password are missing');
            }
            $client = null;
            if ($this->token !== null) {
                $client = new SendGridClient($this->token, $this->options);
            } elseif(($this->user !== null) && ($this->password !== null)) {
                $client = new SendGridClient($this->user, $this->password, $this->options);
            }
            if ($client === null) {
                throw new InvalidParamException('Email transport must be configured');
            }
            $sendGridMail = new SendGridMail();
            $replyTo = $message->getReplyTo();
            if ($replyTo !== null) {
                $sendGridMail->setReplyTo($replyTo);
            }
            $sendGridMail->setFrom($message->getFrom());
            if ($message->getFromName() !== null) {
                $sendGridMail->setFromName($message->getFromName());
            }
            foreach($message->getTo() as $email => $name) {

The expression $message->getTo() of type array|null is not guaranteed to be traversable. How about adding an additional type check?

                $sendGridMail->addTo($email, $name);
            }
            foreach($message->getCc() as $email => $name) {
                $sendGridMail->addCc($email, $name);
            }
            foreach($message->getBcc() as $email => $name) {
                $sendGridMail->addBcc($email, $name);
            }
            $sendGridMail->setSubject($message->getSubject());
            foreach($message->getHeaders() as $header) {

The expression $message->getHeaders() of type array|null is not guaranteed to be traversable. How about adding an additional type check?

                list($key, $value) = each($header);
                $sendGridMail->addHeader($key, $value);
            }
            foreach($message->getAttachments() as $attachment) {

The expression $message->getAttachments() of type array|null is not guaranteed to be traversable. How about adding an additional type check?

                $cid = isset($attachment['ContentID']) ? $attachment['ContentID'] : null;
                $sendGridMail->addAttachment($attachment['File'], $attachment['Name'], $cid);
            }

            $templateId = $message->getTemplateId();
            if ($templateId === null) {
                $data = $message->getHtmlBody();
                if ($data !== null) {
                    $sendGridMail->setHtml($data);
                }
                $data = $message->getTextBody();
                if ($data !== null) {
                    $sendGridMail->setText($data);
                }
            } else {
                $sendGridMail->setTemplateId($templateId);
                // trigger html template
                $sendGridMail->setHtml(' ');
                // trigger text template
                $sendGridMail->setText(' ');
                $templateModel = $message->getTemplateModel();
                if (empty($templateModel) === false) {
                    $sendGridMail->setSubstitutions($message->getTemplateModel());
                }
            }
            $result = $client->send($sendGridMail);
            /* @var \SendGrid\Response $result */
            return $result->code == 200;
        } catch (SendGridException $e) {
            throw $e;
        }
    }
}