Level-2 / Transphporm

src/Parser/BracketMatcher.php

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
 		$close = strpos($this->str, $closingChr, $open);
 
 		$cPos = $open+1;
-		while (($cPos = strpos($this->str, $openChr, $cPos+1)) !== false && $cPos < $close) $close = strpos($this->str, $closingChr, $close+1);
+		while (($cPos = strpos($this->str, $openChr, $cPos+1)) !== false && $cPos < $close) {
+			$close = strpos($this->str, $closingChr, $close+1);
+		}
 
 		$this->startPos = $open;
 		$this->endPos = $close;

src/Property/Repeat.php

Doc Comments +3 added lines

@@ -34,6 +34,9 @@
 		return false;
 	}
 
+	/**
+	 * @param \Transphporm\Hook\PseudoMatcher $pseudoMatcher
+	 */
 	private function createHook($newRules, $pseudoMatcher, $properties) {
 		$hook = new \Transphporm\Hook\PropertyHook($newRules, $pseudoMatcher, new \Transphporm\Parser\Value($this->data));
 		foreach ($properties as $name => $property) $hook->registerProperty($name, $property);

Braces +9 added lines, -3 removed lines

@@ -13,13 +13,17 @@ 
 	}
 
 	public function run($value, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if ($element->getAttribute('transphporm') === 'added') return $element->parentNode->removeChild($element);
+		if ($element->getAttribute('transphporm') === 'added') {
+			return $element->parentNode->removeChild($element);
+		}
 
 		$count = 0;
 		foreach ($value as $key => $iteration) {
 			$clone = $element->cloneNode(true);
 			//Mark all but one of the nodes as having been added by transphporm, when the hook is run again, these are removed
-			if ($count++ > 0) $clone->setAttribute('transphporm', 'added');
+			if ($count++ > 0) {
+				$clone->setAttribute('transphporm', 'added');
+			}
 			$this->data->bind($clone, $iteration, 'iteration');
 			$this->data->bind($clone, $key, 'key');
 			$element->parentNode->insertBefore($clone, $element);
@@ -36,7 +40,9 @@ 
 
 	private function createHook($newRules, $pseudoMatcher, $properties) {
 		$hook = new \Transphporm\Hook\PropertyHook($newRules, $pseudoMatcher, new \Transphporm\Parser\Value($this->data));
-		foreach ($properties as $name => $property) $hook->registerProperty($name, $property);
+		foreach ($properties as $name => $property) {
+			$hook->registerProperty($name, $property);
+		}
 		return $hook;
 	}
 }
\ No newline at end of file

src/Hook/PropertyHook.php

Braces +9 added lines, -3 removed lines

@@ -20,11 +20,15 @@ 
 
 	public function run(\DomElement $element) {	
 		//Don't run if there's a pseudo element like nth-child() and this element doesn't match it
-		if (!$this->pseudoMatcher->matches($element)) return;
+		if (!$this->pseudoMatcher->matches($element)) {
+			return;
+		}
 
 		foreach ($this->rules as $name => $value) {
 			$result = $this->callProperty($name, $element, $this->valueParser->parse(trim($value), $element));
-			if ($result === false) break;
+			if ($result === false) {
+				break;
+			}
 		}
 	}
 
@@ -45,7 +49,9 @@ 
 	}
 
 	private function callProperty($name, $element, $value) {
-		if (isset($this->properties[$name])) return $this->properties[$name]->run($value, $element, $this->rules, $this->pseudoMatcher, $this->properties);
+		if (isset($this->properties[$name])) {
+			return $this->properties[$name]->run($value, $element, $this->rules, $this->pseudoMatcher, $this->properties);
+		}
 		return false;
 	}
 }

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 		if (!$this->pseudoMatcher->matches($element)) return;
 
 		foreach ($this->rules as $name => $value) {
-			$result = $this->callProperty($name, $element, $this->valueParser->parse(trim($value), $element)) ;
+			$result = $this->callProperty($name, $element, $this->valueParser->parse(trim($value), $element));
 			if ($result === false) break;
 		}
 	}

src/Property/Display.php

Braces +7 added lines, -3 removed lines

