App\Models\enso\Avatars\Avatar

Complexity

Total Complexity

2

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	user()	0	3	1
A	store()	0	7	1

<?php

namespace App\Models\enso\Avatars;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\UploadedFile;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;
use App\Models\User;
use App\Contracts\enso\files\Attachable;
use App\Traits\enso\files\HasFile;
use LaravelEnso\Helpers\App\Traits\CascadesMorphMap;

/**
 * @property int $id
 * @property int $user_id
 * @property string $created_at
 * @property string $updated_at
 * @property User $user
 */
class Avatar extends Model implements Attachable
{
    use CascadesMorphMap, HasFile;

The trait App\Traits\enso\files\HasFile requires some properties which are not provided by App\Models\enso\Avatars\Avatar: $saved_name, $extensions, $file

    /**
     * The table associated with the model.
     * 
     * @var string
     */
    protected $table = 'avatars';

    /**
     * @var array
     */

    public const Width = 250;
    public const Height = 250;

    protected $fillable = ['user_id', 'original_name', 'saved_name'];

    protected $hidden = ['user_id', 'created_at', 'updated_at'];

    protected $casts = ['user_id' => 'int'];

    protected $optimizeImages = true;

    protected $resizeImages = [
        'width' => self::Width,
        'height' => self::Height,
    ];

    protected $mimeTypes = ['image/png', 'image/jpg', 'image/jpeg', 'image/gif'];

    protected $folder = 'avatars';

    public function user()
    {
        return $this->belongsTo(User::class);
    }

    public function store(UploadedFile $file)
    {
        return DB::transaction(function () use ($file) {
            $this->delete();
            $avatar = self::create(['user_id' => Auth::user()->id]);

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            return tap($avatar)->upload($file);
        });
    }
}