Level-2 / Transphporm

src/FunctionSet.php

Braces +2 added lines, -1 removed lines

@@ -19,8 +19,9 @@
 	public function __call($name, $args) {
 		if (isset($this->functions[$name])) {
 			return $this->functions[$name]->run($args[0], $args[1]);
+		} else {
+			return \Transphporm\Parser\Value::IS_NOT_FUNCTION;
 		}
-		else return \Transphporm\Parser\Value::IS_NOT_FUNCTION;
 	}
 
 	public function addFunction($name, \Transphporm\TSSFunction $function) {

src/Hook/ElementData.php

Braces +6 added lines, -2 removed lines

@@ -18,7 +18,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->elementMap[$element]) ? $this->elementMap[$element] : [];
 		$content[$type] = $data;
 		$this->elementMap[$element] = $content;
@@ -31,7 +33,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->elementMap[$element]) && isset($this->elementMap[$element][$type])) return $this->elementMap[$element][$type];
+			if (isset($this->elementMap[$element]) && isset($this->elementMap[$element][$type])) {
+				return $this->elementMap[$element][$type];
+			}
 			$element = $element->parentNode;
 		}
 		return $this->data;

src/TSSFunction/Data.php

Spacing +2 added lines, -2 removed lines

@@ -6,12 +6,12 @@
  * @version         1.0                                                             */
 namespace Transphporm\TSSFunction;
 /* Handles data() and iteration() function calls from the stylesheet */
-class Data implements \Transphporm\TSSFunction{
+class Data implements \Transphporm\TSSFunction {
 	private $data;
 	private $dataType;
 	private $functionSet;
 
-	public function __construct(\Transphporm\Hook\ElementData $data,  \Transphporm\FunctionSet $functionSet, $dataKey = 'data') {
+	public function __construct(\Transphporm\Hook\ElementData $data, \Transphporm\FunctionSet $functionSet, $dataKey = 'data') {
 		$this->data = $data;
 		$this->dataKey = $dataKey;
 		$this->functionSet = $functionSet;

Braces +17 added lines, -8 removed lines

@@ -25,14 +25,18 @@ 
 		$valueParser = new \Transphporm\Parser\Value($this->functionSet);
 
 		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;
 	}
@@ -41,14 +45,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;
+		}
 	}
 
 

src/Property/Display.php

Braces +7 added lines, -3 removed lines

@@ -7,8 +7,12 @@
 namespace Transphporm\Property;
 class Display implements \Transphporm\Property {
 	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if ($pseudoMatcher->hasFunction('attr')) $element->removeAttribute($pseudoMatcher->getFuncArgs('attr'));
-		else if (strtolower($values[0][0]) === 'none') $element->setAttribute('transphporm', 'remove');
-		else $element->setAttribute('transphporm', 'show');
+		if ($pseudoMatcher->hasFunction('attr')) {
+			$element->removeAttribute($pseudoMatcher->getFuncArgs('attr'));
+		} else if (strtolower($values[0][0]) === 'none') {
+			$element->setAttribute('transphporm', 'remove');
+		} else {
+			$element->setAttribute('transphporm', 'show');
+		}
 	}
 }
\ No newline at end of file

src/Property/Repeat.php

Braces +12 added lines, -4 removed lines

@@ -15,11 +15,15 @@ 
 	}
 
 	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		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
@@ -43,7 +47,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) {
@@ -52,7 +58,9 @@ 
 
 	private function createHook($newRules, $pseudoMatcher, $properties) {
 		$hook = new \Transphporm\Hook\PropertyHook($newRules, $pseudoMatcher, new \Transphporm\Parser\Value($this->functionSet));
-		foreach ($properties as $name => $property) $hook->registerProperty($name, $property);
+		foreach ($properties as $name => $property) {
+			$hook->registerProperty($name, $property);
+		}
 		return $hook;
 	}
 }
\ No newline at end of file

src/TSSFunction/Template.php

Braces +6 added lines, -2 removed lines

@@ -26,7 +26,9 @@ 
 		$newTemplate = new \Transphporm\Builder($this->baseDir . $args[0], $tss);
 
 		$doc = $newTemplate->output($this->elementData->getData($element), true)->body;
-		if ($selector != '') return $this->templateSubsection($doc, $selector);
+		if ($selector != '') {
+			return $this->templateSubsection($doc, $selector);
+		}
 		
 		$newNode = $doc->documentElement;
 		$result = [];
