Nip\Records\Traits\HasForeignKey\RecordsTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getPrimaryFK()	0	8	2
A	initPrimaryFK()	0	4	1
A	setForeignKey()	0	4	1
A	generatePrimaryFK()	0	6	1
A	setPrimaryFK()	0	4	1

<?php

namespace Nip\Records\Traits\HasForeignKey;

/**
 * Trait RecordsTrait
 * @package Nip\Records\Traits\HasForeignKey
 */
trait RecordsTrait
{
    /**
     * @var null|string
     */
    protected $foreignKey = null;

    /**
     * The name of the field used as a foreign key in other tables
     * @return string
     */
    public function getPrimaryFK()
    {
        if ($this->foreignKey == null) {
            $this->initPrimaryFK();
        }

        return $this->foreignKey;
    }

    public function initPrimaryFK()
    {
        $this->setForeignKey($this->generatePrimaryFK());
    }

    /**
     * @param string $foreignKey
     */
    public function setForeignKey($foreignKey)
    {
        $this->foreignKey = $foreignKey;
    }

    /**
     * @return string
     */
    public function generatePrimaryFK()
    {
        $singularize = inflector()->singularize($this->getController());

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

        return $this->getPrimaryKey() . "_" . inflector()->underscore($singularize);

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

    }

    /**
     * @param $fk
     */
    public function setPrimaryFK($fk)
    {
        $this->foreignKey = $fk;
    }
}