Thinktomorrow\Chief\States\Archivable\Archivable

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	isArchived()	0	3	1
A	scopeArchived()	0	3	1
A	scopeUnarchived()	0	3	1
A	bootArchivable()	0	3	1
A	scopeWithArchived()	0	3	1

<?php

namespace Thinktomorrow\Chief\States\Archivable;

use Thinktomorrow\Chief\States\PageState;

trait Archivable
{
    protected static function bootArchivable()
    {
        static::addGlobalScope(new ArchiveScope());
    }

    public function isArchived(): bool
    {
        return $this->state() === PageState::ARCHIVED;

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

    }

    public function scopeArchived($query)
    {
        $query->withoutGlobalScope(ArchiveScope::class)->where('state', PageState::ARCHIVED);
    }

    public function scopeUnarchived($query)
    {
        $query->withoutGlobalScope(ArchiveScope::class)->where('state','<>', PageState::ARCHIVED);
    }

    public function scopeWithArchived($query)
    {
        $query->withoutGlobalScope(ArchiveScope::class);
    }
}