@@ -53,7 +55,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;
 	}
 }
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	public function run(array $args, \DomElement $element) {
 		$selector = $this->readArray($args, 1);
 		$tss = $this->readArray($args, 2);
-		$newTemplate = new \Transphporm\Builder($this->baseDir . $args[0], $tss ? $this->baseDir . $tss : null);
+		$newTemplate = new \Transphporm\Builder($this->baseDir.$args[0], $tss ? $this->baseDir.$tss : null);
 
 		$doc = $newTemplate->output($this->elementData->getData($element), true)->body;
 		if ($selector != '') return $this->templateSubsection($doc, $selector);

src/Formatter/StringFormatter.php

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
 		return ucwords($val);
 	}
 
+	/**
+	 * @param string $val
+	 */
 	public function html($val) {
 		$doc = new \DomDocument();
 		$doc->loadXML($val);

Spacing +1 added lines, -1 removed lines

@@ -27,6 +27,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>');
 	}
 }
\ No newline at end of file

src/TSSFunction/Json.php

Indentation +12 added lines, -12 removed lines

@@ -1,25 +1,25 @@
 <?php
 namespace Transphporm\TSSFunction;
 class Json implements \Transphporm\TSSFunction {
-        private $baseDir;
+		private $baseDir;
         
-        public function __construct(&$baseDir) {
-                $this->baseDir = &$baseDir;
-        }
+		public function __construct(&$baseDir) {
+				$this->baseDir = &$baseDir;
+		}
         
-        public function run(array $args, \DomElement $element = null) {
-                $json = $args[0];
+		public function run(array $args, \DomElement $element = null) {
+				$json = $args[0];
                 
-                if (trim($json)[0] != '{') {
-                        $path = $this->baseDir . $json;
-                        if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+				if (trim($json)[0] != '{') {
+						$path = $this->baseDir . $json;
+						if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
 			$json = file_get_contents($json);
 		}
 
 		$map = json_decode($json, true);
                 
-                if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+				if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
                 
-                return $map;
-        }
+				return $map;
+		}
 }

Spacing +3 added lines, -3 removed lines

@@ -11,14 +11,14 @@
                 $json = $args[0];
                 
                 if (trim($json)[0] != '{') {
-                        $path = $this->baseDir . $json;
-                        if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+                        $path = $this->baseDir.$json;
+                        if (!file_exists($path)) throw new \Exception('File does not exist at: '.$path);
 			$json = file_get_contents($json);
 		}
 
 		$map = json_decode($json, true);
                 
-                if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+                if (!is_array($map)) throw new \Exception('Could not decode json: '.json_last_error_msg());
                 
                 return $map;
         }

Braces +6 added lines, -2 removed lines

@@ -12,13 +12,17 @@
                 
                 if (trim($json)[0] != '{') {
                         $path = $this->baseDir . $json;
-                        if (!file_exists($path)) throw new \Exception('File does not exist at: ' . $path);
+                        if (!file_exists($path)) {
+                        	throw new \Exception('File does not exist at: ' . $path);
+                        }
 			$json = file_get_contents($json);
 		}
 
 		$map = json_decode($json, true);
                 
-                if (!is_array($map)) throw new \Exception('Could not decode json: ' . json_last_error_msg());
+                if (!is_array($map)) {
+                	throw new \Exception('Could not decode json: ' . json_last_error_msg());
+                }
                 
                 return $map;
         }

src/Pseudo/Attribute.php

Braces +15 added lines, -6 removed lines

@@ -14,10 +14,14 @@ 
 
 	public function match($pseudo, \DomElement $element) {
 		$pos = strpos($pseudo, '[');
-		if ($pos === false) return true;
+		if ($pos === false) {
+			return true;
+		}
 
 		$name = substr($pseudo, 0, $pos);
-		if (!is_callable([$this->functionSet, $name])) return true;
+		if (!is_callable([$this->functionSet, $name])) {
+			return true;
+		}
 
 		$bracketMatcher = new \Transphporm\Parser\BracketMatcher($pseudo);
 		$criteria = $bracketMatcher->match('[', ']');
@@ -42,16 +46,21 @@ 
 
 	private function parseValue($value, $element) {
 		$valueParser = new \Transphporm\Parser\Value($this->functionSet);
-		if ($value == 'true') return true;
-		else if ($value == 'false') return false;
-		else return $valueParser->parse($value, $element)[0];
+		if ($value == 'true') {
+			return true;
+		} else if ($value == 'false') {
+			return false;
+		} else {
+			return $valueParser->parse($value, $element)[0];
+		}
 	}
 
 	private function getOperator($field) {
 		if ($field[strlen($field)-1] == '!') {
 			return '!';
+		} else {
+			return '';
 		}
-		else return '';
 	}
 
 }