Level-2 / Transphporm

src/Property/Content.php

Doc Comments +19 added lines

@@ -11,6 +11,9 @@ 
 	private $formatter;
 
 
+	/**
+	 * @param \Transphporm\FunctionSet $data
+	 */
 	public function __construct($data, &$headers, \Transphporm\Hook\Formatter $formatter) {
 		$this->data = $data;
 		$this->headers = &$headers;
@@ -30,6 +33,9 @@ 
 		}
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function shouldRun($element) {
 		if ($element->getAttribute('transphporm') === 'remove') return false;
 		do {
@@ -43,6 +49,10 @@ 
 		return (isset($rules['content-mode'])) ? $rules['content-mode'] : 'append';
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 * @param \Transphporm\Hook\PseudoMatcher $pseudoMatcher
+	 */
 	private function processPseudo($value, $element, $pseudoMatcher) {
 		$pseudoContent = ['attr', 'header', 'before', 'after'];
 		foreach ($pseudoContent as $pseudo) {
@@ -93,6 +103,9 @@ 
 		}			 
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function replaceContent($element, $content) {
 		//If this rule was cached, the elements that were added last time need to be removed prior to running the rule again.
 		foreach ($this->getNode($content, $element->ownerDocument) as $node) {
@@ -101,12 +114,18 @@ 
 		$element->setAttribute('transphporm', 'remove');
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function appendContent($element, $content) {
 		foreach ($this->getNode($content, $element->ownerDocument) as $node) {
 			$element->appendChild($node);
 		}
 	}
 	
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function removeAllChildren($element) {
 		while ($element->hasChildNodes()) $element->removeChild($element->firstChild);
 	}

Braces +18 added lines, -8 removed lines

@@ -18,15 +18,20 @@ 
 	}
 
 	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if (!$this->shouldRun($element)) return false;
+		if (!$this->shouldRun($element)) {
+			return false;
+		}
 		$values = $this->fixValues($this->formatter->format($values, $rules));
 
 		if (!$this->processPseudo($values, $element, $pseudoMatcher)) {
 			//Remove the current contents
 			$this->removeAllChildren($element);
 			//Now make a text node
-			if ($this->getContentMode($rules) === 'replace') $this->replaceContent($element, $values);
-			else $this->appendContent($element, $values);
+			if ($this->getContentMode($rules) === 'replace') {
+				$this->replaceContent($element, $values);
+			} else {
+				$this->appendContent($element, $values);
+			}
 		}
 	}
 
@@ -38,7 +43,9 @@ 
 
 	private function shouldRun($element) {
 		do {
-			if ($element->getAttribute('transphporm') == 'includedtemplate') return false;
+			if ($element->getAttribute('transphporm') == 'includedtemplate') {
+				return false;
+			}
 		}
 		while (($element = $element->parentNode) instanceof \DomElement);
 		return true;
@@ -65,9 +72,10 @@ 
 				$new = $document->importNode($n, true);
 				//Removing this might cause problems with caching...
 				//$new->setAttribute('transphporm', 'added');
-			}
-			else {
-				if ($n instanceof \DomText) $n = $n->nodeValue;
+			} else {
+				if ($n instanceof \DomText) {
+					$n = $n->nodeValue;
+				}
 				$new = $document->createElement('text');
 
 				$new->appendChild($document->createTextNode($n));
@@ -114,6 +122,8 @@ 
 	}
 
 	private function removeAllChildren($element) {
-		while ($element->hasChildNodes()) $element->removeChild($element->firstChild);
+		while ($element->hasChildNodes()) {
+			$element->removeChild($element->firstChild);
+		}
 	}
 }

src/Formatter/StringFormatter.php

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
 		return ucwords($val);
 	}
 
+	/**
+	 * @param string $val
+	 */
 	public function html($val) {
 		$doc = new \DomDocument();
 		$doc->loadXML($val);

Spacing +1 added lines, -1 removed lines

@@ -27,6 +27,6 @@
 	public function debug($val) {
 		ob_start();
 		var_dump($val);
-		return $this->html('<pre>' . ob_get_clean() . '</pre>');
+		return $this->html('<pre>'.ob_get_clean().'</pre>');
 	}
 }
\ No newline at end of file

src/TSSFunction/Template.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	public function run(array $args, \DomElement $element) {
 		$selector = $this->readArray($args, 1);
 		$tss = $this->readArray($args, 2);
-		$newTemplate = new \Transphporm\Builder($this->baseDir . $args[0], $tss ? $this->baseDir . $tss : null);
+		$newTemplate = new \Transphporm\Builder($this->baseDir.$args[0], $tss ? $this->baseDir.$tss : null);
 
 		$doc = $newTemplate->output($this->elementData->getData($element), true)->body;
 		if ($selector != '') return $this->templateSubsection($doc, $selector);

Braces +6 added lines, -2 removed lines

@@ -30,7 +30,9 @@ 
 		$newTemplate = new \Transphporm\Builder($this->baseDir . $args[0], $tss ? $this->baseDir . $tss : null);
 
 		$doc = $newTemplate->output($this->elementData->getData($element), true)->body;
-		if ($selector != '') return $this->templateSubsection($doc, $selector);
+		if ($selector != '') {
+			return $this->templateSubsection($doc, $selector);
+		}
 
 		return $this->getTemplateContent($doc, $tss);
 
@@ -60,7 +62,9 @@ 
 
 	private function getClonedElement($node, $tss) {
 		$clone = $node->cloneNode(true);
-		if ($tss != null && $clone instanceof \DomElement) $clone->setAttribute('transphporm', 'includedtemplate');
+		if ($tss != null && $clone instanceof \DomElement) {
+			$clone->setAttribute('transphporm', 'includedtemplate');
+		}
 		return $clone;
 	}
 }

src/Hook/ElementData.php

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
 	/** Returns the data that has been bound to $element, or, if no data is bound to $element climb the DOM tree to find the data bound to a parent node*/
 	public function getData(\DomElement $element = null, $type = 'data') {
 		while ($element) {
-			if (isset($this->elementMap[$element]) && isset($this->elementMap[$element][$type])) return $this->elementMap[$element][$type];
+			if (isset($this->elementMap[$element]) && isset($this->elementMap[$element][$type])) {
+				return $this->elementMap[$element][$type];
+			}
 			$element = $element->parentNode;
 		}
 		return $this->data;

src/Parser/Tokenizer.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 			$string = $this->extractString($i);
 			$length = strlen($string)+1;
 			$char = $this->getChar($char);
-			$string = str_replace('\\' . $char, $char, $string);
+			$string = str_replace('\\'.$char, $char, $string);
 			$tokens[] = ['type' => self::STRING, 'value' => $string];
 			return $length;
 		}

Braces +25 added lines, -10 removed lines

@@ -81,10 +81,15 @@ 
 	}
 
 	private function processLiterals(&$tokens, $name) {
-		if (is_numeric($name)) $tokens[] = ['type' => self::NUMERIC, 'value' => $name];
-		else if ($name == 'true') $tokens[] = ['type' => self::BOOL, 'value' => true];
-		else if ($name == 'false') $tokens[] = ['type' => self::BOOL, 'value' => false];
-		else $tokens[] = ['type' => self::NAME, 'value' => $name];
+		if (is_numeric($name)) {
+			$tokens[] = ['type' => self::NUMERIC, 'value' => $name];
+		} else if ($name == 'true') {
+			$tokens[] = ['type' => self::BOOL, 'value' => true];
+		} else if ($name == 'false') {
+			$tokens[] = ['type' => self::BOOL, 'value' => false];
+		} else {
+			$tokens[] = ['type' => self::NAME, 'value' => $name];
+		}
 	}
 
 	private function doBrackets(&$tokens, $char, $i) {
@@ -118,7 +123,9 @@ 
 	private function extractString($pos) {
 		$char = $this->str[$pos];
 		$end = strpos($this->str, $char, $pos+1);
-		while ($end !== false && $this->str[$end-1] == '\\') $end = strpos($this->str, $char, $end+1);
+		while ($end !== false && $this->str[$end-1] == '\\') {
+			$end = strpos($this->str, $char, $end+1);
+		}
 
 		return substr($this->str, $pos+1, $end-$pos-1);
 	}
@@ -127,18 +134,26 @@ 
 		$close = strpos($this->str, $closeBracket, $open);
 
 		$cPos = $open+1;
-		while (($cPos = strpos($this->str, $startBracket, $cPos+1)) !== false && $cPos < $close) $close = strpos($this->str, $closeBracket, $close+1);
+		while (($cPos = strpos($this->str, $startBracket, $cPos+1)) !== false && $cPos < $close) {
+			$close = strpos($this->str, $closeBracket, $close+1);
+		}
 		return substr($this->str, $open+1, $close-$open-1);
 	}
 
 	private function identifyChar($chr) {
-		if (isset($this->chars[$chr])) return $this->chars[$chr];
-		else return self::NAME;
+		if (isset($this->chars[$chr])) {
+			return $this->chars[$chr];
+		} else {
+			return self::NAME;
+		}
 	}
 
 	private function getChar($num) {
 		$chars = array_reverse($this->chars);
-		if (isset($chars[$num])) return $chars[$num];
-		else return false;
+		if (isset($chars[$num])) {
+			return $chars[$num];
+		} else {
+			return false;
+		}
 	}
 }

