| @@ -102,14 +102,14 @@ | ||
| 102 | 102 | } | 
| 103 | 103 | |
| 104 | 104 |  	private function getProperties($tokens) { | 
| 105 | - $rules = $tokens->splitOnToken(Tokenizer::SEMI_COLON); | |
| 105 | + $rules = $tokens->splitOnToken(Tokenizer::SEMI_COLON); | |
| 106 | 106 | |
| 107 | - $return = []; | |
| 108 | -        foreach ($rules as $rule) { | |
| 109 | - $name = $rule->from(Tokenizer::NAME, true)->to(Tokenizer::COLON)->read(); | |
| 110 | - $return[$name] = $rule->from(Tokenizer::COLON)->trim(); | |
| 111 | - } | |
| 107 | + $return = []; | |
| 108 | +		foreach ($rules as $rule) { | |
| 109 | + $name = $rule->from(Tokenizer::NAME, true)->to(Tokenizer::COLON)->read(); | |
| 110 | + $return[$name] = $rule->from(Tokenizer::COLON)->trim(); | |
| 111 | + } | |
| 112 | 112 | |
| 113 | - return $return; | |
| 114 | - } | |
| 113 | + return $return; | |
| 114 | + } | |
| 115 | 115 | } | 
| @@ -44,17 +44,17 @@ | ||
| 44 | 44 | |
| 45 | 45 | |
| 46 | 46 | //write the sheet to cache | 
| 47 | -    public function write($file, $rules, $imports = []) { | |
| 47 | +	public function write($file, $rules, $imports = []) { | |
| 48 | 48 |  		if (is_file($file)) { | 
| 49 | 49 | $existing = $this->cache->load($file, filemtime($file)); | 
| 50 | 50 | if (isset($existing['import'])) $imports = $existing['import']; | 
| 51 | 51 | $this->cache->write($this->cacheName, ['rules' => $rules, 'import' => $imports, 'minFreq' => $this->getMinUpdateFreq($rules), 'ctime' => $this->time, 'cacheKey' => $this->cacheKey]); | 
| 52 | 52 | } | 
| 53 | 53 | return $rules; | 
| 54 | - } | |
| 54 | + } | |
| 55 | 55 | |
| 56 | -    public function setKey($key) { | |
| 57 | - $this->cacheKey = $key; | |
| 58 | - } | |
| 56 | +	public function setKey($key) { | |
| 57 | + $this->cacheKey = $key; | |
| 58 | + } | |
| 59 | 59 | |
| 60 | 60 | } | 
| 61 | 61 | \ No newline at end of file | 
| @@ -36,7 +36,7 @@ | ||
| 36 | 36 |  		if ($this->cacheKey) { | 
| 37 | 37 | $parser = new Parser\Value($data); | 
| 38 | 38 | $parsedKey = $parser->parseTokens($this->cacheKey)[0]; | 
| 39 | - $this->cacheName = $parsedKey . $this->tss; | |
| 39 | + $this->cacheName = $parsedKey.$this->tss; | |
| 40 | 40 | return $parsedKey; | 
| 41 | 41 | } | 
| 42 | 42 | else return ''; | 
| @@ -24,7 +24,9 @@ discard block | ||
| 24 | 24 |  		if ($rules) { | 
| 25 | 25 |  			foreach ($rules['import'] as $file) { | 
| 26 | 26 | //Check that the import file hasn't been changed since the cache was written | 
| 27 | - if (filemtime($file) > $rules['ctime']) return false; | |
| 27 | +				if (filemtime($file) > $rules['ctime']) { | |
| 28 | + return false; | |
| 29 | + } | |
| 28 | 30 | } | 
| 29 | 31 | } | 
| 30 | 32 | return $rules; | 
| @@ -32,14 +34,17 @@ discard block | ||
| 32 | 34 | |
| 33 | 35 |  	public function getCacheKey($tss, $data) { | 
| 34 | 36 | //Read the rules so that $this->cacheKey is set | 
| 35 | - if (is_file($tss)) $this->getRulesFromCache($tss); | |
| 37 | +		if (is_file($tss)) { | |
| 38 | + $this->getRulesFromCache($tss); | |
| 39 | + } | |
| 36 | 40 |  		if ($this->cacheKey) { | 
| 37 | 41 | $parser = new Parser\Value($data); | 
| 38 | 42 | $parsedKey = $parser->parseTokens($this->cacheKey)[0]; | 
| 39 | 43 | $this->cacheName = $parsedKey . $this->tss; | 
| 40 | 44 | return $parsedKey; | 
| 45 | +		} else { | |
| 46 | + return ''; | |
| 41 | 47 | } | 
| 42 | - else return ''; | |
| 43 | 48 | } | 
| 44 | 49 | |
| 45 | 50 | |
| @@ -47,7 +52,9 @@ discard block | ||
| 47 | 52 |      public function write($file, $rules, $imports = []) { | 
| 48 | 53 |  		if (is_file($file)) { | 
| 49 | 54 | $existing = $this->cache->load($file, filemtime($file)); | 
| 50 | - if (isset($existing['import'])) $imports = $existing['import']; | |
| 55 | +			if (isset($existing['import'])) { | |
| 56 | + $imports = $existing['import']; | |
| 57 | + } | |
| 51 | 58 | $this->cache->write($this->cacheName, ['rules' => $rules, 'import' => $imports, 'minFreq' => $this->getMinUpdateFreq($rules), 'ctime' => $this->time, 'cacheKey' => $this->cacheKey]); | 
| 52 | 59 | } | 
| 53 | 60 | return $rules; | 
| @@ -41,7 +41,7 @@ discard block | ||
| 41 | 41 | } | 
| 42 | 42 | |
| 43 | 43 |  	public function updateRequired($data) { | 
| 44 | - $this->cacheName = $this->getCacheKey($data) . $this->fileName; | |
| 44 | + $this->cacheName = $this->getCacheKey($data).$this->fileName; | |
| 45 | 45 | |
| 46 | 46 | $rules = $this->getRulesFromCache($this->fileName, $data); | 
| 47 | 47 | //Nothing was cached or the TSS file has changed, update is required | 
| @@ -58,7 +58,7 @@ discard block | ||
| 58 | 58 |  		if ($this->cacheKey) { | 
| 59 | 59 | $parser = new \Transphporm\Parser\Value($data); | 
| 60 | 60 | $cacheKey = $parser->parseTokens($this->cacheKey)[0]; | 
| 61 | - $this->cacheName = $cacheKey . $this->fileName; | |
| 61 | + $this->cacheName = $cacheKey.$this->fileName; | |
| 62 | 62 | return $cacheKey; | 
| 63 | 63 | } | 
| 64 | 64 | else return ''; | 
| @@ -29,7 +29,9 @@ discard block | ||
| 29 | 29 |  		if ($rules) { | 
| 30 | 30 |  			foreach ($rules['import'] as $file) { | 
| 31 | 31 | //Check that the import file hasn't been changed since the cache was written | 
| 32 | - if (filemtime($file) > $rules['ctime']) return false; | |
| 32 | +				if (filemtime($file) > $rules['ctime']) { | |
| 33 | + return false; | |
| 34 | + } | |
| 33 | 35 | } | 
| 34 | 36 | } | 
| 35 | 37 | |
| @@ -45,10 +47,14 @@ discard block | ||
| 45 | 47 | |
| 46 | 48 | $rules = $this->getRulesFromCache($this->fileName, $data); | 
| 47 | 49 | //Nothing was cached or the TSS file has changed, update is required | 
| 48 | - if (empty($rules)) return true; | |
| 50 | +		if (empty($rules)) { | |
| 51 | + return true; | |
| 52 | + } | |
| 49 | 53 | |
| 50 | 54 | //Find the sheet's minimum update-frequency, if it hasn't passed then no updates are required | 
| 51 | - if ($rules['ctime']+$rules['minFreq'] <= $this->time) return true; | |
| 55 | +		if ($rules['ctime']+$rules['minFreq'] <= $this->time) { | |
| 56 | + return true; | |
| 57 | + } | |
| 52 | 58 | |
| 53 | 59 | return false; | 
| 54 | 60 | } | 
| @@ -60,15 +66,19 @@ discard block | ||
| 60 | 66 | $cacheKey = $parser->parseTokens($this->cacheKey)[0]; | 
| 61 | 67 | $this->cacheName = $cacheKey . $this->fileName; | 
| 62 | 68 | return $cacheKey; | 
| 69 | +		} else { | |
| 70 | + return ''; | |
| 63 | 71 | } | 
| 64 | - else return ''; | |
| 65 | 72 | } | 
| 66 | 73 | |
| 67 | 74 |  	public function getRules($cssToXpath, $valueParser, $sheetLoader, $indexStart) { | 
| 68 | 75 | $rules = $this->getRulesFromCache($this->fileName)['rules']; | 
| 69 | 76 | $this->filePath->addPath(dirname(realpath($this->fileName))); | 
| 70 | - if (empty($rules)) $tss = file_get_contents($this->fileName); | |
| 71 | - else return $rules; | |
| 77 | +		if (empty($rules)) { | |
| 78 | + $tss = file_get_contents($this->fileName); | |
| 79 | +		} else { | |
| 80 | + return $rules; | |
| 81 | + } | |
| 72 | 82 | |
| 73 | 83 | return $tss == null ? [] : (new \Transphporm\Parser\Sheet($tss, $cssToXpath, $valueParser, $this->filePath, $sheetLoader))->parse($indexStart); | 
| 74 | 84 | } | 
| @@ -76,7 +86,9 @@ discard block | ||
| 76 | 86 | //write the sheet to cache | 
| 77 | 87 |      public function write($rules, $imports = []) { | 
| 78 | 88 | $existing = $this->cache->load($this->fileName, filemtime($this->fileName)); | 
| 79 | - if (isset($existing['import']) && empty($imports)) $imports = $existing['import']; | |
| 89 | +		if (isset($existing['import']) && empty($imports)) { | |
| 90 | + $imports = $existing['import']; | |
| 91 | + } | |
| 80 | 92 | $this->cache->write($this->cacheName, ['rules' => $rules, 'import' => $imports, 'minFreq' => $this->getMinUpdateFreq($rules), 'ctime' => $this->time, 'cacheKey' => $this->cacheKey]); | 
| 81 | 93 | |
| 82 | 94 | return $rules; | 
| @@ -88,7 +100,9 @@ discard block | ||
| 88 | 100 | |
| 89 | 101 |  		foreach ($rules as $rule) { | 
| 90 | 102 | $ruleFreq = $rule->getUpdateFrequency(); | 
| 91 | - if ($ruleFreq < $min) $min = $ruleFreq; | |
| 103 | +			if ($ruleFreq < $min) { | |
| 104 | + $min = $ruleFreq; | |
| 105 | + } | |
| 92 | 106 | } | 
| 93 | 107 | |
| 94 | 108 | return $min; | 
| @@ -39,8 +39,9 @@ discard block | ||
| 39 | 39 |  			if ($token['type'] == \Transphporm\Parser\Tokenizer::NAME && $token['value'] == 'data') { | 
| 40 | 40 | $tokens->next(); | 
| 41 | 41 | $newTokens = array_merge($newTokens, iterator_to_array($tokens->current()['value'])); | 
| 42 | +			} else { | |
| 43 | + $newTokens[] = $token; | |
| 42 | 44 | } | 
| 43 | - else $newTokens[] = $token; | |
| 44 | 45 | } | 
| 45 | 46 | |
| 46 | 47 | $this->tss->setCacheKey(new \Transphporm\Parser\Tokens($newTokens)); | 
| @@ -58,7 +59,9 @@ discard block | ||
| 58 | 59 | usort($rules, [$this, 'sortRules']); | 
| 59 | 60 | |
| 60 | 61 |  		foreach ($rules as $rule) { | 
| 61 | - if ($rule->shouldRun($this->time)) $this->executeTssRule($rule, $template, $config); | |
| 62 | +			if ($rule->shouldRun($this->time)) { | |
| 63 | + $this->executeTssRule($rule, $template, $config); | |
| 64 | + } | |
| 62 | 65 | } | 
| 63 | 66 | |
| 64 | 67 | //if (is_file($this->tss)) $this->write($this->tss, $rules, $this->import); | 
| @@ -83,10 +86,15 @@ discard block | ||
| 83 | 86 | |
| 84 | 87 |  	private function sortRules($a, $b) { | 
| 85 | 88 | //If they have the same depth, compare on index | 
| 86 | - if ($a->query === $b->query) return $this->sortPseudo($a, $b); | |
| 89 | +		if ($a->query === $b->query) { | |
| 90 | + return $this->sortPseudo($a, $b); | |
| 91 | + } | |
| 87 | 92 | |
| 88 | - if ($a->depth === $b->depth) $property = 'index'; | |
| 89 | - else $property = 'depth'; | |
| 93 | +		if ($a->depth === $b->depth) { | |
| 94 | + $property = 'index'; | |
| 95 | +		} else { | |
| 96 | + $property = 'depth'; | |
| 97 | + } | |
| 90 | 98 | |
| 91 | 99 | return ($a->$property < $b->$property) ? -1 : 1; | 
| 92 | 100 | } |