Level-2 / Transphporm

src/Parser/Tokenizer/Literals.php

Braces +7 added lines, -3 removed lines

@@ -37,9 +37,13 @@
 	}
 
 	private function processLiterals($tokens, $name, $str) {
-		if (is_numeric($name)) $tokens->add(['type' => Tokenizer::NUMERIC, 'value' => $name]);
-		else if (method_exists($this, $name)) $this->$name($tokens);
-		else $tokens->add(['type' => Tokenizer::NAME, 'value' => $name, 'line' => $str->lineNo()]);
+		if (is_numeric($name)) {
+			$tokens->add(['type' => Tokenizer::NUMERIC, 'value' => $name]);
+		} else if (method_exists($this, $name)) {
+			$this->$name($tokens);
+		} else {
+			$tokens->add(['type' => Tokenizer::NAME, 'value' => $name, 'line' => $str->lineNo()]);
+		}
 	}
 
 	private function true($tokens) {

src/Parser/ValueResult.php

Indentation +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 
 	//Postprocessing - replace values with null where allowed, or override a value at position
 	public function postProcess(ValueData $data, $val, $overrideVal, $allowNull) {
-        if ($this->getMode() !== Tokenizer::ARG) return;
+		if ($this->getMode() !== Tokenizer::ARG) return;
 		foreach ($this->getResult() as $i => $value) {
 			if (is_scalar($value)) {
 				$val = ($overrideVal == $val) ? $data->read($value) : $overrideVal;

Braces +11 added lines, -6 removed lines

@@ -33,14 +33,17 @@ 
 		];
 
 		if ($funcs[$this->mode] === 'concat' && is_numeric($newValue)
-			&& is_numeric($this->result[count($this->result)-1]))
-			$this->add($newValue);
-		else
-			$this->{$funcs[$this->mode]}($newValue);
+			&& is_numeric($this->result[count($this->result)-1])) {
+					$this->add($newValue);
+		} else {
+					$this->{$funcs[$this->mode]}($newValue);
+		}
 	}
 
 	public function in($value) {
-		if (!is_array($value)) throw new \Exception(' `in` can only be used with arrays');
+		if (!is_array($value)) {
+			throw new \Exception(' `in` can only be used with arrays');
+		}
 		$this->result[count($this->result)-1] = in_array($this->result[count($this->result)-1], $value);
 	}
 
@@ -108,7 +111,9 @@ 
 
 	//Postprocessing - replace values with null where allowed, or override a value at position
 	public function postProcess(ValueData $data, $val, $overrideVal, $allowNull) {
-        if ($this->getMode() !== Tokenizer::ARG) return;
+        if ($this->getMode() !== Tokenizer::ARG) {
+        	return;
+        }
 		foreach ($this->getResult() as $i => $value) {
 			if (is_scalar($value)) {
 				$val = ($overrideVal == $val) ? $data->read($value) : $overrideVal;

src/FunctionSet.php

Braces +4 added lines, -3 removed lines

@@ -26,11 +26,12 @@
 	private function getArgs0($name, $args) {
 		if (isset($this->functions[$name]) && !($this->functions[$name] instanceof TSSFunction\Data)) {
 			$tokens = $args[0];
-			if ($tokens->count() == 0) return [];
+			if ($tokens->count() == 0) {
+				return [];
+			}
 			$parser = new \Transphporm\Parser\Value($this);
 			return $parser->parseTokens($tokens, $this->elementData->getData($this->element));
-		}
-		else {
+		} else {
 			return iterator_to_array($args[0]);
 		}
 

src/Property/Content.php

Braces +28 added lines, -13 removed lines

@@ -14,7 +14,9 @@ 
 	}
 
 	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
-		if (!$this->shouldRun($element)) return false;
+		if (!$this->shouldRun($element)) {
+			return false;
+		}
 
 		$values = $this->formatter->format($values, $rules);
 
@@ -22,14 +24,19 @@ 
 			//Remove the current contents
 			$this->removeAllChildren($element);
 			//Now make a text node
-			if ($this->getContentMode($rules) === 'replace') $this->replaceContent($element, $values);
-			else $this->appendContent($element, $values);
+			if ($this->getContentMode($rules) === 'replace') {
+				$this->replaceContent($element, $values);
+			} else {
+				$this->appendContent($element, $values);
+			}
 		}
 	}
 
 	private function shouldRun($element) {
 		do {
-			if ($element->getAttribute('transphporm') == 'includedtemplate') return false;
+			if ($element->getAttribute('transphporm') == 'includedtemplate') {
+				return false;
+			}
 		}
 		while (($element = $element->parentNode) instanceof \DomElement);
 		return true;
@@ -56,9 +63,10 @@ 
 	public function getNode($node, $document) {
 		foreach ($node as $n) {
 			if (is_array($n)) {
-				foreach ($this->getNode($n, $document) as $new) yield $new;
-			}
-			else {
+				foreach ($this->getNode($n, $document) as $new) {
+					yield $new;
+				}
+			} else {
 				yield $this->convertNode($n, $document);
 			}
 		}
@@ -69,9 +77,10 @@ 
 			$new = $document->importNode($node, true);
 			//Removing this might cause problems with caching...
 			//$new->setAttribute('transphporm', 'added');
-		}
-		else {
-			if ($node instanceof \DomText) $node = $node->nodeValue;
+		} else {
+			if ($node instanceof \DomText) {
+				$node = $node->nodeValue;
+			}
 			$new = $document->createElement('text');
 
 			$new->appendChild($document->createTextNode($node));
@@ -82,14 +91,18 @@ 
 
 
 	private function replaceContent($element, $content) {
-		if ($element->getAttribute('transphporm') == 'added') return;
+		if ($element->getAttribute('transphporm') == 'added') {
+			return;
+		}
 		//If this rule was cached, the elements that were added last time need to be removed prior to running the rule again.
 		if ($element->getAttribute('transphporm')) {
 			$this->replaceCachedContent($element);
 		}
 
 		foreach ($this->getNode($content, $element->ownerDocument) as $node) {
-			if (!$node->getAttribute('transphporm'))  $node->setAttribute('transphporm', 'added');
+			if (!$node->getAttribute('transphporm')) {
+				$node->setAttribute('transphporm', 'added');
+			}
 			$element->parentNode->insertBefore($node, $element);
 		}
 
@@ -123,6 +136,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