Nip\Records\Relations\Traits\HasPivotTable

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Test Coverage

Coverage

76.47%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addPivotParams()	0	4	1
A	generatePivotTable()	0	9	1
A	generateTable()	0	3	1
A	hydrateQueryWithPivotConstraints()	0	5	1
A	getDB()	0	3	2

<?php

namespace Nip\Records\Relations\Traits;

use Nip\Database\Connections\Connection;
use Nip\Database\Query\Select as SelectQuery;

/**
 * Trait HasPivotTable
 * @package Nip\Records\Relations\Traits
 */
trait HasPivotTable
{
    use HasPivotForeignKey;
    use HasPivotPrimaryKey;

    /**
     * @param $params
     */
    public function addPivotParams($params)
    {
        $this->checkParamPivotFk($params);
        $this->checkParamPivotPrimaryKey($params);
    }

    /**
     * @return Connection
     */
    public function getDB()
    {
        return $this->getParam("link-db") == 'with' ? $this->getWith()->getDB() : parent::getDB();

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

    }

    /** @noinspection PhpMissingParentCallCommonInspection
     * @return string
     */
    public function generateTable()
    {
        return $this->generatePivotTable();
    }

    /**
     * Builds the name of a has-and-belongs-to-many association table
     * @return string
     */
    public function generatePivotTable()
    {
        $tables = [
            $this->getManager()->getTable(),

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

            $this->getWith()->getTable()
        ];
        sort($tables);

        return implode("_", $tables);
    }

    /**
     * @param SelectQuery $query
     */
    protected function hydrateQueryWithPivotConstraints($query)
    {
        $pk = $this->getPivotPrimaryKey();
        $fk = $this->getPivotFK();
        $query->where("`{$this->getTable()}`.`$fk` = `{$this->getWith()->getTable()}`.`$pk`");

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

    }
}