App\Lib\Model\Table\AppTable

Complexity

Total Complexity

14

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	8

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	toggle()	0	12	2
A	exists()	0	8	2
A	increment()	0	18	4
A	_dispatchEvent()	0	6	1
A	dispatchSaitoEvent()	0	7	2
A	getConfig()	0	11	3

<?php

namespace App\Lib\Model\Table;

use Cake\Core\Configure;
use Cake\Core\InstanceConfigTrait;
use Cake\Database\Expression\QueryExpression;
use Cake\Event\Event;
use Cake\Event\EventManager;
use Cake\ORM\Entity;
use Cake\ORM\Table;
use Saito\Event\SaitoEventManager;

class AppTable extends Table
{
    use InstanceConfigTrait {
        getConfig as private traitGetConfig;
    }

    /** @var array default config for InstanceConfigTrait */
    protected $_defaultConfig = [];

    public $SharedObjects;

    /**
     * @var SaitoEventManager
     */
    protected $_SEM;

    /**
     * Toggle bool field value.
     *
     * @param int $recordId The record-ID.
     * @param string $field The name of the field to toggle.
     * @return int new field The new value after the toggle.
     */
    public function toggle($recordId, $field)
    {
        $entity = $this->query()
            ->select(['id', $field])
            ->where(['id' => $recordId])
            ->first();
        $new = ($entity->get($field) == 0) ? 1 : 0;
        $this->patchEntity($entity, [$field => $new]);

It seems like $entity defined by $this->query()->select(array('id', $field))->where(array('id' => $recordId))->first() on line 39 can also be of type array or null; however, Cake\ORM\Table::patchEntity() does only seem to accept object<Cake\Datasource\EntityInterface>, maybe add an additional type check?

        $this->save($entity);

It seems like $entity defined by $this->query()->select(array('id', $field))->where(array('id' => $recordId))->first() on line 39 can also be of type array or null; however, Cake\ORM\Table::save() does only seem to accept object<Cake\Datasource\EntityInterface>, maybe add an additional type check?

        return $new;
    }

    /**
     * Check that a record exists.
     *
     * @param int|array|\ArrayAccess $conditions Record-ID or query conditions.
     * @return bool
     */
    public function exists($conditions)
    {
        if (is_int($conditions)) {
            $conditions = ['id' => $conditions];
        }

        return parent::exists($conditions);
    }

    /**
     * Increments value of a field
     *
     * @param int|array $where entry-id or array with conditions
     * @param string $field fielt to increment
     * @param int $amount Increment size.
     * @return void
     * @throws \InvalidArgumentException
     */
    public function increment($where, $field, $amount = 1)
    {
        if (!is_int($amount)) {
            throw new \InvalidArgumentException;
        }

        if (is_numeric($where)) {
            $where = ['id' => $where];
        }

        $operator = '+';
        if ($amount < 0) {
            $operator = '-';
            $amount *= -1;
        }
        $expression = new QueryExpression("$field = $field $operator $amount");
        $this->updateAll($expression, $where);

$expression is of type object<Cake\Database\Expression\QueryExpression>, but the function expects a callable.

    }

    /**
     * Dispatches an event
     *
     * - Always passes the issuing model class as subject
     * - Wrapper for CakeEvent boilerplate code
     * - Easier to test
     *
     * @param string $event event identifier `Model.<modelname>.<event>`
     * @param array $data additional event data
     * @return void
     */
    protected function _dispatchEvent($event, $data = [])
    {
        EventManager::instance()->dispatch(new Event($event, $this, $data));
        // propagate event on Saito's event bus
        $this->dispatchSaitoEvent($event, $data);
    }

    /**
     * Dispatch Saito Event
     *
     * @param string $event event
     * @param array $data data
     * @return void
     */
    public function dispatchSaitoEvent($event, $data)
    {
        if (!$this->_SEM) {
            $this->_SEM = SaitoEventManager::getInstance();
        }
        $this->_SEM->dispatch($event, $data + ['Model' => $this]);
    }

    /**
     * {@inheritdoc}
     */
    public function getConfig($key = null, $default = null)
    {
        if (is_string($key)) {
            $setting = Configure::read('Saito.Settings.' . $key);
            if ($setting !== null) {
                return $setting;
            }
        }

        return $this->traitGetConfig($key, $default);
    }
}