Passed
Pull Request — develop (#17)
by Atymic
02:24
created

HasQuickBooksToken   A

Complexity

Total Complexity 3

Size/Duplication

Total Lines 19
Duplicated Lines 0 %

Test Coverage

Coverage 33.33%

Importance

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

2 Methods

Rating   Name   Duplication   Size   Complexity  
A quickBooksToken() 0 3 1
A getClient() 0 7 2
1
<?php
2
3
namespace Spinen\QuickBooks;
4
5
trait HasQuickBooksToken
6
{
7
    /**
8
     * Have a quickBooksToken.
9
     *
10
     * @return \Illuminate\Database\Eloquent\Relations\HasOne
11
     */
12 1
    public function quickBooksToken()
13
    {
14 1
        return $this->hasOne(Token::class);
0 ignored issues
show
Bug introduced by
It seems like hasOne() 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

14
        return $this->/** @scrutinizer ignore-call */ hasOne(Token::class);
Loading history...
15
    }
16
17
    public function getClient(): Client
18
    {
19
        if (!$this->quickBooksToken) {
20
            throw new \Exception('Token does not exist for this model');
21
        }
22
23
        return new Client(config('quickbooks'), $this->quickBooksToken);
24
    }
25
}
26