Level-2 / Transphporm

src/Property/Repeat.php

Braces +15 added lines, -5 removed lines

@@ -18,12 +18,16 @@ 
 
 	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
 		$values = $this->fixEmpty($values);
-		if ($element->getAttribute('transphporm') === 'added') return $element->parentNode->removeChild($element);
+		if ($element->getAttribute('transphporm') === 'added') {
+			return $element->parentNode->removeChild($element);
+		}
 		$max = $this->getMax($values);
 		$count = 0;
 
 		foreach ($values[0] as $key => $iteration) {
-			if ($count+1 > $max) break;
+			if ($count+1 > $max) {
+				break;
+			}
 			$clone = $this->cloneElement($element, $iteration, $key, $count++);
 			//Re-run the hook on the new element, but use the iterated data
 			//Don't run repeat on the clones element or it will loop forever
@@ -36,7 +40,9 @@ 
 	}
 
 	private function fixEmpty($value) {
-		if (empty($value[0])) $value[0] = [];
+		if (empty($value[0])) {
+			$value[0] = [];
+		}
 		return $value;
 	}
 
@@ -52,7 +58,9 @@ 
 
 	private function tagElement($element, $count) {
 		//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) $element->setAttribute('transphporm', 'added');
+		if ($count > 0) {
+			$element->setAttribute('transphporm', 'added');
+		}
 	}
 
 	private function getMax($values) {
@@ -61,7 +69,9 @@ 
 
 	private function createHook($newRules, $pseudoMatcher, $properties) {
 		$hook = new \Transphporm\Hook\PropertyHook($newRules, $this->baseDir, $this->baseDir, $pseudoMatcher, new \Transphporm\Parser\Value($this->functionSet), $this->functionSet);
-		foreach ($properties as $name => $property) $hook->registerProperty($name, $property);
+		foreach ($properties as $name => $property) {
+			$hook->registerProperty($name, $property);
+		}
 		return $hook;
 	}
 }

src/Parser/Value.php

Braces +19 added lines, -15 removed lines

@@ -49,7 +49,9 @@ 
 		$this->data = new ValueData($data ? $data : $this->baseData);
 		$this->last = null;
 
