arthurkushman / querypath

src/QueryPathIterator.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
 	public function current()
 	{
-		if (! isset($this->qp)) {
+		if (!isset($this->qp)) {
 			$this->qp = QueryPath::with(parent::current(), null, $this->options);
 		} else {
 			$splos = new SplObjectStorage();

src/DOMQuery.php

Spacing +19 added lines, -19 removed lines

@@ -196,7 +196,7 @@ 
 		$xpath = new DOMXPath($this->document);
 
 		// Register a default namespace.
-		if (! empty($options['namespace_prefix']) && ! empty($options['namespace_uri'])) {
+		if (!empty($options['namespace_prefix']) && !empty($options['namespace_uri'])) {
 			$xpath->registerNamespace($options['namespace_prefix'], $options['namespace_uri']);
 		}
 
@@ -286,7 +286,7 @@ 
 			return ($this->count() > $index) ? $this->getNthMatch($index) : null;
 		}
 		// Retain support for legacy.
-		if (! $asObject) {
+		if (!$asObject) {
 			$matches = [];
 			foreach ($this->matches as $m) {
 				$matches[] = $m;
@@ -399,7 +399,7 @@ 
 			$matches = [];
 			preg_match($regex, $data, $matches);
 
-			if (! empty($matches)) {
+			if (!empty($matches)) {
 				$result = [
 					'mime' => $matches[1] . '/' . $matches[2],
 					'data' => base64_decode($matches[3]),
@@ -685,12 +685,12 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
 		// Added by eabrand.
-		if (! $first->ownerDocument->documentElement) {
+		if (!$first->ownerDocument->documentElement) {
 			return null;
 		}
 
@@ -735,12 +735,12 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
 		// Added by eabrand.
-		if (! $first->ownerDocument->documentElement) {
+		if (!$first->ownerDocument->documentElement) {
 			return null;
 		}
 
@@ -811,11 +811,11 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
-		if (! $first->hasChildNodes()) {
+		if (!$first->hasChildNodes()) {
 			return '';
 		}
 
@@ -850,11 +850,11 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
-		if (! $first->hasChildNodes()) {
+		if (!$first->hasChildNodes()) {
 			return '';
 		}
 
@@ -882,11 +882,11 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
-		if (! $first->hasChildNodes()) {
+		if (!$first->hasChildNodes()) {
 			return '';
 		}
 
@@ -1030,7 +1030,7 @@ 
 		foreach ($this->matches as $m) {
 			$p = $m;
 			while (isset($p->previousSibling) && $p->previousSibling->nodeType === XML_TEXT_NODE) {
-				$p      = $p->previousSibling;
+				$p = $p->previousSibling;
 				$buffer .= $p->textContent;
 			}
 		}
@@ -1049,7 +1049,7 @@ 
 		foreach ($this->matches as $m) {
 			$n = $m;
 			while (isset($n->nextSibling) && $n->nextSibling->nodeType === XML_TEXT_NODE) {
-				$n      = $n->nextSibling;
+				$n = $n->nextSibling;
 				$buffer .= $n->textContent;
 			}
 		}
@@ -1134,7 +1134,7 @@ 
 		// Only return the first item -- that's what JQ does.
 		$first = $this->getFirstMatch();
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
@@ -1217,7 +1217,7 @@ 
 		$first = $this->getFirstMatch();
 
 		// Catch cases where first item is not a legit DOM object.
-		if (! ($first instanceof DOMNode)) {
+		if (!($first instanceof DOMNode)) {
 			return null;
 		}
 
@@ -1524,7 +1524,7 @@ 
 	public function __call($name, $arguments)
 	{
 
-		if (! ExtensionRegistry::$useRegistry) {
+		if (!ExtensionRegistry::$useRegistry) {
 			throw new Exception("No method named $name found (Extensions disabled).");
 		}
 
@@ -1544,7 +1544,7 @@ 
 		}
 
 		// Note that an empty ext registry indicates that extensions are disabled.
-		if (! empty($this->ext) && ExtensionRegistry::hasMethod($name)) {
+		if (!empty($this->ext) && ExtensionRegistry::hasMethod($name)) {
 			$owner  = ExtensionRegistry::getMethodClass($name);
 			$method = new ReflectionMethod($owner, $name);
 

src/DOM.php

Spacing +8 added lines, -8 removed lines

@@ -84,7 +84,7 @@ 
 		$this->options = $options + Options::get() + $this->options;
 
 		$parser_flags = $options['parser_flags'] ?? self::DEFAULT_PARSER_FLAGS;
-		if (! empty($this->options['ignore_parser_warnings'])) {
+		if (!empty($this->options['ignore_parser_warnings'])) {
 			// Don't convert parser warnings into exceptions.
 			$this->errTypes = 257; //E_ERROR | E_USER_ERROR;
 		} elseif (isset($this->options['exception_level'])) {
@@ -109,7 +109,7 @@ 
 				$this->matches = $document;
 				if ($document->count() !== 0) {
 					$first = $this->getFirstMatch();
-					if (! empty($first->ownerDocument)) {
+					if (!empty($first->ownerDocument)) {
 						$this->document = $first->ownerDocument;
 					}
 				}
@@ -140,7 +140,7 @@ 
 			}
 		} elseif (is_array($document)) {
 			//trigger_error('Detected deprecated array support', E_USER_NOTICE);
-			if (! empty($document) && $document[0] instanceof DOMNode) {
+			if (!empty($document) && $document[0] instanceof DOMNode) {
 				$found = new SplObjectStorage();
 				foreach ($document as $item) {
 					$found->attach($item);
@@ -199,13 +199,13 @@ 
 
 			// This is to avoid cases where low ascii digits have slipped into HTML.
 			// AFAIK, it should not adversly effect UTF-8 documents.
-			if (! empty($this->options['strip_low_ascii'])) {
+			if (!empty($this->options['strip_low_ascii'])) {
 				$string = filter_var($string, FILTER_UNSAFE_RAW, FILTER_FLAG_ENCODE_LOW);
 			}
 
 			// Allow users to override parser settings.
 			$useParser = '';
-			if (! empty($this->options['use_parser'])) {
+			if (!empty($this->options['use_parser'])) {
 				$useParser = strtolower($this->options['use_parser']);
 			}
 
@@ -297,10 +297,10 @@ 
 	protected function deepestNode(DOMNode $ele, $depth = 0, $current = null, &$deepest = null)
 	{
 		// FIXME: Should this use SplObjectStorage?
-		if (! isset($current)) {
+		if (!isset($current)) {
 			$current = [$ele];
 		}
-		if (! isset($deepest)) {
+		if (!isset($deepest)) {
 			$deepest = $depth;
 		}
 		if ($ele->hasChildNodes()) {
@@ -436,7 +436,7 @@ 
 
 		// If a context is specified, we basically have to do the reading in
 		// two steps:
-		if (! empty($context)) {
+		if (!empty($context)) {
 			try {
 				set_error_handler(['\QueryPath\ParseException', 'initializeFromError'], $this->errTypes);
 				$contents = file_get_contents($filename, false, $context);

src/Extension/QPXML.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 	{
 		$doc = $this->qp->branch()->top()->get(0)->ownerDocument;
 
-		if (! $doc->schemaValidate($file)) {
+		if (!$doc->schemaValidate($file)) {
 			throw new Exception('Document did not validate against the schema.');
 		}
 	}

src/Extension/QPXSL.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 	 */
 	public function xslt($style)
 	{
-		if (! ($style instanceof QueryPath)) {
+		if (!($style instanceof QueryPath)) {
 			$style = QueryPath::with($style);
 		}
 		$sourceDoc = $this->src->top()->get(0)->ownerDocument;

src/Extension/Format.php

Spacing +7 added lines, -7 removed lines

@@ -92,11 +92,11 @@ 
 			array_shift($args);
 		}
 
-		$getter = function ($qp) {
+		$getter = function($qp) {
 			return $qp->text();
 		};
 
-		$setter = function ($qp, $value) {
+		$setter = function($qp, $value) {
 			$qp->text($value);
 		};
 
@@ -145,11 +145,11 @@ 
 			$args = array_slice(func_get_args(), 2);
 		}
 
-		$getter = function ($qp) use ($attrName) {
+		$getter = function($qp) use ($attrName) {
 			return $qp->attr($attrName);
 		};
 
-		$setter = function ($qp, $value) use ($attrName) {
+		$setter = function($qp, $value) use ($attrName) {
 			return $qp->attr($attrName, $value);
 		};
 
@@ -168,7 +168,7 @@ 
 	protected function forAll($callback, $args, $getter, $setter): Query
 	{
 		[$callback, $pos] = $this->prepareCallback($callback);
-		if (! is_callable($callback)) {
+		if (!is_callable($callback)) {
 			throw new Exception('Callback is not callable.');
 		}
 
@@ -190,7 +190,7 @@ 
 	{
 		if (is_string($callback)) {
 			[$callback, $trail] = $this->splitFunctionName($callback);
-			$pos = (int) $trail;
+			$pos = (int)$trail;
 		} elseif (is_array($callback) && isset($callback[2])) {
 			$pos      = $callback[2];
 			$callback = [$callback[0], $callback[1]];
@@ -220,7 +220,7 @@ 
 	 */
 	protected function prepareArgs($args, $pos): array
 	{
-		$padded = array_pad((array) $args, (0 < $pos) ? $pos - 1 : 0, null);
+		$padded = array_pad((array)$args, (0 < $pos) ? $pos - 1 : 0, null);
 		array_splice($padded, $pos, 0, [null]); // insert null as a place holder
 
 		return $padded;

src/CSS/DOMTraverser/PseudoClass.php

Spacing +10 added lines, -10 removed lines

@@ -79,10 +79,10 @@ 
 				return false;
 			case 'indeterminate':
 				// Because sometimes screwing with people is fun.
-				return (boolean) mt_rand(0, 1);
+				return (boolean)mt_rand(0, 1);
 			case 'lang':
 				// No value = exception.
-				if (! isset($value)) {
+				if (!isset($value)) {
 					throw new NotImplementedException(':lang() requires a value.');
 				}
 
@@ -128,17 +128,17 @@ 
 			// Additional pseudo-classes defined in jQuery:
 			case 'lt':
 				// I'm treating this as "less than or equal to".
-				$rule = sprintf('-n + %d', (int) $value);
+				$rule = sprintf('-n + %d', (int)$value);
 
 				// $rule = '-n+15';
 				return $this->isNthChild($node, $rule);
 			case 'gt':
 				// I'm treating this as "greater than"
 				// return $this->nodePositionFromEnd($node) > (int) $value;
-				return $this->nodePositionFromStart($node) > (int) $value;
+				return $this->nodePositionFromStart($node) > (int)$value;
 			case 'nth':
 			case 'eq':
-				$rule = (int) $value;
+				$rule = (int)$value;
 
 				return $this->isNthChild($node, $rule);
 			case 'first':
@@ -153,7 +153,7 @@ 
 			case 'empty':
 				return $this->isEmpty($node);
 			case 'parent':
-				return ! $this->isEmpty($node);
+				return !$this->isEmpty($node);
 
 			case 'enabled':
 			case 'disabled':
@@ -368,7 +368,7 @@ 
 	 */
 	protected function isNot($node, $selector): bool
 	{
-		return ! $this->has($node, $selector);
+		return !$this->has($node, $selector);
 	}
 
 	/**
@@ -380,7 +380,7 @@ 
 		$tag = $node->tagName;
 		while (isset($node->previousSibling)) {
 			$node = $node->previousSibling;
-			if ($node->nodeType === XML_ELEMENT_NODE && ( ! $byType || $node->tagName === $tag)) {
+			if ($node->nodeType === XML_ELEMENT_NODE && (!$byType || $node->tagName === $tag)) {
 				++$i;
 			}
 		}
@@ -402,7 +402,7 @@ 
 		$tag = $node->tagName;
 		while (isset($node->nextSibling)) {
 			$node = $node->nextSibling;
-			if ($node->nodeType === XML_ELEMENT_NODE && ( ! $byType || $node->tagName === $tag)) {
+			if ($node->nodeType === XML_ELEMENT_NODE && (!$byType || $node->tagName === $tag)) {
 				++$i;
 			}
 		}
@@ -470,7 +470,7 @@ 
 
 	protected function isLocalLink($node): bool
 	{
-		if (! $node->hasAttribute('href')) {
+		if (!$node->hasAttribute('href')) {
 			return false;
 		}
 		$url    = $node->getAttribute('href');

src/CSS/DOMTraverser/Util.php

Spacing +6 added lines, -6 removed lines

@@ -26,7 +26,7 @@ 
 	 */
 	public static function matchesAttribute($node, $name, $value = null, $operation = EventHandler::IS_EXACTLY): bool
 	{
-		if (! $node->hasAttribute($name)) {
+		if (!$node->hasAttribute($name)) {
 			return false;
 		}
 
@@ -47,7 +47,7 @@ 
 		$value = null,
 		$operation = EventHandler::IS_EXACTLY
 	) {
-		if (! $node->hasAttributeNS($nsuri, $name)) {
+		if (!$node->hasAttributeNS($nsuri, $name)) {
 			return false;
 		}
 
@@ -134,7 +134,7 @@ 
 		}
 
 		if (is_numeric($rule)) {
-			return [0, (int) $rule];
+			return [0, (int)$rule];
 		}
 
 		$regex   = '/^\s*([+\-]?[0-9]*)n\s*([+\-]?)\s*([0-9]*)\s*$/';
@@ -142,7 +142,7 @@ 
 		$res     = preg_match($regex, $rule, $matches);
 
 		// If it doesn't parse, return 0, 0.
-		if (! $res) {
+		if (!$res) {
 			return [0, 0];
 		}
 
@@ -150,12 +150,12 @@ 
 		if ($aVal === '-') {
 			$aVal = -1;
 		} else {
-			$aVal = (int) $aVal;
+			$aVal = (int)$aVal;
 		}
 
 		$bVal = 0;
 		if (isset($matches[3])) {
-			$bVal = (int) $matches[3];
+			$bVal = (int)$matches[3];
 			if (isset($matches[2]) && $matches[2] === '-') {
 				$bVal *= -1;
 			}

src/CSS/Scanner.php

Spacing +4 added lines, -4 removed lines

@@ -233,14 +233,14 @@ 
 			// The second conjunct is probably not necessary.
 			while ($this->token !== false && $this->token > -1) {
 				//print "Char: $this->value \n";
-				if ($this->token == Token::BSLASH && ! $escape) {
+				if ($this->token == Token::BSLASH && !$escape) {
 					// XXX: The backslash (\) is removed here.
 					// Turn on escaping.
 					//$buf .= $this->value;
 					$escape = true;
 				} elseif ($escape) {
 					// Turn off escaping
-					$buf    .= $this->value;
+					$buf .= $this->value;
 					$escape = false;
 				} elseif ($this->token === $end) {
 					// At end of string; skip token and break.
@@ -270,14 +270,14 @@ 
 			// The second conjunct is probably not necessary.
 			while ($this->token !== false && $this->token > -1) {
 				//print "Char: $this->value \n";
-				if ($this->token === Token::BSLASH && ! $escape) {
+				if ($this->token === Token::BSLASH && !$escape) {
 					// XXX: The backslash (\) is removed here.
 					// Turn on escaping.
 					//$buf .= $this->value;
 					$escape = true;
 				} elseif ($escape) {
 					// Turn off escaping
-					$buf    .= $this->value;
+					$buf .= $this->value;
 					$escape = false;
 				} // Allow nested pseudoclasses.
 				elseif ($this->token === Token::LPAREN) {