miBadger\ActiveRecord\Traits\ManyToManyRelation

Complexity

Total Complexity

3

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Test Coverage

Coverage

91.67%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	createTableConstraints()	0	12	1
A	initManyToManyRelation()	0	25	2

<?php

namespace miBadger\ActiveRecord\Traits;

use miBadger\Query\Query;
use miBadger\ActiveRecord\ColumnProperty;
use miBadger\ActiveRecord\AbstractActiveRecord;

Trait ManyToManyRelation
{
	// These variables are relevant for internal bookkeeping (constraint generation etc)

	/** @var string The name of the left column of the relation. */
	private $_leftColumnName;

	/** @var string The name of the right column of the relation. */
	private $_rightColumnName;

	/** @var string The name of the left table of the relation. */
	private $_leftEntityTable;

	/** @var string The name of the right table of the relation. */
	private $_rightEntityTable;

	/** @var \PDO The PDO object. */
	protected $pdo;
	/**
	 * Initializes the the ManyToManyRelation trait on the included object
	 * 
	 * @param AbstractActiveRecord $leftEntity The left entity of the relation
	 * @param int $leftVariable The reference to the variable where the id for the left entity will be stored
	 * @param AbstractActiveRecord $rightEntity The left entity of the relation
	 * @param int $leftVariable The reference to the variable where the id for the right entity will be stored
	 * @return void
	 */
	protected function initManyToManyRelation(AbstractActiveRecord $leftEntity, &$leftVariable, AbstractActiveRecord $rightEntity, &$rightVariable)
	{
		$this->_leftEntityTable = $leftEntity->getActiveRecordTable();
		$this->_rightEntityTable = $rightEntity->getActiveRecordTable();

		if (get_class($leftEntity) === get_class($rightEntity)) {
			$this->_leftColumnName = sprintf("id_%s_left", $leftEntity->getActiveRecordTable());
			$this->_rightColumnName = sprintf("id_%s_right", $rightEntity->getActiveRecordTable());
		} else {
			$this->_leftColumnName = sprintf("id_%s", $leftEntity->getActiveRecordTable());
			$this->_rightColumnName = sprintf("id_%s", $rightEntity->getActiveRecordTable());
		}

		$this->extendTableDefinition($this->_leftColumnName, [
			'value' => &$leftVariable,
			'validate' => null,
			'type' => AbstractActiveRecord::COLUMN_TYPE_ID,
			'properties' => ColumnProperty::NOT_NULL
		]);

		$this->extendTableDefinition($this->_rightColumnName, [
			'value' => &$rightVariable,
			'validate' => null,
			'type' => AbstractActiveRecord::COLUMN_TYPE_ID,
			'properties' => ColumnProperty::NOT_NULL
		]);
	}

	/**
	 * Build the constraints for the many-to-many relation table
	 * @return void
	 */
	public function createTableConstraints()
	{
		$childTable = $this->getActiveRecordTable();

Are you sure the assignment to $childTable is correct as $this->getActiveRecordTable() targeting miBadger\ActiveRecord\Traits\ManyToManyRelation::getActiveRecordTable() seems to always return null.

		$leftParentTable = $this->_leftEntityTable;
		$rightParentTable = $this->_rightEntityTable;

		$leftConstraint = $this->buildConstraint($leftParentTable, 'id', $childTable, $this->_leftColumnName);

Are you sure the assignment to $leftConstraint is correct as $this->buildConstraint($leftParentTable, 'id', $childTable, $this->_leftColumnName) targeting miBadger\ActiveRecord\Traits\ManyToManyRelation::buildConstraint() seems to always return null.

		$rightConstraint = $this->buildConstraint($rightParentTable, 'id', $childTable, $this->_rightColumnName);

Are you sure the assignment to $rightConstraint is correct as $this->buildConstraint($rightParentTable, 'id', $childTable, $this->_rightColumnName) targeting miBadger\ActiveRecord\Traits\ManyToManyRelation::buildConstraint() seems to always return null.

		$this->pdo->query($leftConstraint);

$leftConstraint of type void is incompatible with the type string expected by parameter $statement of PDO::query().

		$this->pdo->query($rightConstraint);
	}

	/**
	 * @return void
	 */	
	abstract function getActiveRecordTable();

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function buildConstraint($parentTable, $parentColumn, $childTable, $childColumn);

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function extendTableDefinition($columnName, $definition);

It is generally recommended to explicitly declare the visibility for methods.

	
	/**
	 * @return void
	 */
	abstract function registerSearchHook($columnName, $fn);

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function registerDeleteHook($columnName, $fn);

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function registerUpdateHook($columnName, $fn);

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function registerReadHook($columnName, $fn);

It is generally recommended to explicitly declare the visibility for methods.

	/**
	 * @return void
	 */
	abstract function registerCreateHook($columnName, $fn);

It is generally recommended to explicitly declare the visibility for methods.

}