Nip\Records\Traits\HasPrimaryKey\RecordsTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Test Coverage

Coverage

55.56%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getPrimaryKey()	0	7	2
A	initPrimaryKey()	0	3	1
A	generatePrimaryKey()	0	12	4
A	setPrimaryKey()	0	3	1

<?php

namespace Nip\Records\Traits\HasPrimaryKey;

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

    /**
     * @return string
     */
    public function getPrimaryKey()
    {
        if ($this->primaryKey === null) {
            $this->initPrimaryKey();
        }

        return $this->primaryKey;
    }

    /**
     * @param null|string $primaryKey
     */
    public function setPrimaryKey($primaryKey)
    {
        $this->primaryKey = $primaryKey;
    }

    protected function initPrimaryKey()
    {
        $this->setPrimaryKey($this->generatePrimaryKey());
    }

    /**
     * @return string
     */
    public function generatePrimaryKey()
    {
        $structure = $this->getTableStructure();

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

        $primaryKey = false;
        if (is_array($structure) && isset($structure['indexes']['PRIMARY']['fields'])) {
            $primaryKey = $structure['indexes']['PRIMARY']['fields'];
            if (count($primaryKey) == 1) {
                $primaryKey = reset($primaryKey);
            }
        }

        return $primaryKey;
    }
}