PeriodicNotice\Concerns\HasPeriodicNotice

Complexity

Total Complexity

3

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Test Coverage

Coverage

88.89%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	periodicSentEntries()	0	3	1
A	sendPeriodicalNotification()	0	6	1
A	sendNowPeriodicalNotification()	0	3	1

<?php

namespace PeriodicNotice\Concerns;

use Illuminate\Database\Eloquent\Relations\MorphMany;
use PeriodicNotice\Models\PeriodicSentEntry;

/**
 * Trait related to notification receiver.
 *
 * @mixin \PeriodicNotice\Contracts\NotificationReceiver
 * @mixin \Illuminate\Database\Eloquent\Model
 */
trait HasPeriodicNotice
{
    public function periodicSentEntries(): MorphMany
    {
        return $this->morphMany(PeriodicSentEntry::class, 'receiver');

It seems like morphMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function sendNowPeriodicalNotification(string $group = 'default')
    {
        $this->periodicNoticeDirector($group)->sendPeriodicalNotification($this);

It seems like periodicNoticeDirector() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function sendPeriodicalNotification(string $group = 'default')
    {
        dispatch(function () use ($group) {
            $this->sendNowPeriodicalNotification($group);
        })->onConnection(config('periodic-notice.defaults.connection'))
          ->onQueue(config('periodic-notice.defaults.queue'));
    }
}