App\Models\Eloquent\Messager\UniversalMessager

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	formatUniversalMessage()	0	5	1
A	sendUniversalMessage()	0	16	3

<?php

namespace App\Models\Eloquent\Messager;

use App\Models\Eloquent\Message;
use Auth;

class UniversalMessager
{
    public static function sendUniversalMessage($config, $official = true)
    {
        $message = new Message;
        $message->sender = $config['sender'];
        $message->receiver = $config['receiver'];
        $message->title = $config['title'];
        $message->level = $config['level'];
        if (isset($config['data']) && isset($config['type'])) {
            $message->type = $config['type'] ?? null;
            $message->data = json_encode($config['data']);
        } else {
            $message->content = $config['content'];
        }
        $message->official = $official;
        $message->save();
        return true;
    }

    public static function formatUniversalMessage($key = null, $replace = [], $locale = null)
    {
        $replace['userName'] = Auth::guard('web')->user()->name;

Accessing name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $replace['siteName'] = config('app.name');
        return __($key, $replace, $locale);
    }
}