src/TSSFunction/Data.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @version         1.0                                                             */
 namespace Transphporm\TSSFunction;
 /* Handles data() and iteration() function calls from the stylesheet */
-class Data implements \Transphporm\TSSFunction{
+class Data implements \Transphporm\TSSFunction {
 	private $data;
 	private $dataKey;
 	private $functionSet;

Braces +5 added lines, -2 removed lines

@@ -18,8 +18,11 @@
 	}
 
 	public function run(array $args, \DomElement $element = null) {
-		if ($this->dataKey === "root") $data = $this->data->getData(null, 'data');
-		else $data = $this->data->getData($element, $this->dataKey);
+		if ($this->dataKey === "root") {
+			$data = $this->data->getData(null, 'data');
+		} else {
+			$data = $this->data->getData($element, $this->dataKey);
+		}
 		$parser = new \Transphporm\Parser\Value($this->functionSet, true);
 		$return = $parser->parseTokens($args, $data);
 		return $return[0];

src/Hook/PropertyHook.php

Doc Comments +6 added lines

@@ -31,6 +31,9 @@ 
 		}
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function getArgs($value, $element) {
 		return $this->valueParser->parse($value, $element);
 	}
@@ -39,6 +42,9 @@ 
 		$this->properties[$name] = $property;
 	}
 
