Test Failed
Push — ft/states ( eebc9c )
by Ben
29:42 queued 08:49
created

Archivable::isArchived()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 1
c 1
b 0
f 0
dl 0
loc 3
rs 10
cc 1
nc 1
nop 0
1
<?php
2
3
namespace Thinktomorrow\Chief\States\Archivable;
4
5
use Thinktomorrow\Chief\States\PageState;
6
7
trait Archivable
8
{
9
    protected static function bootArchivable()
10
    {
11
        static::addGlobalScope(new ArchiveScope());
12
    }
13
14
    public function isArchived(): bool
15
    {
16
        return $this->state() === PageState::ARCHIVED;
0 ignored issues
show
Bug introduced by
It seems like state() must be provided by classes using this trait. How about adding it as abstract method to this trait? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

16
        return $this->/** @scrutinizer ignore-call */ state() === PageState::ARCHIVED;
Loading history...
17
    }
18
19
    public function scopeArchived($query)
20
    {
21
        $query->withoutGlobalScope(ArchiveScope::class)->where('state', PageState::ARCHIVED);
22
    }
23
24
    public function scopeUnarchived($query)
25
    {
26
        $query->withoutGlobalScope(ArchiveScope::class)->where('state','<>', PageState::ARCHIVED);
27
    }
28
29
    public function scopeWithArchived($query)
30
    {
31
        $query->withoutGlobalScope(ArchiveScope::class);
32
    }
33
}
34