Level-2 / Transphporm

src/CssToXpath.php

Braces +5 added lines, -2 removed lines

@@ -37,8 +37,9 @@ 
 				$selector = $this->createSelector();
 				$selector->type = $css[$i];
 				$selectors[] = $selector;
+			} else {
+				$selector->string .= $css[$i];
 			}
-			else $selector->string .= $css[$i];			
 		}
 		return $selectors;
 	}
@@ -49,7 +50,9 @@ 
 		$this->depth = count($selectors);
 		$xpath = '/';
 		foreach ($selectors as $selector) {
-			if (isset($this->translators[$selector->type])) $xpath .= $this->translators[$selector->type]($selector->string);
+			if (isset($this->translators[$selector->type])) {
+				$xpath .= $this->translators[$selector->type]($selector->string);
+			}
 		}
 
 		$xpath = str_replace('/[', '/*[', $xpath);

src/Formatter/Date.php

Braces +16 added lines, -6 removed lines

@@ -44,8 +44,11 @@ 
 
 		$diffDays = $diff->invert === 1 ? $diff->days : 0- $diff->days;
 
-		if ($diffDays !== 0) return $this->dayOffset($diffDays);
-		else return $this->timeOffset($diff);
+		if ($diffDays !== 0) {
+			return $this->dayOffset($diffDays);
+		} else {
+			return $this->timeOffset($diff);
+		}
 	}
 
 	/** Calculates offset in hours/minutes/seconds */
@@ -80,15 +83,22 @@ 
 			[-365, -28, $strings['future'], 28, 'months'],
 			[-999999, -365, $strings['future'], 365, 'years'],
 		];
-		if (isset($strings['day_before_yesterday'])) array_unshift($ranges, [2, 2, $strings['day_before_yesterday'], 1, '']);
-		if (isset($strings['day_after_tomorrow'])) array_unshift($ranges, [-2, -2, $strings['day_after_tomorrow'], 1, '']);
+		if (isset($strings['day_before_yesterday'])) {
+			array_unshift($ranges, [2, 2, $strings['day_before_yesterday'], 1, '']);
+		}
+		if (isset($strings['day_after_tomorrow'])) {
+			array_unshift($ranges, [-2, -2, $strings['day_after_tomorrow'], 1, '']);
+		}
 		return $ranges;
 	}
 
 	/** Converts "week" to "weeks", "month" to "months" etc when plural is required using language from $locale */
 	private function getPlural($strings, $num, $interval) {
-		if ($interval !== '') return $num == 1 ? $strings[$interval . '_singular'] : $strings[$interval . '_plural'];
-		else return '';
+		if ($interval !== '') {
+			return $num == 1 ? $strings[$interval . '_singular'] : $strings[$interval . '_plural'];
+		} else {
+			return '';
+		}
 	}
 
 	/** Calculates offset in days/weeks/month/years */

src/Formatter/Number.php

Braces +5 added lines, -2 removed lines

@@ -13,7 +13,10 @@
 
 	public function currency($num) {
 		$num = $this->decimal($num, $this->locale['currency_decimals']);
-		if ($this->locale['currency_position'] === 'before') return $this->locale['currency'] . $num;
-		else return $num . $this->locale['currency'];
+		if ($this->locale['currency_position'] === 'before') {
+			return $this->locale['currency'] . $num;
+		} else {
+			return $num . $this->locale['currency'];
+		}
 	}
 }
\ No newline at end of file

src/Hook/BasicProperties.php

Braces +17 added lines, -6 removed lines

@@ -58,12 +58,15 @@ 
 				$node = $element->ownerDocument->importNode($node, true);
 				$element->appendChild($node);
 			}
+		} else {
+			$element->appendChild($element->ownerDocument->createTextNode(implode('', $content)));
 		}
-		else $element->appendChild($element->ownerDocument->createTextNode(implode('', $content)));		
 	}
 
 	private function removeAllChildren($element) {
-		while ($element->hasChildNodes()) $element->removeChild($element->firstChild);
+		while ($element->hasChildNodes()) {
+			$element->removeChild($element->firstChild);
+		}
 	}
 
 	public function registerFormatter($formatter) {
@@ -71,11 +74,15 @@ 
 	}
 
 	private function format($value, $rules) {
-		if (!isset($rules['format'])) return $value;
+		if (!isset($rules['format'])) {
+			return $value;
+		}
 		$format = new \Transphporm\StringExtractor($rules['format']);
 		$options = explode(' ', $format);
 		$functionName = array_shift($options);
-		foreach ($options as &$f) $f = trim($format->rebuild($f), '"');
+		foreach ($options as &$f) {
+			$f = trim($format->rebuild($f), '"');
+		}
 
 		return $this->processFormat($options, $functionName, $value);		
 	}
@@ -104,7 +111,9 @@ 
 			unset($newRules['repeat']);
 
 			$hook = new Rule($newRules, $rule->getPseudoMatcher(), $this->data);
