Adminetic\Announcement\Models\Admin\Announcement

Complexity

Total Complexity

9

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	10	1
A	getColorAttribute()	0	5	1
A	user()	0	3	1
A	getAudienceNamesAttribute()	0	3	1
A	cacheKey()	0	3	2
A	mediums()	0	13	3

<?php

namespace Adminetic\Announcement\Models\Admin;

use App\Models\User;

The type App\Models\User was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\Cache;
use Spatie\Activitylog\Traits\LogsActivity;

The type Spatie\Activitylog\Traits\LogsActivity was not found. Maybe you did not declare it correctly or list all dependencies?

class Announcement extends Model
{
    use LogsActivity;

    protected $guarded = [];

    // Forget cache on updating or saving and deleting
    public static function boot()
    {
        parent::boot();

        static::saving(function () {
            self::cacheKey();
        });

        static::deleting(function () {
            self::cacheKey();
        });
    }

    // Cache Keys
    private static function cacheKey()
    {
        Cache::has('announcements') ? Cache::forget('announcements') : '';
    }

    // Logs
    protected static $logName = 'announcement';

    // Appends
    protected $appends = ['audience_names', 'color'];

    // Casts
    protected $casts = [
        'medium' => 'array',
        'audience' => 'array',
    ];

    // Relation
    public function user()
    {
        return $this->belongsTo(User::class, 'user_id');
    }

    // Accessors
    public function getAudienceNamesAttribute()
    {
        return User::find($this->audience)->pluck('name')->toArray();
    }

    public function getColorAttribute()
    {
        $colors = ['primary', 'secondary', 'success', 'info', 'warning', 'danger', 'pink', 'grey'];

        return $colors[array_rand($colors, 1)];
    }

    public function mediums()
    {
        if (isset($this->medium)) {
            foreach ($this->medium as $val) {
                $mediums[] = [
                    1 => 'database',
                    2 => 'mail',
                ][$val];
            }

            return $mediums;

The variable $mediums seems to be defined by a foreach iteration on line 70. Are you sure the iterator is never empty, otherwise this variable is not defined?

        } else {
            return null;
        }
    }
}