Cms\Model\Javascript

Complexity

Total Complexity

5

Size/Duplication

Total Lines	47
Duplicated Lines	23.4 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getSource()	0	4	1
A	afterUpdate()	0	6	1
A	findCachedById()	11	11	1
A	setText()	0	4	1
A	getText()	0	4	1

<?php
/**
     * @copyright Copyright (c) 2011 - 2014 Aleksandr Torosh (http://wezoom.net)
     * @author Aleksandr Torosh <webtorua@gmail.com>
     */

namespace Cms\Model;

use Phalcon\Mvc\Model;

class Javascript extends Model
{

    public function getSource()
    {
        return "cms_javascript";
    }

    public $id;
    public $text;

    public function afterUpdate()
    {
        $cache = $this->getDi()->get('cache');

The method get cannot be called on $this->getDi() (of type null).

        $key = HOST_HASH . md5("Javascript::getCachedScript({$this->id})");
        $cache->delete($key);
    }

    public static function findCachedById($id)
    {
        $key = HOST_HASH . md5("Javascript::getCachedScript($id)");
        $result = self::findFirst(array("id ='{$id}'",

Are you sure the assignment to $result is correct as self::findFirst(array("id ='{$id}'", 'cache' => array('key' => $key, 'lifetime' => 1200))) (which targets Phalcon\Mvc\Model::findFirst()) seems to always return null.

            'cache' => array(
                'key' => $key,
                'lifetime' => 1200, //20 min
            )
        ));
        return $result;
    }

    /**
     * @param mixed $text
     */
    public function setText($text)
    {
        $this->text = $text;
    }

    /**
     * @param mixed
     */
    public function getText()
    {
        return $this->text;
    }

}