Passed
Pull Request — master (#448)
by
unknown
13:50
created

Identity   A

Complexity

Total Complexity 6

Size/Duplication

Total Lines 41
Duplicated Lines 0 %

Test Coverage

Coverage 83.33%

Importance

Changes 2
Bugs 0 Features 1
Metric Value
eloc 15
c 2
b 0
f 1
dl 0
loc 41
ccs 10
cts 12
cp 0.8333
rs 10
wmc 6

6 Methods

Rating   Name   Duplication   Size   Complexity  
A regenerateCookieLoginKey() 0 3 1
A getCookieLoginKey() 0 3 1
A getUser() 0 3 1
A validateCookieLoginKey() 0 3 1
A __construct() 0 3 1
A getId() 0 3 1
1
<?php
2
3
declare(strict_types=1);
4
5
namespace App\Auth;
6
7
use App\User\User;
8
use Cycle\Annotated\Annotation\Column;
9
use Cycle\Annotated\Annotation\Entity;
10
use Cycle\Annotated\Annotation\Relation\BelongsTo;
11
use Yiisoft\Security\Random;
12
use Yiisoft\User\Login\Cookie\CookieLoginIdentityInterface;
13
14
#[Entity(repository: IdentityRepository::class)]
15
class Identity implements CookieLoginIdentityInterface
16
{
17
    #[Column(type: 'primary')]
18
    private ?int $id = null;
0 ignored issues
show
introduced by
The private property $id is not used, and could be removed.
Loading history...
19
20
    #[Column(type: 'string(32)')]
21
    private string $authKey;
22
23
    #[BelongsTo(target: User::class, nullable: false, load: 'eager')]
24
    private ?User $user = null;
25
    private ?int $user_id = null;
0 ignored issues
show
introduced by
The private property $user_id is not used, and could be removed.
Loading history...
26
27 2
    public function __construct()
28
    {
29 2
        $this->regenerateCookieLoginKey();
30
    }
31
32 1
    public function getId(): ?string
33
    {
34 1
        return $this->user->getId();
0 ignored issues
show
Bug introduced by
The method getId() does not exist on null. ( Ignorable by Annotation )

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

34
        return $this->user->/** @scrutinizer ignore-call */ getId();

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
35
    }
36
37 1
    public function getCookieLoginKey(): string
38
    {
39 1
        return $this->authKey;
40
    }
41
42 1
    public function getUser(): ?User
43
    {
44 1
        return $this->user;
45
    }
46
47
    public function validateCookieLoginKey(string $key): bool
48
    {
49
        return $this->authKey === $key;
50
    }
51
52 3
    public function regenerateCookieLoginKey(): void
53
    {
54 3
        $this->authKey = Random::string(32);
55
    }
56
}
57