terabytesoft\mailer\user\Mailer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	sendMessage()	0	13	1

<?php

namespace terabytesoft\helpers;

use yii\base\Component;
use yii\db\ActiveRecord;

/**
 * Class Mailer
 *
 **/
class Mailer extends Component
{
    /**
     * sendMessage
     **/
    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();
    }
}