@@ -7,8 +7,12 @@
 namespace Transphporm\Property;
 class Display implements \Transphporm\Property {
 	public function run($value, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if ($pseudoMatcher->hasFunction('attr')) $element->removeAttribute($pseudoMatcher->getFuncArgs('attr'));
-		else if (strtolower($value[0]) === 'none') $element->setAttribute('transphporm', 'remove');
-		else $element->setAttribute('transphporm', 'show');
+		if ($pseudoMatcher->hasFunction('attr')) {
+			$element->removeAttribute($pseudoMatcher->getFuncArgs('attr'));
+		} else if (strtolower($value[0]) === 'none') {
+			$element->setAttribute('transphporm', 'remove');
+		} else {
+			$element->setAttribute('transphporm', 'show');
+		}
 	}
 }
\ No newline at end of file

src/Pseudo/Nth.php

Braces +5 added lines, -2 removed lines

@@ -15,8 +15,11 @@
 			$bracketMatcher = new \Transphporm\Parser\BracketMatcher($element->getNodePath());
 			$num = $bracketMatcher->match('[', ']');
 			
-			if (is_callable([$this, $criteria])) return $this->$criteria($num);
-			else return $num == $criteria;			
+			if (is_callable([$this, $criteria])) {
+				return $this->$criteria($num);
+			} else {
+				return $num == $criteria;
+			}
 		}
 		return true;
 	}

src/Hook/PseudoMatcher.php

Braces +12 added lines, -4 removed lines

@@ -31,7 +31,9 @@ 
 	
 	public function hasFunction($name) {
 		foreach ($this->pseudo as $pseudo) {
-			if (strpos($pseudo, $name) === 0) return true;
+			if (strpos($pseudo, $name) === 0) {
+				return true;
+			}
 		}
 	}
 
@@ -50,10 +52,16 @@ 
 		$parenthesis = strpos($pseudo, '(');
 		$square = strpos($pseudo, ']');
 
-		if ($parenthesis === false) $parenthesis = 999;
-		if ($square === false) $square = 999;
+		if ($parenthesis === false) {
+			$parenthesis = 999;
+		}
+		if ($square === false) {
+			$square = 999;
+		}
 
-		if ($parenthesis < $square) return ['(', ')'];
+		if ($parenthesis < $square) {
+			return ['(', ')'];
+		}
 		return ['[', ']'];
 	}
 	

src/Pseudo/Not.php

Braces +3 added lines, -1 removed lines

@@ -30,7 +30,9 @@
 			//Find all nodes matched by the expressions in the brackets :not(EXPR)
 			foreach ($xpath->query($xpathString) as $matchedElement) {
 				//Check to see whether this node was matched by the not query
-				if ($element->isSameNode($matchedElement)) return false;
+				if ($element->isSameNode($matchedElement)) {
+					return false;
+				}
 			}
 		}
 		return true;

src/Builder.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 		
 		$cachedOutput = $this->loadTemplate();
 		//To be a valid XML document it must have a root element, automatically wrap it in <template> to ensure it does
-		$template = new Template($this->isValidDoc($cachedOutput['body']) ? str_ireplace('<!doctype', '<!DOCTYPE', $cachedOutput['body']) : '<template>' . $cachedOutput['body'] . '</template>' );
+		$template = new Template($this->isValidDoc($cachedOutput['body']) ? str_ireplace('<!doctype', '<!DOCTYPE', $cachedOutput['body']) : '<template>'.$cachedOutput['body'].'</template>');
 
 		$this->processRules($template, $data, $featureSet);
 		
