Akibatech\Crud\Traits\FieldWithRelation

Complexity

Total Complexity

8

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	withRelation()	0	6	1
A	getRelation()	0	11	2
A	getOptions()	0	12	2
A	beforeValidation()	0	13	1
A	getTableValue()	0	5	1
A	getViewVariables()	0	6	1

<?php

namespace Akibatech\Crud\Traits;

use Akibatech\Crud\Exceptions\InvalidRelationException;
use Illuminate\Database\Eloquent\Relations\Relation;
use Illuminate\Support\Fluent;
use Illuminate\Validation\Validator;

/**
 * Class FieldWithRelation
 *
 * @package Akibatech\Crud\Traits
 */
trait FieldWithRelation
{
    /**
     * @var string $relation
     */
    protected $relation;

    /**
     * @var array
     */
    protected $options;

    /**
     * @param   string $relation
     * @return  self
     */
    public function withRelation($relation)
    {
        $this->relation = $relation;

        return $this;
    }

    /**
     * @param   void
     * @return  Relation
     */
    public function getRelation()
    {
        if (empty($this->relation))
        {
            throw new InvalidRelationException("{$this->getIdentifier()} does not contain any relation.");

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

        }

        $relation = $this->relation;

        return $this->getFields()->getEntry()->getModel()->$relation();

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

    }

    /**
     * {@inheritdoc}
     */
    public function getOptions()
    {
        if (is_null($this->options))
        {
            $related = get_class($this->getRelation()->getRelated());
            $options = $related::get()->pluck('name', 'id')->toArray();
            
            $this->options = $options;
        }

        return $this->options;
    }

    /**
     * @param   Validator $validator
     * @return  Validator
     */
    public function beforeValidation(Validator $validator)
    {
        $related_class = $this->getRelation()->getRelated();
        $related_instance = new $related_class;
        $related_table = $related_class->getTable();
        $related_pk = $related_instance->getKeyName();

        $validator->mergeRules($this->getIdentifier(), [

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

            "exists:$related_table,$related_pk"
        ]);

        return $validator;
    }

    /**
     * {@inheritdoc}
     */
    public function getTableValue()
    {
        $relation = $this->relation;
        return $this->getFields()->getEntry()->getModel()->$relation->name;

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

    }

    /**
     * {@inheritdoc}
     */
    public function getViewVariables()
    {
        return [
            'options' => $this->getOptions(),
        ];
    }
}