Transphporm\SheetCache

Complexity

Total Complexity

12

Size/Duplication

Total Lines	52
Duplicated Lines	48.08 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getRulesFromCache()	14	14	4
A	getCacheKey()	11	11	3
A	write()	0	8	3
A	setKey()	0	3	1

<?php
/* @description     Transformation Style Sheets - Revolutionising PHP templating    *
 * @author          Tom Butler tom@r.je                                             *
 * @copyright       2017 Tom Butler <tom@r.je> | https://r.je/                      *
 * @license         http://www.opensource.org/licenses/bsd-license.php  BSD License *
 * @version         1.2                                                             */
namespace Transphporm;
//Separates out TSS file loading/caching from parsing
class SheetCache {
	private $cacheKey;
	private $cacheName;
	private $cache;

	public function __construct(Cache $cache) {
		$this->cache = $cache;
	}

	private function getRulesFromCache($file) {

This method seems to be duplicated in your project.

		//Try to load the cached rules, if not set in the cache (or expired) parse the supplied sheet
		$rules = $this->cache->load($this->cacheName, filemtime($file));

		$this->cacheKey = $this->cacheKey ?? $rules['cacheKey'] ?? null;

		if ($rules) {
			foreach ($rules['import'] as $file) {
				//Check that the import file hasn't been changed since the cache was written
				if (filemtime($file) > $rules['ctime']) return false;
			}
		}
		return $rules;
	}

	public function getCacheKey($tss, $data) {

This method seems to be duplicated in your project.

		//Read the rules so that $this->cacheKey is set
		if (is_file($tss)) $this->getRulesFromCache($tss);
		if ($this->cacheKey) {
			$parser = new Parser\Value($data);
			$parsedKey = $parser->parseTokens($this->cacheKey)[0];
			$this->cacheName = $parsedKey . $this->tss;

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

			return $parsedKey;
		}
		else return '';
	}


	//write the sheet to cache
    public function write($file, $rules, $imports = []) {
		if (is_file($file)) {
			$existing = $this->cache->load($file, filemtime($file));
			if (isset($existing['import'])) $imports = $existing['import'];
			$this->cache->write($this->cacheName, ['rules' => $rules, 'import' => $imports, 'minFreq' => $this->getMinUpdateFreq($rules), 'ctime' => $this->time, 'cacheKey' => $this->cacheKey]);

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

The method getMinUpdateFreq() does not seem to exist on object<Transphporm\SheetCache>.

		}
		return $rules;
    }

    public function setKey($key) {
    	$this->cacheKey = $key;
    }

}