Level-2 / Transphporm

src/Hook/DataFunction.php

Braces +25 added lines, -9 removed lines

@@ -22,7 +22,9 @@ 
 	/** Binds data to an element */
 	public function bind(\DomElement $element, $data, $type = 'data') {
 		//This is a bit of a hack to workaround #24, might need a better way of doing this if it causes a problem
-		if (is_array($data) && $this->isObjectArray($data)) $data = $data[0];
+		if (is_array($data) && $this->isObjectArray($data)) {
+			$data = $data[0];
+		}
 		$content = isset($this->dataStorage[$element]) ? $this->dataStorage[$element] : [];
 		$content[$type] = $data;
 		$this->dataStorage[$element] = $content;
@@ -46,7 +48,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*/
 	private function getData(\DomElement $element = null, $type = 'data') {
 		while ($element) {
-			if (isset($this->dataStorage[$element]) && isset($this->dataStorage[$element][$type])) return $this->dataStorage[$element][$type];
+			if (isset($this->dataStorage[$element]) && isset($this->dataStorage[$element][$type])) {
+				return $this->dataStorage[$element][$type];
+			}
 			$element = $element->parentNode;
 		}
 		return $this->data;
@@ -64,17 +68,25 @@ 
 		$obj = $data;
 		$valueParser = new \Transphporm\Parser\Value($this);
 		foreach ($parts as $part) {
-			if ($part === '') continue;
+			if ($part === '') {
+				continue;
+			}
 			$part = $valueParser->parse($part, $element)[0];
-			if (is_callable([$obj, $part])) $obj = call_user_func([$obj, $part]); 
-			else $obj = $this->ifNull($obj, $part);
+			if (is_callable([$obj, $part])) {
+				$obj = call_user_func([$obj, $part]);
+			} else {
+				$obj = $this->ifNull($obj, $part);
+			}
 		}
 		return $obj;
 	}
 
 	private function ifNull($obj, $key) {
-		if (is_array($obj)) return isset($obj[$key]) ? $obj[$key] : null;
-		else return isset($obj->$key) ? $obj->$key : null;
+		if (is_array($obj)) {
+			return isset($obj[$key]) ? $obj[$key] : null;
+		} else {
+			return isset($obj->$key) ? $obj->$key : null;
+		}
 	}
 
 	public function attr($val, $element) {
@@ -100,14 +112,18 @@ 
 
 		$doc = $newTemplate->output([], true)->body;
 
-		if (isset($val[1])) return $this->templateSubsection($val[1], $doc, $element);
+		if (isset($val[1])) {
+			return $this->templateSubsection($val[1], $doc, $element);
+		}
 		
 		$newNode = $element->ownerDocument->importNode($doc->documentElement, true);
 
 		$result = [];
 
 		if ($newNode->tagName === 'template') {
-			foreach ($newNode->childNodes as $node) $result[] = $node->cloneNode(true);
+			foreach ($newNode->childNodes as $node) {
+				$result[] = $node->cloneNode(true);
+			}
 		}		
 		//else $result[] = $newNode;
 

src/Hook/PseudoMatcher.php

Braces +29 added lines, -11 removed lines

@@ -29,16 +29,22 @@ 
 		$close = strpos($string, $closingChr, $open);
 
 		$cPos = $open+1;
-		while (($cPos = strpos($string, $openChr, $cPos+1)) !== false && $cPos < $close) $close = strpos($string, $closingChr, $close+1);
+		while (($cPos = strpos($string, $openChr, $cPos+1)) !== false && $cPos < $close) {
+			$close = strpos($string, $closingChr, $close+1);
+		}
 
 		return substr($string, $open+1, $close-$open-1);
 	}
 	
 	private function attribute($pseudo, $element) {
-		if (strpos($pseudo, '=') === false) return true;
+		if (strpos($pseudo, '=') === false) {
+			return true;
+		}
 
 		$pos = strpos($pseudo, '[');
-		if ($pos === false) return true;
+		if ($pos === false) {
+			return true;
+		}
 		
 		$name = substr($pseudo, 0, $pos);
 		$criteria = $this->betweenBrackets($pseudo, '[', ']');
@@ -56,16 +62,21 @@ 
 	}
 
 	private function parseValue($value) {
-		if ($value == 'true') return true;
-		else if ($value == 'false') return false;
-		else return $value;
+		if ($value == 'true') {
+			return true;
+		} else if ($value == 'false') {
+			return false;
+		} else {
+			return $value;
+		}
 	}
 
 	private function getOperator($field) {
 		if ($field[strlen($field)-1] == '!') {
 			return '!';
+		} else {
+			return '';
 		}
-		else return '';
 	}
 
 	private function nth($pseudo, $element) {
@@ -73,8 +84,11 @@ 
 			$criteria = $this->getBetween($pseudo, '(', ')');
 			$num = $this->getBetween($element->getNodePath(), '[', ']');
 			
-			if (is_callable([$this, $criteria])) return $this->$criteria($num);
-			else return $num == $criteria;
+			if (is_callable([$this, $criteria])) {
+				return $this->$criteria($num);
+			} else {
+				return $num == $criteria;
+			}
 			
 		}
 		return true;
@@ -94,7 +108,9 @@ 
 	public function header($element)  {
 		if ($this->matches($element)) {
 			foreach ($this->pseudo as $pseudo) {
-				if (strpos($pseudo, 'header') === 0) return $this->getBetween($pseudo, '[', ']');
+				if (strpos($pseudo, 'header') === 0) {
+					return $this->getBetween($pseudo, '[', ']');
+				}
 			}
 		}
 	}
@@ -109,7 +125,9 @@ 
 
 	private function getBetween($string, $start, $end) {
 		$open = strpos($string, $start);
-		if ($open === false) return false;
+		if ($open === false) {
+			return false;
+		}
 		$close = strpos($string, $end, $open);
 		return substr($string, $open+1, $close-$open-1);
 	}