@@ -98,11 +98,11 @@ 
 	//N.b. only files can be cached
 	private function getRules($template, $valueParser) {		
 		if (is_file($this->tss)) {
-			$this->baseDir = dirname(realpath($this->tss)) . DIRECTORY_SEPARATOR;
+			$this->baseDir = dirname(realpath($this->tss)).DIRECTORY_SEPARATOR;
 			//The cache for the key: the filename and template prefix
 			//Each template may have a different prefix which changes the parsed TSS,
 			//Because of this the cache needs to be generated for each template prefix.
-			$key = $this->tss . $template->getPrefix() . $this->baseDir;
+			$key = $this->tss.$template->getPrefix().$this->baseDir;
 			//Try to load the cached rules, if not set in the cache (or expired) parse the supplied sheet
 			$rules = $this->cache->load($key, filemtime($this->tss));
 			if (!$rules) return $this->cache->write($key, (new Parser\Sheet(file_get_contents($this->tss), $this->baseDir, $valueParser, $template->getPrefix()))->parse());

Braces +18 added lines, -7 removed lines

@@ -26,7 +26,9 @@ 
 		$this->cache = new Cache(new \ArrayObject());
 
 		$modules = is_array($modules) ? $modules : $this->defaultModules;
-		foreach ($modules as $module) $this->loadModule(new $module);
+		foreach ($modules as $module) {
+			$this->loadModule(new $module);
+		}
 	}
 
 	//Allow setting the time used by Transphporm for caching. This is for testing purposes
@@ -47,7 +49,9 @@ 
 		$data = new FunctionSet($elementData, $this->baseDir);
 		$config = new Config($data, $elementData, new Hook\Formatter(), $headers, $this->baseDir);
 
-		foreach ($this->modules as $module) $module->load($config);
+		foreach ($this->modules as $module) {
+			$module->load($config);
+		}
 		
 		$cachedOutput = $this->loadTemplate();
 		//To be a valid XML document it must have a root element, automatically wrap it in <template> to ensure it does
@@ -67,7 +71,9 @@ 
 		$rules = $this->getRules($template, $valueParser);
 
 		foreach ($rules as $rule) {
-			if ($rule->shouldRun($this->time)) $this->executeTssRule($rule, $template, $valueParser, $config);
+			if ($rule->shouldRun($this->time)) {
+				$this->executeTssRule($rule, $template, $valueParser, $config);
+			}
 		}
 	}
 
@@ -92,8 +98,9 @@ 
 		if (trim($this->template)[0] !== '<') {			
 			$xml = $this->cache->load($this->template, filemtime($this->template));
 			return $xml ? $xml : ['body' => file_get_contents($this->template), 'headers' => []];
+		} else {
+			return ['body' => $this->template, 'headers' => []];
 		}
-		else return ['body' => $this->template, 'headers' => []];	
 	}
 
 	//Load the TSS rules either from a file or as a string
@@ -107,10 +114,14 @@ 
 			$key = $this->tss . $template->getPrefix() . $this->baseDir;
 			//Try to load the cached rules, if not set in the cache (or expired) parse the supplied sheet
 			$rules = $this->cache->load($key, filemtime($this->tss));
-			if (!$rules) return $this->cache->write($key, (new Parser\Sheet(file_get_contents($this->tss), $this->baseDir, $valueParser, $template->getPrefix()))->parse());
-			else return $rules;
+			if (!$rules) {
+				return $this->cache->write($key, (new Parser\Sheet(file_get_contents($this->tss), $this->baseDir, $valueParser, $template->getPrefix()))->parse());
+			} else {
+				return $rules;
+			}
+		} else {
+			return (new Parser\Sheet($this->tss, $this->baseDir, $valueParser, $template->getPrefix()))->parse();
 		}
-		else return (new Parser\Sheet($this->tss, $this->baseDir, $valueParser, $template->getPrefix()))->parse();
 	}
 
 	public function setCache(\ArrayAccess $cache) {

src/Parser/StringExtractor.php

Spacing +2 added lines, -2 removed lines

@@ -23,8 +23,8 @@
 			$end = strpos($str, '"', $pos+1);
 			if (!$end) break;
 			while ($str[$end-1] == '\\') $end = strpos($str, '"', $end+1);
-			$strings['$___STR' . ++$num] = substr($str, $pos, $end-$pos+1);
-			$str = substr_replace($str, '$___STR' . $num, $pos, $end-$pos+1);
+			$strings['$___STR'.++$num] = substr($str, $pos, $end-$pos+1);
+			$str = substr_replace($str, '$___STR'.$num, $pos, $end-$pos+1);
 		}
 
 		return [$str, $strings];

Braces +6 added lines, -2 removed lines

@@ -21,7 +21,9 @@ 
 		$strings = [];
 		while (isset($str[$pos]) && ($pos = strpos($str, '"', $pos)) !== false) {
 			$end = strpos($str, '"', $pos+1);
-			if (!$end) break;
+			if (!$end) {
+				break;
+			}
 			$end = $this->getNextPosEscaped($str, '\\', '"', $end);
 			$strings['$___STR' . ++$num] = substr($str, $pos, $end-$pos+1);
 			$str = substr_replace($str, '$___STR' . $num, $pos, $end-$pos+1);
@@ -31,7 +33,9 @@ 
 	}
 
 	private function getNextPosEscaped($str, $escape, $chr, $start) {
-		while ($str[$start-1] == $escape) $start = strpos($str, $chr, $start+1);
+		while ($str[$start-1] == $escape) {
+			$start = strpos($str, $chr, $start+1);
+		}
 		return $start;
 	}