Level-2 / Transphporm

src/Hook/PseudoMatcher.php

Braces +31 added lines, -14 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,37 +60,47 @@ 
 			//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();
 			$this->skipWhitespace($tokens);
 			$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 skipWhitespace($tokens) {
-		while ($tokens->current()['type'] === 'WHITESPACE' || $tokens->current()['type'] == 'NEWLINE') $tokens->next();
+		while ($tokens->current()['type'] === 'WHITESPACE' || $tokens->current()['type'] == 'NEWLINE') {
+			$tokens->next();
+		}
 	}
 
 	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'];
+			}
 		}
 	}
 }