User::getName() - Code Metrics - Inspection of "chore: clean up" - kingscode/laravel-api-starter - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 9f3d75...5160e5 )

by Koen

created 2020-10-20 08:22 UTC

User::getName() A

↳ Parent: User

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	1
c	0
b	0
f	0
nc	1
nop	0
dl	0
loc	3
rs	10
ccs	0
cts	0
cp	0
crap	2

<?php

declare(strict_types=1);

namespace App\Models;

use App\Events\User\Deleting;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

final class User extends Authenticatable
{
    use Notifiable;


    protected $fillable = [
        'name', 'email', 'password',
    ];

    protected $dispatchesEvents = [
        'deleting' => Deleting::class,
    ];

    private ?UserToken $currentToken = null;

    public function tokens(): HasMany
    {
        return $this->hasMany(UserToken::class);
    }

    public function setCurrentToken(UserToken $token)
    {
        $this->currentToken = $token;
    }

    public function getCurrentToken()
    {
        return $this->currentToken;
    }

    public function getEmail(): string
    {
        return $this->getAttributeValue('email');

    }

    public function getName(): string
    {
        return $this->getAttributeValue('name');

    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace App\Models;
6
7		use App\Events\User\Deleting;
8		use Illuminate\Database\Eloquent\Relations\HasMany;
9		use Illuminate\Foundation\Auth\User as Authenticatable;
10		use Illuminate\Notifications\Notifiable;
11
12		final class User extends Authenticatable
13		{
14		use Notifiable;
		0 ignored issues – show Bug introduced 2019-09-19 12:07 UTC by Report Bug Copy Issue Report The trait `Illuminate\Notifications\Notifiable` requires the property `$email` which is not provided by `App\Models\User`. Loading history...
15
16		protected $fillable = [
17		'name', 'email', 'password',
18		];
19
20		protected $dispatchesEvents = [
21		'deleting' => Deleting::class,
22		];
23
24		private ?UserToken $currentToken = null;
25
26	6	public function tokens(): HasMany
27		{
28	6	return $this->hasMany(UserToken::class);
29		}
30
31	2	public function setCurrentToken(UserToken $token)
32		{
33	2	$this->currentToken = $token;
34	2	}
35
36	1	public function getCurrentToken()
37		{
38	1	return $this->currentToken;
39		}
40
41	1	public function getEmail(): string
42		{
43	1	return $this->getAttributeValue('email');
		0 ignored issues – show Bug Best Practice introduced 2020-02-17 16:14 UTC by Report Bug Copy Issue Report The expression `return $this->getAttributeValue('email')` could return the type `boolean\|null` which is incompatible with the type-hinted return `string`. Consider adding an additional type-check to rule them out. Loading history...
44		}
45
46		public function getName(): string
47		{
48		return $this->getAttributeValue('name');
		0 ignored issues – show Bug Best Practice introduced 2020-10-20 08:32 UTC by Report Bug Copy Issue Report The expression `return $this->getAttributeValue('name')` could return the type `boolean\|null` which is incompatible with the type-hinted return `string`. Consider adding an additional type-check to rule them out. Loading history...
49		}
50		}
51

kingscode / laravel-api-starter

Push — master ( 9f3d75...5160e5 )

User::getName() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like