miBadger\ActiveRecord\Traits\ManyToManyRelation

Complexity

Total Complexity

3

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

91.67%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	initManyToManyRelation()	0	27	2
A	createTableConstraints()	0	13	1

<?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;

	/**
	 * Initializes the the ManyToManyRelation trait on the included object
	 * 
	 * @param AbstractActiveRecord $leftEntity The left entity of the relation
	 * @param &variable $leftVariable The variable where the id for the left entity will be stored

The doc-type &variable could not be parsed: Unknown type name "&variable" at position 0. (view supported doc-types)

	 * @param AbstractActiveRecord $rightEntity The left entity of the relation
	 * @param &variable $leftVariable The variable where the id for the right entity will be stored

The doc-type &variable could not be parsed: Unknown type name "&variable" at position 0. (view supported doc-types)

	 */
	protected function initManyToManyRelation(AbstractActiveRecord $leftEntity, &$leftVariable, AbstractActiveRecord $rightEntity, &$rightVariable)
	{
		$this->_leftEntityTable = $leftEntity->getActiveRecordTable();

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

		$this->_rightEntityTable = $rightEntity->getActiveRecordTable();

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

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

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

			$this->_rightColumnName = sprintf("id_%s_right", $rightEntity->getActiveRecordTable());

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

		} else {
			$this->_leftColumnName = sprintf("id_%s", $leftEntity->getActiveRecordTable());

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

			$this->_rightColumnName = sprintf("id_%s", $rightEntity->getActiveRecordTable());

The method getActiveRecordTable() cannot be called from this context as it is declared protected in class miBadger\ActiveRecord\AbstractActiveRecord.

		}

		$this->extendTableDefinition($this->_leftColumnName, [

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

			'value' => &$leftVariable,
			'validate' => null,
			'type' => AbstractActiveRecord::COLUMN_TYPE_ID,
			'properties' => ColumnProperty::NOT_NULL
		]);

		$this->extendTableDefinition($this->_rightColumnName, [

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

			'value' => &$rightVariable,
			'validate' => null,
			'type' => AbstractActiveRecord::COLUMN_TYPE_ID,
			'properties' => ColumnProperty::NOT_NULL
		]);
	}

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

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

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

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

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

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

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

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

The property pdo does not exist. Did you maybe forget to declare it?

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