Kohana_Jam_Countcache

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Test Coverage

Coverage

62.5%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	update_counters()	0	18	3
A	increment()	0	4	1
A	decrement()	0	4	1

<?php defined('SYSPATH') OR die('No direct script access.');
/**
 * Handles count cache counting for hasmany
 *
 * @package    Jam
 * @category   Associations
 * @author     Ivan Kerin
 * @copyright  (c) 2012 Despark Ltd.
 * @license    http://www.opensource.org/licenses/isc-license.txt
 */
class Kohana_Jam_Countcache {

	public static function update_counters($model, $ids, array $counters)
	{
		$query = Jam::update($model);

		foreach ($counters as $name => $change)
		{
			$change = (int) $change;
			$operator = $change < 0 ? '-' : '+';

			$query->value($name, DB::expr(strtr('(COALESCE(:name, 0) :operator :value)', array(
				':value' => abs($change),
				':operator' => $operator,
				':name' => Database::instance($query->meta()->db())->quote_column($name),

It seems like $query->meta()->db() targeting Kohana_Jam_Meta::db() can also be of type object<Jam_Meta>; however, Kohana_Database::instance() does only seem to accept string|null, maybe add an additional type check?

			))));
		}

		return $query->where(':primary_key', 'IN', (array) $ids);
	}

	public static function increment($model, $counter, $id)
	{
		Jam_Countcache::update_counters($model, $id, array($counter => +1))->execute();
	}

	public static function decrement($model, $counter, $id)
	{
		Jam_Countcache::update_counters($model, $id, array($counter => -1))->execute();
	}
}