-			foreach ($rule->getProperties() as $obj) $hook->registerProperties($obj);
+			foreach ($rule->getProperties() as $obj) {
+				$hook->registerProperties($obj);
+			}
 			$hook->run($clone);
 		}
 
@@ -115,7 +124,9 @@ 
 	}
 
 	public function display($value, $element) {
-		if (strtolower($value[0]) === 'none') $element->parentNode->removeChild($element);
+		if (strtolower($value[0]) === 'none') {
+			$element->parentNode->removeChild($element);
+		}
 	}
 
 	public function bind($value, $element) {

src/Hook/PseudoMatcher.php

Braces +26 added lines, -10 removed lines

@@ -22,12 +22,16 @@ 
 
 	private function attribute($pseudo, $element) {
 		$pos = strpos($pseudo, '[');
-		if ($pos === false) return true;
+		if ($pos === false) {
+			return true;
+		}
 		$end = strpos($pseudo, ']', $pos);
 
 		$name = substr($pseudo, 0, $pos);
 
-		if (strpos($pseudo, '=') === false) return true;
+		if (strpos($pseudo, '=') === false) {
+			return true;
+		}
 
 		$criteria = substr($pseudo, $pos+1, $end-$pos-1);
 		list ($field, $value) = explode('=', $criteria);
@@ -42,16 +46,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) {
@@ -59,8 +68,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;
@@ -80,7 +92,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, '[', ']');
+				}
 			}
 		}
 	}
@@ -95,7 +109,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);
 	}

src/Hook/Rule.php

Braces +26 added lines, -13 removed lines

@@ -15,11 +15,15 @@ 
 
 	public function run(\DomElement $element) {	
 		//Don't run if there's a pseudo element like nth-child() and this element doesn't match it
-		if (!$this->pseudoMatcher->matches($element)) return;
+		if (!$this->pseudoMatcher->matches($element)) {
+			return;
+		}
 
 		foreach ($this->rules as $name => $value) {
 			$result = $this->callProperty($name, $element, $this->parseValue(trim($value), $element));
-			if ($result === false) break;
+			if ($result === false) {
+				break;
+			}
 		}
 	}
 
@@ -52,8 +56,9 @@ 
 		$pos = $start+1;
 		$end = 0;
 		while ($end = strpos($string, $char, $pos)) {
-			if ($string[$end-1] === $escape) $pos = $end+1;
-			else {
+			if ($string[$end-1] === $escape) {
+				$pos = $end+1;
+			} else {
 				break;
 			}
 		}
@@ -74,14 +79,17 @@ 
 
 			//Count the number of fresh opening ( before $close
 			$cPos = $open+1;
-			while (($cPos = strpos($function, '(', $cPos+1)) !== false && $cPos < $close) $close = strpos($function, ')', $close+1);
+			while (($cPos = strpos($function, '(', $cPos+1)) !== false && $cPos < $close) {
+				$close = strpos($function, ')', $close+1);
+			}
 
 			$name = substr($function, 0, $open);
 
 			$params = substr($function, $open+1, $close-$open-1);
 			return ['name' => $name, 'params' => $params, 'endPoint' => $close];
+		} else {
+			return ['name' => null, 'params' => $function, 'endPoint' => strlen($function)];
 		}
-		else return ['name' => null, 'params' => $function, 'endPoint' => strlen($function)];
 		
 	}
 
@@ -90,23 +98,26 @@ 
 		if ($function && in_array($function[0], ['\'', '"'])) {
 			$finalPos = $this->findMatchingPos($function, $function[0]);
 			$result[] = $this->extractQuotedString($function[0], $function);
-		}
-		else {
+		} else {
 			$func = $this->parseFunction($function);
 			$finalPos = $func['endPoint'];			
 
 			if (($data = $this->callFunc($func['name'], $func['params'], $element)) !== false) {
 				$result = $this->appendToArray($result, $data);
-			} 
-			else $result[] = trim($function);
+			} else {
+				$result[] = trim($function);
+			}
 		}
 		$remaining = trim(substr($function, $finalPos+1));
 		return $this->parseNextValue($remaining, $result, $element);
 	}
 
 	private function appendToArray($array, $value) {
-		if (is_array($value)) $array += $value;
-		else $array[] = $value;
+		if (is_array($value)) {
+			$array += $value;
+		} else {
+			$array[] = $value;
+		}
 		return $array;
 	}
 
@@ -118,7 +129,9 @@ 
 	}
 
 	private function parseNextValue($remaining, $result, $element) {
-		if (strlen($remaining) > 0 && $remaining[0] == ',') $result = array_merge($result, $this->parseValue(trim(substr($remaining, 1)), $element));
+		if (strlen($remaining) > 0 && $remaining[0] == ',') {
+			$result = array_merge($result, $this->parseValue(trim(substr($remaining, 1)), $element));
+		}
 		return $result;
 	}
 

