Sichikawa\LaravelSendgridDriver\SendGrid

Complexity

Total Complexity

10

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	sendgrid()	0	9	3
A	mailDriver()	0	4	2
A	sgEncode()	0	7	2
A	sgDecode()	0	8	3

<?php

namespace Sichikawa\LaravelSendgridDriver;

use Illuminate\Mail\Mailable;
use Sichikawa\LaravelSendgridDriver\Transport\SendgridTransport;
use Swift_Message;

trait SendGrid
{

    /**
     * @param null|array $params
     * @return $this
     */
    public function sendgrid($params)
    {
        if ($this instanceof Mailable && $this->mailDriver() == "sendgrid") {
            $this->withSwiftMessage(function (Swift_Message $message) use ($params) {
                $message->embed(new \Swift_Image(static::sgEncode($params), SendgridTransport::SMTP_API_NAME));

It seems like $params defined by parameter $params on line 16 can also be of type null; however, Sichikawa\LaravelSendgridDriver\SendGrid::sgEncode() does only seem to accept array, maybe add an additional type check?

            });
        }
        return $this;
    }

    /**
     * @return string
     */
    private function mailDriver()

This method is not used, and could be removed.

    {
        return function_exists('config') ? config('mail.driver') : env('MAIL_DRIVER');
    }

    /**
     * @param array $params
     * @return string
     */
    public static function sgEncode($params)
    {
        if (is_string($params)) {
            return $params;
        }
        return json_encode($params);
    }

    /**
     * @param string $strParams
     * @return array
     */
    public static function sgDecode($strParams)
    {
        if (!is_string($strParams)) {
            return (array) $strParams;
        }
        $params = json_decode($strParams, true);
        return is_array($params) ? $params : [];
    }

}