Level-2 / Transphporm

src/Config.php

Indentation +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
 	}
 
 	public function &getLine() {
-        $line = &$this->line;
+		$line = &$this->line;
 		return $line;
 	}
 
@@ -68,9 +68,9 @@ 
 		$this->properties[$name] = $property;
 	}
 
-    public function registerContentPseudo($name, Property\ContentPseudo $pseudo) {
-        if (isset($this->properties['content'])) $this->properties['content']->addContentPseudo($name, $pseudo);
-    }
+	public function registerContentPseudo($name, Property\ContentPseudo $pseudo) {
+		if (isset($this->properties['content'])) $this->properties['content']->addContentPseudo($name, $pseudo);
+	}
 
 	public function registerPseudo($name, Pseudo $pseudo) {
 		$this->pseudo[$name] = $pseudo;

Braces +9 added lines, -3 removed lines

@@ -69,7 +69,9 @@ 
 	}
 
     public function registerContentPseudo($name, Property\ContentPseudo $pseudo) {
-        if (isset($this->properties['content'])) $this->properties['content']->addContentPseudo($name, $pseudo);
+        if (isset($this->properties['content'])) {
+        	$this->properties['content']->addContentPseudo($name, $pseudo);
+        }
     }
 
 	public function registerPseudo($name, Pseudo $pseudo) {
@@ -77,12 +79,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 $name => $pseudoFunction) $pseudoMatcher->registerFunction($name, clone $pseudoFunction);
+		foreach ($this->pseudo as $name => $pseudoFunction) {
+			$pseudoMatcher->registerFunction($name, clone $pseudoFunction);
+		}
 		return $pseudoMatcher;
 	}
 

src/Hook/PseudoMatcher.php

Braces +28 added lines, -13 removed lines

@@ -25,8 +25,12 @@ 
 	public function matches($element) {
 		foreach ($this->pseudo as $i => $tokens) {
 			$parts = $this->getFuncParts($i, $tokens);
-			if ($parts['name'] === null) $parts['name'] = 'data';
-			if (!isset($this->functions[$parts['name']])) return true;
+			if ($parts['name'] === null) {
+				$parts['name'] = 'data';
+			}
+			if (!isset($this->functions[$parts['name']])) {
+				return true;
+			}
 			if ($this->match($parts, $this->functions[$parts['name']], $element) === false) {
 				return false;
 			}
@@ -38,14 +42,17 @@ 
 	private function match($parts, $function, $element) {
 		try {
 			$matches = $function->match($parts['name'], $parts['args'], $element);
-			if ($matches === false) return false;
-		}
-		catch (\Exception $e) {
+			if ($matches === false) {
+				return false;
+			}
+		} catch (\Exception $e) {
 			throw new \Transphporm\RunException(\Transphporm\Exception::PSEUDO, $parts['name'], $e);
 		}
 	}
 	private function getFuncParts($i, $tokens) {
-		if (isset($this->funcParts[$i])) return $this->funcParts[$i];
+		if (isset($this->funcParts[$i])) {
+			return $this->funcParts[$i];
+		}
 		$parts = [];
 		$canCache = true;
 		$parts['name'] = $this->getFuncName($tokens);
@@ -53,32 +60,40 @@ 
 			//If the args are dynamic, it can't be cached as it may change between calls
 			$canCache = false;
 			$parts['args'] = $this->valueParser->parseTokens($tokens);
-		}
-		else if (count($tokens) > 1) {
+		} else if (count($tokens) > 1) {
 			$tokens->rewind();
 			$tokens->next();
 			$parts['args'] = $this->valueParser->parseTokens($tokens->current()['value']);
+		} else {
+			$parts['args'] = [['']];
+		}
+		if ($canCache) {
+			$this->funcParts[$i] = $parts;
 		}
-		else $parts['args'] = [['']];
-		if ($canCache) $this->funcParts[$i] = $parts;
 		return $parts;
 	}
 
 	private function getFuncName($tokens) {
-		if ($tokens->type() === Tokenizer::NAME) return $tokens->read();
+		if ($tokens->type() === Tokenizer::NAME) {
+			return $tokens->read();
+		}
 		return null;
 	}
 
 	public function hasFunction($name) {
 		foreach ($this->pseudo as $tokens) {
-			if ($name === $this->getFuncName($tokens)) return true;
+			if ($name === $this->getFuncName($tokens)) {
+				return true;
+			}
 		}
 	}
 
 	public function getFuncArgs($name) {
 		foreach ($this->pseudo as $i => $tokens) {
 			$parts = $this->getFuncParts($i, $tokens);
-			if ($name === $parts['name']) return $parts['args'];
+			if ($name === $parts['name']) {
+				return $parts['args'];
+			}
 		}
 	}
 }

src/Parser/Value.php

Braces +14 added lines, -10 removed lines

@@ -54,7 +54,9 @@ 
 	}
 
 	public function parseTokens($tokens, $data = null) {
-		if (count($tokens) === 0) return [$data];
+		if (count($tokens) === 0) {
+			return [$data];
+		}
 
 		$this->result = new ValueResult();
 		$this->data = new ValueData($data ? $data : $this->baseData);
@@ -64,7 +66,9 @@ 
 
 
 		foreach ($tokens as $name => $token) {
-			if ($token['type'] == 'WHITESPACE' || $token['type'] == 'NEWLINE') continue;
+			if ($token['type'] == 'WHITESPACE' || $token['type'] == 'NEWLINE') {
+				continue;
+			}
 			$this->{$this->tokenFuncs[$token['type']]}($token);
 		}
 
@@ -91,8 +95,7 @@ 
 		// foo.bar is treated as looking up `bar` in `foo` whereas .foo is treated as the string ".foo"
 		if ($lastResult) {
 			$this->last->makeTraversing();
-		}
-		else if ($this->last->isEmpty())  {
+		} else if ($this->last->isEmpty())  {
 			$this->processString(['value' => '.']);
 			$this->result->setMode(Tokenizer::CONCAT);
 		}
@@ -107,11 +110,12 @@ 
 	private function processSquareBracket($token) {
 		if ($this->hasFunction($this->last->read())) {
 			$this->callTransphpormFunctions($token);
-		}
-		else {
+		} else {
 			$this->last->traverse();
 			$this->last->set($this->getNewParser()->parseTokens($token['value'], null)[0]);
-			if (!is_bool($this->last->read())) $this->last->makeTraversing();
+			if (!is_bool($this->last->read())) {
+				$this->last->makeTraversing();
+			}
 		}
 	}
 
@@ -134,8 +138,7 @@ 
 		if ($this->hasFunction($this->last->read())
 			&& !$this->data->methodExists($this->last->read())) {
 			$this->callTransphpormFunctions($token);
-		}
-		else {
+		} else {
 			$this->last->processNested($this->getNewParser(), $token);
 		}
 	}
@@ -152,8 +155,9 @@ 
 			$parser = new Value($this->data->getData());
 			$parsedArr = $parser->parse($val);
 			$parsedVal = isset($parsedArr[0]) ? $parsedArr[0] : null;
+		} else {
+			$parsedVal = null;
 		}
-		else $parsedVal = null;
 
         $this->result->postProcess($this->data, $val, $parsedVal, $this->allowNullResult);
 

src/Pseudo/Attribute.php

Braces +3 added lines, -1 removed lines

@@ -7,7 +7,9 @@
 namespace Transphporm\Pseudo;
 class Attribute implements \Transphporm\Pseudo {
 	public function match($name, $args, \DomElement $element) {
-		if ($name === null) return true;
+		if ($name === null) {
+			return true;
+		}
 		return $args[0];
 	}
 }