terabytesoft\helpers\Mailer::sendMessage()

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	1

Importance

Changes

0

<?php
    public function sendMessage(string $to, string $subject, string $view, array $params = []): bool
    {
        $mailer = \Yii::$app->mailer;

        $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?

        return $mailer->compose(['html' => $view, 'text' => 'text/' . $view], $params)
            ->setTo($to)
            ->setFrom(
                [\Yii::$app->params['helper.mailer.sender'] => \Yii::$app->params['helper.mailer.sender.name']]
            )
            ->setSubject($subject)
            ->send();
    }