App\Models\Concerns\User\Attribute

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

37.5%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getRoleAttribute()	0	5	1
A	getProfileImageAttribute()	0	3	1
A	setPasswordAttribute()	0	5	1

<?php

namespace App\Models\Concerns\User;

use Illuminate\Support\Facades\Hash;

/**
 * @property string $username
 * @property string $name
 * @property string $email
 * @property \Illuminate\Support\Carbon $email_verified_at
 * @property string $password
 * @property-read string $remember_token
 * @property-read string $role
 * @property-read string $profile_image
 *
 * @see \App\Models\User
 */
trait Attribute
{
    /**
     * Set "password" attribute value.
     *
     * @param  mixed  $value
     * @return void
     */
    public function setPasswordAttribute($value)
    {
        $this->attributes['password'] = Hash::make($value);

The property attributes does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this;

The expression return $this returns the type App\Models\Concerns\User\Attribute which is incompatible with the documented return type void.

    }

    /**
     * Return "role" attribute value.
     *
     * @return string
     */
    public function getRoleAttribute(): string
    {
        $this->load('roles:id,name');

It seems like load() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->roles->first()->name;

The property roles does not exist on App\Models\Concerns\User\Attribute. Did you mean role?

    }

    /**
     * Return "profile_image" attribute value.
     *
     * @return string
     */
    public function getProfileImageAttribute(): string
    {
        return gravatar_image($this->email);
    }
}