terabytesoft\helpers\Mailer::sendMessage()

Complexity

Conditions	6
Paths	32

Size

Total Lines	37
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	6

Importance

Changes

0

<?php
    public function sendMessage(
        string $to,
        string $subject,
        array $options,
        array $params,
        MailerInterface $mailer
    ): bool {
        if (isset(\Yii::$app->params['helper.mailer.viewpath'])) {
            $mailer->viewPath = \Yii::$app->params['helper.mailer.viewpath'];

Accessing viewPath on the interface yii\mail\MailerInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        }

        $views = isset($options['views']) ? $options['views'] : [];

        $this->emailConfig = $mailer
            ->compose($views, $params)
            ->setTo($to)
            ->setFrom(
                [\Yii::$app->params['helper.mailer.sender'] => \Yii::$app->params['helper.mailer.sender.name']]
            )
            ->setSubject($subject);

        if (isset($options['replyTo'])) {
            $this->emailConfig = $this->emailConfig
                ->setReplyTo($options['replyTo']);
        }

        if (isset($options['textBody'])) {
            $this->emailConfig = $this->emailConfig
                ->setTextBody($options['textBody']);
        }

        if (isset($options['textHtml'])) {
            $this->emailConfig = $this->emailConfig
                ->setHtmlBody($options['textHtml']);
        }

        return $mailer->send($this->emailConfig);
    }