Level-2 / Transphporm

src/Property/ContentPseudo/BeforeAfter.php

Doc Comments +3 added lines

@@ -4,6 +4,9 @@
 	private $insertLocation;
 	private $content;
 
+	/**
+	 * @param string $insertLocation
+	 */
 	public function __construct($insertLocation, \Transphporm\Property\Content $content) {
 		$this->insertLocation = $insertLocation;
 		$this->content = $content;

src/Property/ContentPseudo/Attr.php

Spacing +2 added lines, -2 removed lines

@@ -5,10 +5,10 @@
 		$implodedValue = implode('', $value);
 
 		if ($pseudoMatcher->hasFunction('before')) {
-			$attrValue = $implodedValue . $element->getAttribute($pseudoArgs);
+			$attrValue = $implodedValue.$element->getAttribute($pseudoArgs);
 		}
 		else if ($pseudoMatcher->hasFunction('after')) {
-			$attrValue = $element->getAttribute($pseudoArgs) . $implodedValue;
+			$attrValue = $element->getAttribute($pseudoArgs).$implodedValue;
 		}
 		else {
 			$attrValue = implode('', $value);

Braces +2 added lines, -4 removed lines

@@ -6,11 +6,9 @@
 
 		if ($pseudoMatcher->hasFunction('before')) {
 			$attrValue = $implodedValue . $element->getAttribute($pseudoArgs);
-		}
-		else if ($pseudoMatcher->hasFunction('after')) {
+		} else if ($pseudoMatcher->hasFunction('after')) {
 			$attrValue = $element->getAttribute($pseudoArgs) . $implodedValue;
-		}
-		else {
+		} else {
 			$attrValue = implode('', $value);
 		}
 

src/Property/ContentPseudo.php

Indentation +1 added lines, -1 removed lines

@@ -2,5 +2,5 @@
 namespace Transphporm\Property;
 
 interface ContentPseudo {
-    public function run($value, $pseudoArgs, $element, \Transphporm\Hook\PseudoMatcher $pseudoMatcher);
+	public function run($value, $pseudoArgs, $element, \Transphporm\Hook\PseudoMatcher $pseudoMatcher);
 }

src/RunException.php

Indentation +4 added lines, -4 removed lines

@@ -6,9 +6,9 @@
  * @version         1.2                                                             */
 namespace Transphporm;
 class RunException extends \Exception {
-    public function __construct($operationType, $operationName, \Exception $previous = null) {
-        $message = 'TSS Error: Problem carrying out ' . $operationType . ' "' . $operationName . '"';
+	public function __construct($operationType, $operationName, \Exception $previous = null) {
+		$message = 'TSS Error: Problem carrying out ' . $operationType . ' "' . $operationName . '"';
 
-        parent::__construct($message, 0, $previous);
-    }
+		parent::__construct($message, 0, $previous);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 namespace Transphporm;
 class RunException extends \Exception {
     public function __construct($operationType, $operationName, \Exception $previous = null) {
-        $message = 'TSS Error: Problem carrying out ' . $operationType . ' "' . $operationName . '"';
+        $message = 'TSS Error: Problem carrying out '.$operationType.' "'.$operationName.'"';
 
         parent::__construct($message, 0, $previous);
     }

src/TSSValidator.php

Braces +12 added lines, -6 removed lines

@@ -13,14 +13,17 @@ 
         $this->error = null;
         $tokens = $this->tokenize($tss);
 
-        foreach ($tokens as $token)
-            if (!$this->validateRule($token)) return false;
+        foreach ($tokens as $token) {
+                    if (!$this->validateRule($token)) return false;
+        }
 
         return true;
     }
 
     private function validateRule($token) {
-        if ($token['type'] !== Tokenizer::OPEN_BRACE) return true;
+        if ($token['type'] !== Tokenizer::OPEN_BRACE) {
+        	return true;
+        }
 
         return $this->checkBraces($token) && $this->checkSemicolons($token)
             && $this->checkParenthesis($token);
@@ -33,8 +36,9 @@ 
     private function checkSemicolons($braceToken) {
         $splitTokens = $braceToken['value']->splitOnToken(Tokenizer::COLON);
         array_shift($splitTokens); array_pop($splitTokens);
-        foreach ($splitTokens as $tokens)
-            if (!in_array(Tokenizer::SEMI_COLON, array_column(iterator_to_array($tokens), 'type'))) return false;
+        foreach ($splitTokens as $tokens) {
+                    if (!in_array(Tokenizer::SEMI_COLON, array_column(iterator_to_array($tokens), 'type'))) return false;
+        }
 
         return true;
     }
@@ -44,7 +48,9 @@ 
     }
 
     private function tokenize($tss) {
-        if (is_file($tss)) $tss = file_get_contents($tss);
+        if (is_file($tss)) {
+        	$tss = file_get_contents($tss);
+        }
         return (new Parser\Tokenizer($tss))->getTokens();
     }
 }

Indentation +30 added lines, -30 removed lines

@@ -7,44 +7,44 @@
 namespace Transphporm;
 use Transphporm\Parser\Tokenizer;
 class TSSValidator {
-    private $error;
+	private $error;
 
-    public function validate($tss) {
-        $this->error = null;
-        $tokens = $this->tokenize($tss);
+	public function validate($tss) {
+		$this->error = null;
+		$tokens = $this->tokenize($tss);
 
-        foreach ($tokens as $token)
-            if (!$this->validateRule($token)) return false;
+		foreach ($tokens as $token)
+			if (!$this->validateRule($token)) return false;
 
-        return true;
-    }
+		return true;
+	}
 
-    private function validateRule($token) {
-        if ($token['type'] !== Tokenizer::OPEN_BRACE) return true;
+	private function validateRule($token) {
+		if ($token['type'] !== Tokenizer::OPEN_BRACE) return true;
 
-        return $this->checkBraces($token) && $this->checkSemicolons($token)
-            && $this->checkParenthesis($token);
-    }
+		return $this->checkBraces($token) && $this->checkSemicolons($token)
+			&& $this->checkParenthesis($token);
+	}
 
-    private function checkBraces($token) {
-        return strpos($token['string'], '{') === false;
-    }
+	private function checkBraces($token) {
+		return strpos($token['string'], '{') === false;
+	}
 
-    private function checkSemicolons($braceToken) {
-        $splitTokens = $braceToken['value']->splitOnToken(Tokenizer::COLON);
-        array_shift($splitTokens); array_pop($splitTokens);
-        foreach ($splitTokens as $tokens)
-            if (!in_array(Tokenizer::SEMI_COLON, array_column(iterator_to_array($tokens), 'type'))) return false;
+	private function checkSemicolons($braceToken) {
+		$splitTokens = $braceToken['value']->splitOnToken(Tokenizer::COLON);
+		array_shift($splitTokens); array_pop($splitTokens);
+		foreach ($splitTokens as $tokens)
+			if (!in_array(Tokenizer::SEMI_COLON, array_column(iterator_to_array($tokens), 'type'))) return false;
 
-        return true;
-    }
+		return true;
+	}
 
-    private function checkParenthesis($token) {
-        return substr_count($token['string'], '(') === substr_count($token['string'], ')');
-    }
+	private function checkParenthesis($token) {
+		return substr_count($token['string'], '(') === substr_count($token['string'], ')');
+	}
 
-    private function tokenize($tss) {
-        if (is_file($tss)) $tss = file_get_contents($tss);
-        return (new Parser\Tokenizer($tss))->getTokens();
-    }
+	private function tokenize($tss) {
+		if (is_file($tss)) $tss = file_get_contents($tss);
+		return (new Parser\Tokenizer($tss))->getTokens();
+	}
 }

src/FilePath.php

Spacing +3 added lines, -3 removed lines

@@ -19,15 +19,15 @@
 
 	public function getFilePath($filePath) {
 		if (is_file($filePath)) return $filePath;
-		else if (is_file($this->baseDir . DIRECTORY_SEPARATOR . $filePath)) return $this->baseDir . DIRECTORY_SEPARATOR . $filePath;
+		else if (is_file($this->baseDir.DIRECTORY_SEPARATOR.$filePath)) return $this->baseDir.DIRECTORY_SEPARATOR.$filePath;
 		else return $this->loadFromPaths($filePath);
 	}
 
 	private function loadFromPaths($filePath) {
 		foreach ($this->paths as $path) {
-			if (is_file($path . DIRECTORY_SEPARATOR . $filePath)) return $path . DIRECTORY_SEPARATOR . $filePath;
+			if (is_file($path.DIRECTORY_SEPARATOR.$filePath)) return $path.DIRECTORY_SEPARATOR.$filePath;
 		}
 
-		throw new \Exception('File ' . $filePath . ' not found in paths (' . implode(', ', $this->paths) . ')');
+		throw new \Exception('File '.$filePath.' not found in paths ('.implode(', ', $this->paths).')');
 	}
 }

Braces +10 added lines, -4 removed lines

@@ -18,14 +18,20 @@
 	}
 
 	public function getFilePath($filePath) {
-		if (is_file($filePath)) return $filePath;
-		else if (is_file($this->baseDir . DIRECTORY_SEPARATOR . $filePath)) return $this->baseDir . DIRECTORY_SEPARATOR . $filePath;
-		else return $this->loadFromPaths($filePath);
+		if (is_file($filePath)) {
+			return $filePath;
+		} else if (is_file($this->baseDir . DIRECTORY_SEPARATOR . $filePath)) {
+			return $this->baseDir . DIRECTORY_SEPARATOR . $filePath;
+		} else {
+			return $this->loadFromPaths($filePath);
+		}
 	}
 
 	private function loadFromPaths($filePath) {
 		foreach ($this->paths as $path) {
-			if (is_file($path . DIRECTORY_SEPARATOR . $filePath)) return $path . DIRECTORY_SEPARATOR . $filePath;
+			if (is_file($path . DIRECTORY_SEPARATOR . $filePath)) {
+				return $path . DIRECTORY_SEPARATOR . $filePath;
+			}
 		}
 
 		throw new \Exception('File ' . $filePath . ' not found in paths (' . implode(', ', $this->paths) . ')');

src/Template.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 			$this->save = function($content = null) {
 				return $this->document->saveHtml($content);
 			};
-			$this->document->loadHtml('<' . '?xml encoding="UTF-8">' . $doc,  LIBXML_HTML_NODEFDTD | LIBXML_HTML_NOIMPLIED);
+			$this->document->loadHtml('<'.'?xml encoding="UTF-8">'.$doc, LIBXML_HTML_NODEFDTD | LIBXML_HTML_NOIMPLIED);
 
 		}
 		else {
@@ -89,7 +89,7 @@ 
 		if ($document) return $this->document;
 
 		//Print the doctype... saveHtml inexplicably does not support $doc->doctype as an argument
-		$output = ($this->document->doctype) ? $this->document->saveXml($this->document->doctype) . "\n" : '';
+		$output = ($this->document->doctype) ? $this->document->saveXml($this->document->doctype)."\n" : '';
 
 		if ($this->document->documentElement->tagName !== 'template') $output .= call_user_func($this->save, $this->document->documentElement);
 		else $output = $this->printDocument();

Braces +12 added lines, -6 removed lines

@@ -41,8 +41,7 @@ 
 			};
 			$this->document->loadHtml('<' . '?xml encoding="UTF-8">' . $doc,  LIBXML_HTML_NODEFDTD | LIBXML_HTML_NOIMPLIED);
 
-		}
-		else {
+		} else {
 			$this->document->loadXml($doc);
 			//XML was loaded, save as XML.
 			$this->save = function($content = null) {
@@ -79,7 +78,9 @@ 
 	/** Prints out the current DomDocument as HTML */
 	private function printDocument() {
 		$output = '';
-		foreach ($this->document->documentElement->childNodes as $node) $output .= call_user_func($this->save, $node);
+		foreach ($this->document->documentElement->childNodes as $node) {
+			$output .= call_user_func($this->save, $node);
+		}
 		return $output;
 	}
 
@@ -91,13 +92,18 @@ 
 		//Generate the document by taking only the childnodes of the template, ignoring the <template> and </template> tags
 		//TODO: Is there a faster way of doing this without string manipulation on the output or this loop through childnodes?
 		 //Either return a whole DomDocument or return the output HTML
-		if ($document) return $this->document;
+		if ($document) {
+			return $this->document;
+		}
 
 		//Print the doctype... saveHtml inexplicably does not support $doc->doctype as an argument
 		$output = ($this->document->doctype) ? $this->document->saveXml($this->document->doctype) . "\n" : '';
 
-		if ($this->document->documentElement->tagName !== 'template') $output .= call_user_func($this->save, $this->document->documentElement);
-		else $output = $this->printDocument();
+		if ($this->document->documentElement->tagName !== 'template') {
+			$output .= call_user_func($this->save, $this->document->documentElement);
+		} else {
+			$output = $this->printDocument();
+		}
 
 		//repair empty tags. Browsers break on <script /> and <div /> so can't avoid LIBXML_NOEMPTYTAG but they also break on <base></base> so repair them
 		$output = str_replace(['></img>', '></br>', '></meta>', '></base>', '></link>', '></hr>', '></input>'], ' />', $output);

src/TSSFunction/Json.php

Spacing +2 added lines, -2 removed lines

@@ -17,13 +17,13 @@
 
 		if ($this->isJsonFile($json)) {
 			$path = $this->filePath->getFilePath($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($path);
 		}
 
 		$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

@@ -17,13 +17,17 @@
 
 		if ($this->isJsonFile($json)) {
 			$path = $this->filePath->getFilePath($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($path);
 		}
 
 		$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/Hook/ElementData.php

Braces +3 added lines, -1 removed lines

@@ -25,7 +25,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]) && array_key_exists($type, $this->elementMap[$element])) return $this->elementMap[$element][$type];
+			if (isset($this->elementMap[$element]) && array_key_exists($type, $this->elementMap[$element])) {
+				return $this->elementMap[$element][$type];
+			}
 			$element = $element->parentNode;
 		}
 		return $this->data;