Level-2 / Transphporm

src/Hook/TemplateFunction.php

Doc Comments +4 added lines

@@ -11,6 +11,10 @@
 	private $selector;
 	private $tss;
 
+	/**
+	 * @param string $xml
+	 * @param string $tss
+	 */
 	public function __construct($xml, $selector = null, $tss = null) {
 		$this->xml = $xml;
 		$this->selector = $selector;

Braces +6 added lines, -2 removed lines

@@ -34,7 +34,9 @@ 
 
 	private function getClonedElement($node) {
 		$clone = $node->cloneNode(true);
-		if ($this->tss !== null && $clone instanceof \DomElement) $clone->setAttribute('transphporm', 'includedtemplate');
+		if ($this->tss !== null && $clone instanceof \DomElement) {
+			$clone->setAttribute('transphporm', 'includedtemplate');
+		}
 		return $clone;
 	}
 
@@ -42,7 +44,9 @@ 
 		$newTemplate = new \Transphporm\Builder($this->xml, $this->tss);
 
 		$doc = $newTemplate->output($data, true)->body;
-		if ($this->selector != '') return $this->templateSubsection($doc);
+		if ($this->selector != '') {
+			return $this->templateSubsection($doc);
+		}
 		
 		$newNode = $doc->documentElement;
 		$result = [];

src/Property/Content.php

Braces +24 added lines, -10 removed lines

@@ -18,22 +18,31 @@ 
 	}
 
 	public function run($value, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if (!$this->shouldRun($element)) return false;
+		if (!$this->shouldRun($element)) {
+			return false;
+		}
 			
 		$value = $this->formatter->format($value, $rules);
 		if (!$this->processPseudo($value, $element, $pseudoMatcher)) {
 			//Remove the current contents
 			$this->removeAllChildren($element);
 			//Now make a text node
-			if ($this->getContentMode($rules) === 'replace') $this->replaceContent($element, $value);
-			else $this->appendContent($element, $value);
+			if ($this->getContentMode($rules) === 'replace') {
+				$this->replaceContent($element, $value);
+			} else {
+				$this->appendContent($element, $value);
+			}
 		}
 	}
 
 	private function shouldRun($element) {
-		if ($element->getAttribute('transphporm') === 'remove') return false;
+		if ($element->getAttribute('transphporm') === 'remove') {
+			return false;
+		}
 		do {
-			if ($element->getAttribute('transphporm') == 'includedtemplate') return false;
+			if ($element->getAttribute('transphporm') == 'includedtemplate') {
+				return false;
+			}
 		}
 		while (($element = $element->parentNode) instanceof \DomElement);
 		return true;
@@ -60,9 +69,10 @@ 
 				$new = $document->importNode($n, true);
 				//Removing this might cause problems with caching... 
 				//$new->setAttribute('transphporm', 'added');
-			}
-			else {
-				if ($n instanceof \DomText) $n = $n->nodeValue;
+			} else {
+				if ($n instanceof \DomText) {
+					$n = $n->nodeValue;
+				}
 				$new = $document->createElement('text');
 				$new->appendChild($document->createTextNode($n));
 				$new->setAttribute('transphporm', 'text');
@@ -98,7 +108,9 @@ 
 		while ($e = $e->previousSibling && !in_array($e->getAttribute('transphporm'), [null, 'remove'])) {
 			$remove[] = $e;
 		}
-		foreach ($remove as $r) $r->parentNode->removeChild($r);
+		foreach ($remove as $r) {
+			$r->parentNode->removeChild($r);
+		}
 	}
 
 	private function replaceContent($element, $content) {
@@ -117,6 +129,8 @@ 
 	}
 	
 	private function removeAllChildren($element) {
-		while ($element->hasChildNodes()) $element->removeChild($element->firstChild);
+		while ($element->hasChildNodes()) {
+			$element->removeChild($element->firstChild);
+		}
 	}
 }
\ No newline at end of file

src/Hook/DataFunction.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
 	}
 
 	public function template($val, $element) {
-		$templateFunction = new TemplateFunction($this->baseDir . $val[0], isset($val[1]) ? $val[1] : null, isset($val[2]) ? $this->baseDir . $val[2] : null);		
+		$templateFunction = new TemplateFunction($this->baseDir.$val[0], isset($val[1]) ? $val[1] : null, isset($val[2]) ? $this->baseDir.$val[2] : null);		
 		return $templateFunction->getTemplateNodes($this->getData($element));
 	}
 }

Braces +23 added lines, -10 removed lines

@@ -24,7 +24,9 @@ 
 	/** Binds data to an element */
 	public function bind(\DomNode $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;
@@ -48,7 +50,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->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;
@@ -67,13 +71,17 @@ 
 		$valueParser = new \Transphporm\Parser\Value($this);
 
 		foreach ($parts as $part) {
-			if ($part === '') continue;
+			if ($part === '') {
+				continue;
+			}
 			$part = $valueParser->parse($part, $element)[0];
 			$funcResult = $this->traverseObj($part, $obj, $valueParser, $element);
 
-			if ($funcResult !== false) $obj = $funcResult;
-			
-			else $obj = $this->ifNull($obj, $part);
+			if ($funcResult !== false) {
+				$obj = $funcResult;
+			} else {
+				$obj = $this->ifNull($obj, $part);
+			}
 		}
 		return $obj;
 	}
@@ -82,14 +90,19 @@ 
 		if (strpos($part, '(') !== false) {
 			$subObjParser = new \Transphporm\Parser\Value($obj, $valueParser, false);
 			return $subObjParser->parse($part, $element)[0];
+		} else if (method_exists($obj, $part)) {
+			return call_user_func([$obj, $part]);
+		} else {
+			return false;
 		}
-		else if (method_exists($obj, $part)) return call_user_func([$obj, $part]); 
-		else return false;
 	}
 
 	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) {