Kohana_Jam_Association_Collection

Complexity

Total Complexity

26

Size/Duplication

Total Lines	205
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	7

Test Coverage

Coverage

78.21%

Importance

Changes	1
Bugs	1	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	guess_through_table()	0	10	1
A	initialize()	0	9	2
A	load_fields()	0	15	2
A	assign_internals()	0	7	1
A	get()	0	15	2
A	model_after_check()	0	7	3
A	model_after_save()	0	9	4
A	item_get()	0	4	1
A	item_set()	0	4	1
A	item_unset()	0	4	1
	remove_items_query()	0	1	?
	add_items_query()	0	1	?
	collection()	0	1	?
B	save()	0	20	5
A	clear()	0	11	3

<?php defined('SYSPATH') OR die('No direct script access.');
/**
 * Common association for has-many and many-to-many relationships
 *
 * @package    Jam
 * @category   Associations
 * @author     Ivan Kerin
 * @copyright  (c) 2012 Despark Ltd.
 * @license    http://www.opensource.org/licenses/isc-license.txt
 */
abstract class Kohana_Jam_Association_Collection extends Jam_Association {

	/**
	 * Find the join table based on the two model names pluralized,
	 * sorted alphabetically and with an underscore separating them
	 *
	 * @param  string $model1
	 * @param  string $model2
	 * @return string
	 */
	static public function guess_through_table($model1, $model2)
	{
		$through = array(
			Inflector::plural($model1),
			Inflector::plural($model2)
		);

		sort($through);
		return implode('_', $through);
	}

	/**
	 * Assign default forein_model to singular association name
	 * @param  Jam_Meta $meta
	 * @param  string   $name
	 */
	public function initialize(Jam_Meta $meta, $name)
	{
		if ( ! $this->foreign_model)
		{
			$this->foreign_model = Inflector::singular($name);
		}

		parent::initialize($meta, $name);
	}

	/**
	 * Load associated models (from database or after deserialization)
	 * @param  Jam_Validated $model
	 * @param  mixed         $value
	 * @return Jam_Array_Association
	 */
	public function load_fields(Jam_Validated $model, $value)
	{
		if ($value instanceof Jam_Array_Association)
		{
			$collection = $value;
		}
		else
		{
			$collection = new Jam_Array_Association();
			$collection
				->load_fields($value);
		}

		return $this->assign_internals($model, $collection);
	}

	public function assign_internals(Jam_Validated $model, Jam_Array_Association $collection)
	{
		return $collection
			->model($this->foreign_model)
			->parent($model)
			->association($this);
	}

	public function get(Jam_Validated $model, $value, $is_changed)
	{
		$array = new Jam_Array_Association();
		$array
			->model($this->foreign_model)
			->association($this)
			->parent($model);

		if ($is_changed)
		{
			$array->set($value);
		}

		return $array;
	}

	/**
	 * Perform checks on all changed items from this collection
	 * @param  Jam_Model $model
	 */
	public function model_after_check(Jam_Model $model)
	{
		if ($model->changed($this->name) AND ! $model->{$this->name}->check_changed())
		{
			$model->errors()->add($this->name, 'association_collection');

The method add cannot be called on $model->errors() (of type array|null).

		}
	}

	/**
	 * Persist this collection in the database
	 * @param  Jam_Model $model
	 */
	public function model_after_save(Jam_Model $model)
	{
		if ($model->changed($this->name) AND $collection = $model->{$this->name} AND $collection->changed())
		{
			$collection->save_changed();

			$this->save($model, $collection);
		}
	}

	/**
	 * Use this method to perform special actions when an item is requested from the collection
	 * @param  Jam_Model $model
	 * @param  Jam_Model $item
	 */
	public function item_get(Jam_Model $model, Jam_Model $item)
	{
		// Extend
	}

	/**
	 * Use this method to perform special actions when an item is set to the collection
	 * @param  Jam_Model $model
	 * @param  Jam_Model $item
	 */
	public function item_set(Jam_Model $model, Jam_Model $item)
	{
		// Extend
	}

	/**
	 * Use this method to perform special actions when an item is removed from the collection
	 * @param  Jam_Model $model
	 * @param  Jam_Model $item
	 */
	public function item_unset(Jam_Model $model, Jam_Model $item)

The parameter $model is not used and could be removed.

	{
		// Extend
	}

	/**
	 * A database query used to remove items from the model
	 * @param  array     $ids
	 * @param  Jam_Model $model
	 * @return Database_Query
	 */
	abstract public function remove_items_query(Jam_Model $model, array $ids);

	/**
	 * A database query used to add items to the model
	 * @param  array     $ids
	 * @param  Jam_Model $model
	 * @return Database_Query
	 */
	abstract public function add_items_query(Jam_Model $model, array $ids);

	/**
	 * Get the colleciton to get the models of the association
	 * @param  Jam_Model $model
	 * @return Jam_Query_Builder_Collection
	 */
	abstract public function collection(Jam_Model $model);

	/**
	 * Execute remove_items_query and add_items_query to persist the colleciton to the datbaase
	 * @param  Jam_Model                    $model
	 * @param  Jam_Array_Association $collection
	 */
	public function save(Jam_Model $model, Jam_Array_Association $collection)
	{
		if ($old_ids = array_values(array_diff($collection->original_ids(), $collection->ids())))
		{
			$query = $this->remove_items_query($model, $old_ids);
			if ($query)
			{
				$query->execute(Jam::meta($this->model)->db());
			}
		}

		if ($new_ids = array_values(array_diff($collection->ids(), $collection->original_ids())))
		{
			$query = $this->add_items_query($model, $new_ids);
			if ($query)
			{
				$query->execute(Jam::meta($this->model)->db());
			}
		}
	}

	/**
	 * Use the remove query to remove all items from the collection
	 * @param  Jam_Validated                $model
	 * @param  Jam_Array_Association $collection
	 */
	public function clear(Jam_Validated $model, Jam_Array_Association $collection)
	{
		if ($ids = array_filter($collection->ids()))
		{
			$query = $this->remove_items_query($model, $ids);

$model of type object<Jam_Validated> is not a sub-type of object<Jam_Model>. It seems like you assume a child class of the class Jam_Validated to be always present.

			if ($query)
			{
				$query->execute(Jam::meta($this->model)->db());
			}
		}
	}

}