Passed
Pull Request — master (#4)
by steef
10:11
created

ManagesPriorities   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 19
Duplicated Lines 0 %

Test Coverage

Coverage 100%

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 6
c 1
b 0
f 0
dl 0
loc 19
ccs 5
cts 5
cp 1
rs 10
wmc 1

1 Method

Rating   Name   Duplication   Size   Complexity  
A priorities() 0 7 1
1
<?php
2
3
namespace TestMonitor\DoneDone\Actions;
4
5
use TestMonitor\DoneDone\Transforms\TransformsPriorities;
6
7
trait ManagesPriorities
8
{
9
    use TransformsPriorities;
10
11
    /**
12
     * Get a list of of priorities.
13
     *
14
     * @param int $accountId
15
     * @param int $projectId
16
     *
17
     * @return \TestMonitor\DoneDone\Resources\Priority[]
18
     */
19 5
    public function priorities(int $accountId, int $projectId): array
20
    {
21 5
        $result = $this->get("{$accountId}/internal-projects/{$projectId}/priorities");
0 ignored issues
show
Bug introduced by
It seems like get() 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

21
        /** @scrutinizer ignore-call */ 
22
        $result = $this->get("{$accountId}/internal-projects/{$projectId}/priorities");
Loading history...
22
23 1
        return array_map(function ($priority) {
24 1
            return $this->fromDoneDonePriority($priority);
25 1
        }, $result);
26
    }
27
}
28