PeriodicNotice\Concerns\InPeriodicNotice

Complexity

Total Complexity

6

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Test Coverage

Coverage

89.47%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	scopeReleasedAfter()	0	3	1
A	notificationEntityDescription()	0	3	1
A	notificationEntityWebUrl()	0	3	1
A	notificationEntityTitle()	0	3	1
A	scopeDoesntSentInPeriodicNotice()	0	8	1
A	periodicSentEntries()	0	3	1

<?php

namespace PeriodicNotice\Concerns;

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

/**
 * Trait related to sendable entity.
 */
trait InPeriodicNotice
{
    public function notificationEntityTitle(): string
    {
        return $this->title ?? '';
    }

    public function notificationEntityWebUrl(): string
    {
        return url($this->slug ?? '');
    }

    public function notificationEntityDescription(): string
    {
        return $this->description ?? '';
    }

    public function periodicSentEntries(): MorphMany
    {
        return $this->morphMany(PeriodicSentEntry::class, 'sendable');

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

    }

    public function scopeDoesntSentInPeriodicNotice(Builder $query, Model $receiver, string $group)
    {
        $query->whereDoesntHave(
            'periodicSentEntries',
            function (Builder $query) use ($receiver, $group) {
                $query->group($group)
                      ->where('receiver_type', '=', $receiver->getMorphClass())
                      ->where('receiver_id', '=', $receiver->getKey());
            }
        );
    }

    public function scopeReleasedAfter(Builder $query, \DateTimeInterface|string $dateTime, string $group)

The parameter $group is not used and could be removed.

    {
        $query->where('created_at', '>=', $dateTime);
    }
}