-		if (count($tokens) <= 0) return [$data];
+		if (count($tokens) <= 0) {
+			return [$data];
+		}
 
 		foreach (new TokenFilterIterator($tokens, [Tokenizer::WHITESPACE]) as $token) {
 			$this->{$this->tokenFuncs[$token['type']]}($token);
@@ -71,13 +73,15 @@ 
 	//Reads the last selected value from $data regardless if it's an array or object and overrides $this->data with the new value
 	//Dot moves $data to the next object in $data foo.bar moves the $data pointer from `foo` to `bar`
 	private function processDot($token) {
-		if ($this->last !== null) $this->data->traverse($this->last);
-		else {
+		if ($this->last !== null) {
+			$this->data->traverse($this->last);
+		} else {
 			//When . is not preceeded by anything, treat it as part of the string instead of an operator
 			// foo.bar is treated as looking up `bar` in `foo` whereas .foo is treated as the string ".foo"
 			$lastResult = $this->result->pop();
-			if ($lastResult) $this->data = new ValueData($lastResult);
-			else {
+			if ($lastResult) {
+				$this->data = new ValueData($lastResult);
+			} else {
 				$this->processString(['value' => '.']);
 				$this->result->setMode(Tokenizer::CONCAT);
 			}
@@ -90,9 +94,10 @@ 
 		$parser = new Value($this->baseData, $this->autoLookup);
 		if ($this->baseData instanceof \Transphporm\Functionset && $this->baseData->hasFunction($this->last)) {
 			$this->callTransphpormFunctions($token);
-		}
-		else {
-			if ($this->last !== null) $this->data->traverse($this->last);
+		} else {
+			if ($this->last !== null) {
+				$this->data->traverse($this->last);
+			}
 			$this->last = $parser->parseTokens($token['value'], null)[0];
 		}
 	}
@@ -112,8 +117,7 @@ 
 	private function processBrackets($token) {
 		if ($this->baseData instanceof \Transphporm\Functionset && $this->baseData->hasFunction($this->last)) {
 			$this->callTransphpormFunctions($token);
-		}
-		else {
+		} else {
 			$this->processNested($token);
 		}
 	}
@@ -130,7 +134,9 @@ 
 		foreach ($this->result->getResult() as $i => $value) {
 			if (is_scalar($value)) {
 				$val = $this->data->read($value);
-				if ($val) $this->result[$i] = $val;
+				if ($val) {
+					$this->result[$i] = $val;
+				}
 			}
 		}
 		$this->last = null;
@@ -142,12 +148,10 @@ 
 			try {
 				$value = $this->data->extract($this->last, $this->autoLookup);
 				$this->result->processValue($value);
-			}
-			catch (\UnexpectedValueException $e) {
+			} catch (\UnexpectedValueException $e) {
 				if (!$this->autoLookup) {
 					$this->result->processValue($this->last);
-				}
-				else {
+				} else {
 					$this->result->clear();
 					$this->result[0] = false;
 				}

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, $this->valueParser);
-		foreach ($this->pseudo as $pseudoFunction) $pseudoMatcher->registerFunction(clone $pseudoFunction);
+		foreach ($this->pseudo as $pseudoFunction) {
+			$pseudoMatcher->registerFunction(clone $pseudoFunction);
+		}
 		return $pseudoMatcher;
 	}
 

src/Parser/TokenFilterIterator.php

Indentation +28 added lines, -28 removed lines

@@ -6,39 +6,39 @@
  * @version         1.0                                                             */
 namespace Transphporm\Parser;
 class TokenFilterIterator implements \Iterator {
-    private $ignore;
-    private $tokens;
+	private $ignore;
+	private $tokens;
 
-    public function __construct(Tokens $tokens, array $ignore) {
-        $this->ignore = $ignore;
-        $this->tokens = $tokens;
-    }
+	public function __construct(Tokens $tokens, array $ignore) {
+		$this->ignore = $ignore;
+		$this->tokens = $tokens;
+	}
 
-    public function current() {
-        return $this->tokens->current();
-    }
+	public function current() {
+		return $this->tokens->current();
+	}
 
-    public function key() {
-        return $this->tokens->key();
-    }
+	public function key() {
+		return $this->tokens->key();
+	}
 
-    public function valid() {
-        return $this->tokens->valid();
-    }
+	public function valid() {
+		return $this->tokens->valid();
+	}
 
-    public function next() {
-        do {
-            $this->tokens->next();
-        }
-        while ($this->shouldContinue());
-    }
+	public function next() {
+		do {
+			$this->tokens->next();
+		}
+		while ($this->shouldContinue());
+	}
 
-    public function rewind() {
-        $this->tokens->rewind();
-        while ($this->shouldContinue()) $this->tokens->next();
-    }
+	public function rewind() {
+		$this->tokens->rewind();
+		while ($this->shouldContinue()) $this->tokens->next();
+	}
 
-    private function shouldContinue() {
-        return $this->tokens->valid() && in_array($this->tokens->current()['type'], $this->ignore);
-    }
+	private function shouldContinue() {
+		return $this->tokens->valid() && in_array($this->tokens->current()['type'], $this->ignore);
+	}
 }

Braces +3 added lines, -1 removed lines

@@ -35,7 +35,9 @@
 
     public function rewind() {
         $this->tokens->rewind();
-        while ($this->shouldContinue()) $this->tokens->next();
+        while ($this->shouldContinue()) {
+        	$this->tokens->next();
+        }
     }
 
     private function shouldContinue() {

src/Formatter/HTMLFormatter.php

Doc Comments +3 added lines

@@ -7,6 +7,9 @@
         $this->templateFunction = $templateFunction;
     }
 
+    /**
+     * @param string $val
+     */
     public function html($val) {
 		return $this->templateFunction->run([$val]);
 	}

Indentation +5 added lines, -5 removed lines

@@ -1,13 +1,13 @@
 <?php
 namespace Transphporm\Formatter;
 class HTMLFormatter {
-    private $templateFunction;
+	private $templateFunction;
 
-    public function __construct(\Transphporm\TSSFunction\Template $templateFunction) {
-        $this->templateFunction = $templateFunction;
-    }
+	public function __construct(\Transphporm\TSSFunction\Template $templateFunction) {
+		$this->templateFunction = $templateFunction;
+	}
 
-    public function html($val) {
+	public function html($val) {
 		return $this->templateFunction->run([$val]);
 	}
 

Spacing +1 added lines, -1 removed lines

@@ -14,6 +14,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>');
 	}
 }

src/TSSFunction/Template.php

Spacing +2 added lines, -2 removed lines

@@ -26,9 +26,9 @@
 		$tss = $this->readArray($args, 2);
 
 		if (trim($args[0])[0] === '<') $xml = $args[0];
-		else $xml = $this->baseDir . $args[0];
+		else $xml = $this->baseDir.$args[0];
 
-		$newTemplate = new \Transphporm\Builder($xml, $tss ? $this->baseDir . $tss : null);
+		$newTemplate = new \Transphporm\Builder($xml, $tss ? $this->baseDir.$tss : null);
 
 		$doc = $newTemplate->output($this->elementData->getData($element), true)->body;
 		if ($selector != '') return $this->templateSubsection($doc, $selector);

Braces +11 added lines, -4 removed lines

@@ -25,13 +25,18 @@ 
 		$selector = $this->readArray($args, 1);
 		$tss = $this->readArray($args, 2);
 
-		if (trim($args[0])[0] === '<') $xml = $args[0];
-		else $xml = $this->baseDir . $args[0];
+		if (trim($args[0])[0] === '<') {
+			$xml = $args[0];
+		} else {
+			$xml = $this->baseDir . $args[0];
+		}
 
 		$newTemplate = new \Transphporm\Builder($xml, $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);
 
@@ -62,7 +67,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/FunctionSet.php

Braces +2 added lines, -4 removed lines

@@ -21,15 +21,13 @@
 				$tokens = $args[0];
 				$parser = new \Transphporm\Parser\Value($this);
 				$args[0] = $parser->parseTokens($tokens, $this->elementData->getData($this->element));
-			}
-			else if ($args[0] instanceof Parser\Tokens) {
+			} else if ($args[0] instanceof Parser\Tokens) {
 				$args[0] = iterator_to_array($args[0]);
 			}
 			if (isset($this->functions[$name])) {
 				return $this->functions[$name]->run($args[0], $this->element);
 			}
-		}
-		catch (\Exception $e) {
+		} catch (\Exception $e) {
 			throw new RunException(Exception::TSS_FUNCTION, $name, $e);
 		}
 		return true;

src/RunException.php

Indentation +4 added lines, -4 removed lines

@@ -1,9 +1,9 @@
 <?php
 namespace Transphporm;
 class RunException extends \Exception {
-    public function __construct($operationType, $operationName, \Exception $previous) {
-        $message = 'TSS Error: Problem carrying out ' . $operationType . ' \'' . $operationName . '\'';
+	public function __construct($operationType, $operationName, \Exception $previous) {
+		$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

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

src/Exception.php

Indentation +8 added lines, -8 removed lines

@@ -1,14 +1,14 @@
 <?php
 namespace Transphporm;
 class Exception extends \Exception {
-    const PROPERTY = 'property';
-    const TSS_FUNCTION = 'function';
-    const PSEUDO = 'pseudo';
-    const FORMATTER = 'formatter';
+	const PROPERTY = 'property';
+	const TSS_FUNCTION = 'function';
+	const PSEUDO = 'pseudo';
+	const FORMATTER = 'formatter';
 
-    public function __construct(RunException $runException, $file, $line) {
-        $message = $runException->getMessage() . ' on Line ' . $line . ' of ' . ($file === null ? 'tss' : $file);
+	public function __construct(RunException $runException, $file, $line) {
+		$message = $runException->getMessage() . ' on Line ' . $line . ' of ' . ($file === null ? 'tss' : $file);
 
-        parent::__construct($message, 0, $runException->getPrevious());
-    }
+		parent::__construct($message, 0, $runException->getPrevious());
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
     const FORMATTER = 'formatter';
 
     public function __construct(RunException $runException, $file, $line) {
-        $message = $runException->getMessage() . ' on Line ' . $line . ' of ' . ($file === null ? 'tss' : $file);
+        $message = $runException->getMessage().' on Line '.$line.' of '.($file === null ? 'tss' : $file);
 
         parent::__construct($message, 0, $runException->getPrevious());
     }