App\Models\Concerns\User\Relation

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Test Coverage

Coverage

71.43%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getCompanyRelationValue()	0	3	1
A	company()	0	3	1
A	setCompanyRelationValue()	0	5	1

<?php

namespace App\Models\Concerns\User;

use App\Models\Company;
use Illuminate\Database\Eloquent\Relations\BelongsTo;

/**
 * @property int $company_id Foreign key of \App\Models\Company.
 * @property-read \App\Models\Company $company
 *
 * @see \App\Models\User
 */
trait Relation
{
    /**
     * Define an inverse one-to-one or many relationship with \App\Models\Company.
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    public function company(): BelongsTo
    {
        return $this->belongsTo(Company::class);

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

    }

    /**
     * Return \App\Models\Company model relation value.
     *
     * @return \App\Models\Company
     */
    public function getCompanyRelationValue(): Company
    {
        return $this->getRelationValue('company');

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

    }

    /**
     * Set \App\Models\Company model relation value.
     *
     * @param  \App\Models\Company  $company
     * @return $this
     */
    public function setCompanyRelationValue(Company $company)
    {
        $this->company()->associate($company);

        return $this;
    }
}