Nip\Records\Relations\Traits\HasPivotForeignKey

Complexity

Total Complexity

7

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

70.59%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	checkParamPivotFk()	0	7	2
A	getPivotFK()	0	8	2
A	setPivotFK()	0	4	1
A	initPivotFK()	0	4	1
A	generatePivotFK()	0	4	1

<?php

namespace Nip\Records\Relations\Traits;

use Nip\Records\AbstractModels\RecordManager;

/**
 * Trait HasPivotForeignKey
 * @package Nip\Records\Relations\Traits
 *
 * @method RecordManager getWith
 */
trait HasPivotForeignKey
{
    /**
     * @var null|string
     */
    protected $pivotForeignKey = null;

    /**
     * @param $params
     */
    public function checkParamPivotFk($params)
    {
        if (isset($params['pivotFK'])) {
            $this->setPivotFK($params['pivotFK']);
            unset($params['pivotFK']);
        }
    }


    /**
     * @return string
     */
    public function getPivotFK()
    {
        if ($this->pivotForeignKey == null) {
            $this->initPivotFK();
        }

        return $this->pivotForeignKey;
    }

    /**
     * @param $name
     */
    public function setPivotFK($name)
    {
        $this->pivotForeignKey = $name;
    }

    protected function initPivotFK()
    {
        $this->setPivotFK($this->generatePivotFK());
    }

    /**
     * @return string
     */
    protected function generatePivotFK()
    {
        return $this->getWith()->getPrimaryFK();

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

    }
}