App\User

Complexity

Total Complexity

9

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	adminlte_image()	0	7	4
A	adminlte_desc()	0	8	2
A	fullname()	0	3	1
A	adminlte_profile_url()	0	3	1
A	sendEmailVerificationNotification()	0	3	1

<?php

namespace App;

use App\Src\UseCases\Domain\Ports\OrganizationRepository;
use Illuminate\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;
use Illuminate\Support\Facades\Mail;
use Spatie\Permission\Traits\HasRoles;

class User extends Authenticatable implements \Illuminate\Contracts\Auth\MustVerifyEmail
{
    use Notifiable, HasRoles, MustVerifyEmail;

The trait Spatie\Permission\Traits\HasRoles requires some properties which are not provided by App\User: $name, $map, $permissions, $roles, $guard_name

The trait Illuminate\Auth\MustVerifyEmail requires some properties which are not provided by App\User: $email_verified_at, $email

The trait Illuminate\Notifications\Notifiable requires the property $email which is not provided by App\User.

    protected $fillable = [
        'firstname', 'lastname', 'email', 'password', 'uuid', 'organization_id', "path_picture", "providers"
    ];

    protected $hidden = [
        'password', 'remember_token',
    ];

    protected $casts = [
        'email_verified_at' => 'datetime',
        'providers' => 'json',
        'wiki_stats' => 'json',
    ];

    public function adminlte_image()
    {
        $urlPicture = $this->path_picture != "" ? asset('storage/'.str_replace('app/public/', '', $this->path_picture)) : null;

The property path_picture does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

        if(!isset($urlPicture) || $urlPicture === ""){
            $urlPicture = url('').'/'.config('adminlte.logo_img');
        }
        return $urlPicture;
    }

    public function adminlte_desc()
    {
        $desc = ucfirst($this->firstname).' '.ucfirst($this->lastname);

The property lastname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

It seems like $this->firstname can also be of type Illuminate\Database\Eloquent\Builder&Illuminate\Database\Eloquent\Relations\Relation and Illuminate\Database\Eloquent\Relations\Relation; however, parameter $str of ucfirst() does only seem to accept string, maybe add an additional type check?

The property firstname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

        if($this->organization_id !== null){

The property organization_id does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

            $organization = app(OrganizationRepository::class)->get($this->organization_id);
            $desc .= ' - organisme : '.$organization->name();
        }
        return $desc;
    }

    public function adminlte_profile_url()
    {
        return 'user/edit/profile';
    }

    public function fullname()
    {
        return ucfirst($this->firstname).' '.ucfirst($this->lastname);

The property firstname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

The property lastname does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

It seems like $this->firstname can also be of type Illuminate\Database\Eloquent\Builder&Illuminate\Database\Eloquent\Relations\Relation and Illuminate\Database\Eloquent\Relations\Relation; however, parameter $str of ucfirst() does only seem to accept string, maybe add an additional type check?

    }

    public function sendEmailVerificationNotification()
    {
        Mail::to($this->email)->send(new \App\Mail\Auth\VerifyEmail($this));

The property email does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

    }
}