+	/**
+	 * @param \DOMElement $element
+	 */
 	private function callProperty($name, $element, $value) {
 		if (isset($this->properties[$name])) return $this->properties[$name]->run($value, $element, $this->rules, $this->pseudoMatcher, $this->properties);
 		return false;

Braces +12 added lines, -4 removed lines

@@ -23,11 +23,15 @@ 
 	public function run(\DomElement $element) {	
 		$this->functionSet->setElement($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->getArgs($value, $element));
-			if ($result === false) break;
+			if ($result === false) {
+				break;
+			}
 		}
 	}
 
@@ -40,8 +44,12 @@ 
 	}
 
 	private function callProperty($name, $element, $value) {	
-		if (empty($value[0])) $value[0] = [];
-		if (isset($this->properties[$name])) return $this->properties[$name]->run($value, $element, $this->rules, $this->pseudoMatcher, $this->properties);
+		if (empty($value[0])) {
+			$value[0] = [];
+		}
+		if (isset($this->properties[$name])) {
+			return $this->properties[$name]->run($value, $element, $this->rules, $this->pseudoMatcher, $this->properties);
+		}
 		return false;
 	}
 }

src/TSSFunction/Json.php

Indentation +15 added lines, -15 removed lines

@@ -1,25 +1,25 @@
 <?php
 namespace Transphporm\TSSFunction;
 class Json implements \Transphporm\TSSFunction {
-    private $baseDir;
+	private $baseDir;
 
-    public function __construct(&$baseDir) {
-        $this->baseDir = &$baseDir;
-    }
+	public function __construct(&$baseDir) {
+		$this->baseDir = &$baseDir;
+	}
 
-    public function run(array $args, \DomElement $element = null) {
-        $json = $args[0];
+	public function run(array $args, \DomElement $element = null) {
+		$json = $args[0];
 
-        if (trim($json)[0] != '{') {
-            $path = $this->baseDir . $json;
-            if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
-            $json = file_get_contents($json);
-        }
+		if (trim($json)[0] != '{') {
+			$path = $this->baseDir . $json;
+			if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+			$json = file_get_contents($json);
+		}
 
-        $map = json_decode($json, true);
+		$map = json_decode($json, true);
 
-        if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+		if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
 
-        return $map;
-    }
+		return $map;
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -11,14 +11,14 @@
         $json = $args[0];
 
         if (trim($json)[0] != '{') {
-            $path = $this->baseDir . $json;
-            if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+            $path = $this->baseDir.$json;
+            if (!file_exists($path)) throw new \Exception('File does not exist at: '.$path);
             $json = file_get_contents($json);
         }
 
         $map = json_decode($json, true);
 
-        if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+        if (!is_array($map)) throw new \Exception('Could not decode json: '.json_last_error_msg());
 
         return $map;
     }

Braces +6 added lines, -2 removed lines

@@ -12,13 +12,17 @@
 
         if (trim($json)[0] != '{') {
             $path = $this->baseDir . $json;
-            if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+            if (!file_exists($path)) {
+            	throw new \Exception('File does not exist at: ' . $path);
+            }
             $json = file_get_contents($json);
         }
 
         $map = json_decode($json, true);
 
-        if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+        if (!is_array($map)) {
+        	throw new \Exception('Could not decode json: ' . json_last_error_msg());
+        }
 
         return $map;
     }

src/Config.php

Braces +6 added lines, -2 removed lines

@@ -67,12 +67,16 @@
 	}
 
 	public function loadProperties(Hook\PropertyHook $hook) {
-		foreach ($this->properties as $name => $property) $hook->registerProperty($name, $property);
+		foreach ($this->properties as $name => $property) {
+			$hook->registerProperty($name, $property);
+		}
 	}
 
 	public function createPseudoMatcher($pseudo) {
 		$pseudoMatcher = new Hook\PseudoMatcher($pseudo);
-		foreach ($this->pseudo as $pseudoFunction) $pseudoMatcher->registerFunction($pseudoFunction);
+		foreach ($this->pseudo as $pseudoFunction) {
+			$pseudoMatcher->registerFunction($pseudoFunction);
+		}
 		return $pseudoMatcher;
 	}