src/Sheet.php

Braces +14 added lines, -5 removed lines

@@ -26,7 +26,9 @@ 
 			$rules = $this->writeRule($rules, $selector, $rule);
 		}
 		//there may be processing instructions at the end
-		if ($processing = $this->processingInstructions($this->tss, $pos, strlen($this->tss))) $rules = array_merge($processing['rules'], $rules);
+		if ($processing = $this->processingInstructions($this->tss, $pos, strlen($this->tss))) {
+			$rules = array_merge($processing['rules'], $rules);
+		}
 		usort($rules, [$this, 'sortRules']);
 		return $rules;
 	}
@@ -43,7 +45,9 @@ 
 	}
 
 	private function writeRule($rules, $selector, $newRule) {
-		if (isset($rules[$selector])) $newRule->properties = array_merge($rules[$selector], $newRule->properties);
+		if (isset($rules[$selector])) {
+			$newRule->properties = array_merge($rules[$selector], $newRule->properties);
+		}
 		$rules[$selector] = $newRule;
 		
 		return $rules;
@@ -58,8 +62,9 @@ 
 				$pos = strpos($tss, ';', $spacePos);
 				$args = substr($tss, $spacePos+1, $pos-$spacePos-1);
 				$rules = array_merge($rules, $this->$funcName($args));
+			} else {
+				break;
 			}
-			else break;
 		}
 
 		return empty($rules) ? false : ['endPos' => $pos, 'rules' => $rules];
@@ -72,7 +77,9 @@ 
 
 	private function sortRules($a, $b) {
 		//If they have the same depth, compare on index
-		if ($a->depth === $b->depth) return $a->index < $b->index ? -1 : 1;
+		if ($a->depth === $b->depth) {
+			return $a->index < $b->index ? -1 : 1;
+		}
 
 		return ($a->depth < $b->depth) ? -1 : 1;
 	}
@@ -93,7 +100,9 @@ 
 		$return = [];
 
 		foreach ($rules as $rule) {
-			if (trim($rule) === '') continue;
+			if (trim($rule) === '') {
+				continue;
+			}
 			$parts = explode(':', $rule, 2);
 
 			$parts[1] = $stringExtractor->rebuild($parts[1]);

src/StringExtractor.php

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
 		$strings = [];
 		while (($pos = strpos($str, '"', $pos+1)) !== false) {
 			$end = strpos($str, '"', $pos+1);
-			while ($str[$end-1] == '\\') $end = strpos($str, '"', $end+1);
+			while ($str[$end-1] == '\\') {
+				$end = strpos($str, '"', $end+1);
+			}
 			$strings['$+STR' . ++$num] = substr($str, $pos, $end-$pos+1);
 			$str = substr_replace($str, '$+STR' . $num, $pos, $end-$pos+1);
 		}

src/Template.php

Braces +14 added lines, -5 removed lines

@@ -33,14 +33,18 @@ 
 	/** Loops through all assigned hooks, runs the Xpath query and calls the hook */
 	private function processHooks() {
 		foreach ($this->hooks as list($query, $hook)) {
-			foreach ($this->xpath->query($query) as $element) $hook->run($element);
+			foreach ($this->xpath->query($query) as $element) {
+				$hook->run($element);
+			}
 		}
 	}
 
 	/** Prints out the current DomDocument as HTML */
 	private function printDocument(\DomDocument $doc) {
 		$output = '';
-		foreach ($doc->documentElement->childNodes as $node) $output .= $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+		foreach ($doc->documentElement->childNodes as $node) {
+			$output .= $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+		}
 		return $output;
 	}
 
@@ -52,13 +56,18 @@ 
 		//Generate the document by taking only the childnodes of the template, ignoring the <template> and </template> tags
 		//TODO: Is there a faster way of doing this without string manipulation on the output or this loop through childnodes?
 		 //Either return a whole DomDocument or return the output HTML
-		if ($document) return $this->document;
+		if ($document) {
+			return $this->document;
+		}
 
 
 		$output = ($this->document->doctype) ? $this->document->saveXml($this->document->doctype) . "\n" : '';
 
-		if ($this->document->documentElement->tagName !== 'template') $output .= $this->document->saveXml($this->document->documentElement, LIBXML_NOEMPTYTAG);
-		else $output = $this->printDocument($this->document);
+		if ($this->document->documentElement->tagName !== 'template') {
+			$output .= $this->document->saveXml($this->document->documentElement, LIBXML_NOEMPTYTAG);
+		} else {
+			$output = $this->printDocument($this->document);
+		}
 
 		//repair empty tags. Browsers break on <script /> and <div /> so can't avoid LIBXML_NOEMPTYTAG but they also break on <base></base> so repair them
 		$output = str_replace(['></img>', '></br>', '></meta>', '></base>', '></link>', '></hr>', '></input>'], ' />', $output);