arthurkushman / querypath

src/CSS/QueryPathEventHandler.php

Spacing +19 added lines, -19 removed lines

@@ -254,11 +254,11 @@ 
 				$found->attach($item);
 			}
 
-			if (! empty($nsuri)) {
+			if (!empty($nsuri)) {
 				$nl = $item->getElementsByTagNameNS($nsuri, $lname);
 				// If something is found, merge them:
 				//if (!empty($nl)) $found = array_merge($found, $this->nodeListToArray($nl));
-				if (! empty($nl)) {
+				if (!empty($nl)) {
 					$this->attachNodeList($nl, $found);
 				}
 			} else {
@@ -301,7 +301,7 @@ 
 		//$this->findAnyElement = TRUE;
 		$nsuri = $this->dom->lookupNamespaceURI($ns);
 		$found = new SplObjectStorage();
-		if (! empty($nsuri)) {
+		if (!empty($nsuri)) {
 			$matches = $this->candidateList();
 			foreach ($matches as $item) {
 				if ($item instanceof DOMNode && $nsuri == $item->namespaceURI) {
@@ -309,7 +309,7 @@ 
 				}
 			}
 		}
-		$this->matches        = $found;//UniqueElementList::get($found);
+		$this->matches        = $found; //UniqueElementList::get($found);
 		$this->findAnyElement = false;
 	}
 
@@ -327,7 +327,7 @@ 
 			}
 		}
 
-		$this->matches        = $found;//UniqueElementList::get($found);
+		$this->matches        = $found; //UniqueElementList::get($found);
 		$this->findAnyElement = false;
 	}
 
@@ -439,7 +439,7 @@ 
 			break;
 			case 'lang':
 				// No value = exception.
-				if (! isset($value)) {
+				if (!isset($value)) {
 					throw new NotImplementedException("No handler for lang pseudoclass without value.");
 				}
 				$this->lang($value);
@@ -543,7 +543,7 @@ 
 				$matches = $this->candidateList();
 				$found   = new SplObjectStorage();
 				foreach ($matches as $match) {
-					if (! empty($match->firstChild)) {
+					if (!empty($match->firstChild)) {
 						$found->attach($match);
 					}
 				}
@@ -726,7 +726,7 @@ 
 		} elseif ($rule == 'n') {
 			return [1, 0];
 		} elseif (is_numeric($rule)) {
-			return [0, (int) $rule];
+			return [0, (int)$rule];
 		}
 
 		$rule = explode('n', $rule);
@@ -736,9 +736,9 @@ 
 
 		// Each of these is legal: 1, -1, and -. '-' is shorthand for -1.
 		$aVal = trim($rule[0]);
-		$aVal = ($aVal == '-') ? -1 : (int) $aVal;
+		$aVal = ($aVal == '-') ? -1 : (int)$aVal;
 
-		$bVal = ! empty($rule[1]) ? (int) trim($rule[1]) : 0;
+		$bVal = !empty($rule[1]) ? (int)trim($rule[1]) : 0;
 
 		return [$aVal, $bVal];
 	}
@@ -769,7 +769,7 @@ 
 			// Build up an array of all of children of this parent, and store the
 			// index of each element for reference later. We only need to do this
 			// once per parent, though.
-			if (! $parents->contains($parent)) {
+			if (!$parents->contains($parent)) {
 				$c = 0;
 				foreach ($parent->childNodes as $child) {
 					// We only want nodes, and if this call is preceded by an element
@@ -962,7 +962,7 @@ 
 			// Build up an array of all of children of this parent, and store the
 			// index of each element for reference later. We only need to do this
 			// once per parent, though.
-			if (! $parents->contains($parent)) {
+			if (!$parents->contains($parent)) {
 				$c = 0;
 				foreach ($parent->childNodes as $child) {
 					// This doesn't totally make sense, since the CSS 3 spec does not require that
@@ -1111,7 +1111,7 @@ 
 			$parent = $item->parentNode;
 			foreach ($parent->childNodes as $kid) {
 				if ($kid->nodeType == XML_ELEMENT_NODE && $kid->tagName == $type) {
-					if (! $found->contains($kid)) {
+					if (!$found->contains($kid)) {
 						$found->attach($kid);
 					}
 					break;
@@ -1134,7 +1134,7 @@ 
 			for ($i = $parent->childNodes->length - 1; $i >= 0; --$i) {
 				$kid = $parent->childNodes->item($i);
 				if ($kid->nodeType == XML_ELEMENT_NODE && $kid->tagName == $type) {
-					if (! $found->contains($kid)) {
+					if (!$found->contains($kid)) {
 						$found->attach($kid);
 					}
 					break;
@@ -1200,7 +1200,7 @@ 
 		$matches = $this->candidateList();
 		$found   = new SplObjectStorage();
 		foreach ($matches as $item) {
-			if (! $item->parentNode) {
+			if (!$item->parentNode) {
 				$this->matches = new SplObjectStorage();
 			}
 			$parent     = $item->parentNode;
@@ -1276,11 +1276,11 @@ 
 				foreach ($matches as $item) {
 					$str   = $item->textContent;
 					$lines = explode("\n", $str);
-					if (! empty($lines)) {
+					if (!empty($lines)) {
 						$line = trim($lines[0]);
-						if (! empty($line)) {
+						if (!empty($line)) {
 							$o->textContent = $line;
-							$found->attach($o);//trim($lines[0]);
+							$found->attach($o); //trim($lines[0]);
 						}
 					}
 				}
@@ -1294,7 +1294,7 @@ 
 				$o       = new stdClass();
 				foreach ($matches as $item) {
 					$str = $item->textContent;
-					if (! empty($str)) {
+					if (!empty($str)) {
 						$str            = substr($str, 0, 1);
 						$o->textContent = $str;
 						$found->attach($o);

src/CSS/DOMTraverser.php

Spacing +17 added lines, -17 removed lines

@@ -88,9 +88,9 @@ 
 			$splos->rewind();
 			$first = $splos->current();
 			if ($first instanceof DOMDocument) {
-				$this->dom = $first;//->documentElement;
+				$this->dom = $first; //->documentElement;
 			} else {
-				$this->dom = $first->ownerDocument;//->documentElement;
+				$this->dom = $first->ownerDocument; //->documentElement;
 			}
 
 			$this->scopeNode = $scopeNode;
@@ -297,7 +297,7 @@ 
 	 */
 	public function combineAdjacent($node, $selectors, $index)
 	{
-		while (! empty($node->previousSibling)) {
+		while (!empty($node->previousSibling)) {
 			$node = $node->previousSibling;
 			if ($node->nodeType == XML_ELEMENT_NODE) {
 				//$this->debug(sprintf('Testing %s against "%s"', $node->tagName, $selectors[$index]));
@@ -327,7 +327,7 @@ 
 	 */
 	public function combineSibling($node, $selectors, $index)
 	{
-		while (! empty($node->previousSibling)) {
+		while (!empty($node->previousSibling)) {
 			$node = $node->previousSibling;
 			if ($node->nodeType == XML_ELEMENT_NODE && $this->matchesSimpleSelector($node, $selectors, $index)) {
 				return true;
@@ -383,7 +383,7 @@ 
 	 */
 	public function combineAnyDescendant($node, $selectors, $index)
 	{
-		while (! empty($node->parentNode)) {
+		while (!empty($node->parentNode)) {
 			$node = $node->parentNode;
 
 			// Catch case where element is child of something
@@ -429,16 +429,16 @@ 
 		// this should give us only a single matched element
 		// to work with.
 		if (/*$element == '*' &&*/
-		! empty($selector->id)) {
+		!empty($selector->id)) {
 			$initialMatches = $this->initialMatchOnID($selector, $matches);
 		} // If a namespace is set, find the namespace matches.
-		elseif (! empty($selector->ns)) {
+		elseif (!empty($selector->ns)) {
 			$initialMatches = $this->initialMatchOnElementNS($selector, $matches);
 		}
 		// If the element is a wildcard, using class can
 		// substantially reduce the number of elements that
 		// we start with.
-		elseif ($element === '*' && ! empty($selector->classes)) {
+		elseif ($element === '*' && !empty($selector->classes)) {
 			$initialMatches = $this->initialMatchOnClasses($selector, $matches);
 		} else {
 			$initialMatches = $this->initialMatchOnElement($selector, $matches);
@@ -467,7 +467,7 @@ 
 
 		// Issue #145: DOMXPath will through an exception if the DOM is
 		// not set.
-		if (! ($this->dom instanceof DOMDocument)) {
+		if (!($this->dom instanceof DOMDocument)) {
 			return $found;
 		}
 		$baseQuery = ".//*[@id='{$id}']";
@@ -481,7 +481,7 @@ 
 			}
 
 			$nl = $this->initialXpathQuery($xpath, $node, $baseQuery);
-			if (! empty($nl) && $nl instanceof DOMNodeList) {
+			if (!empty($nl) && $nl instanceof DOMNodeList) {
 				$this->attachNodeList($nl, $found);
 			}
 		}
@@ -511,7 +511,7 @@ 
 
 		// Issue #145: DOMXPath will through an exception if the DOM is
 		// not set.
-		if (! ($this->dom instanceof DOMDocument)) {
+		if (!($this->dom instanceof DOMDocument)) {
 			return $found;
 		}
 		$baseQuery = './/*[@class]';
@@ -593,7 +593,7 @@ 
 				$found->attach($node);
 			}
 			$nl = $node->getElementsByTagName($element);
-			if (! empty($nl) && $nl instanceof DOMNodeList) {
+			if (!empty($nl) && $nl instanceof DOMNodeList) {
 				$this->attachNodeList($nl, $found);
 			}
 		}
@@ -664,7 +664,7 @@ 
 		}
 
 		// Handle namespace.
-		if (! empty($ns) && $ns !== '*') {
+		if (!empty($ns) && $ns !== '*') {
 			// Check whether we have a matching NS URI.
 			$nsuri = $node->lookupNamespaceURI($ns);
 			if (empty($nsuri) || $node->namespaceURI !== $nsuri) {
@@ -726,7 +726,7 @@ 
 			// Namespaced attributes.
 			if (isset($attr['ns']) && $attr['ns'] !== '*') {
 				$nsuri = $node->lookupNamespaceURI($attr['ns']);
-				if (empty($nsuri) || ! $node->hasAttributeNS($nsuri, $attr['name'])) {
+				if (empty($nsuri) || !$node->hasAttributeNS($nsuri, $attr['name'])) {
 					return false;
 				}
 				$matches = Util::matchesAttributeNS($node, $attr['name'], $nsuri, $val, $attr['op']);
@@ -746,7 +746,7 @@ 
 				$matches = Util::matchesAttribute($node, $attr['name'], $val, $attr['op']);
 			}
 
-			if (! $matches) {
+			if (!$matches) {
 				return false;
 			}
 		}
@@ -785,7 +785,7 @@ 
 			return true;
 		}
 
-		if (! $node->hasAttribute('class')) {
+		if (!$node->hasAttribute('class')) {
 			return false;
 		}
 
@@ -815,7 +815,7 @@ 
 			$name = $pseudoClass['name'];
 			// Avoid E_STRICT violation.
 			$value = $pseudoClass['value'] ?? null;
-			$ret   &= $this->psHandler->elementMatches($name, $node, $this->scopeNode, $value);
+			$ret &= $this->psHandler->elementMatches($name, $node, $this->scopeNode, $value);
 		}
 
 		return $ret;

src/CSS/InputStream.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 	public function consume()
 	{
 		$ret = array_shift($this->stream);
-		if (! empty($ret)) {
+		if (!empty($ret)) {
 			$this->position++;
 		}
 

src/Helpers/QueryMutators.php

Spacing +3 added lines, -3 removed lines

@@ -661,7 +661,7 @@ 
 					$vals = array_filter(explode(' ', $m->getAttribute('class')));
 					$buf  = [];
 					foreach ($vals as $v) {
-						if (! in_array($v, $to_remove)) {
+						if (!in_array($v, $to_remove)) {
 							$buf[] = $v;
 						}
 					}
@@ -795,7 +795,7 @@ 
 	 */
 	public function remove($selector = null): Query
 	{
-		if (! empty($selector)) {
+		if (!empty($selector)) {
 			// Do a non-destructive find.
 			$query = new QueryPathEventHandler($this->matches);
 			$query->find($selector);
@@ -1041,7 +1041,7 @@ 
 		$css = [];
 		foreach ($this->matches as $match) {
 			$style = $match->getAttribute('style');
-			if (! empty($style)) {
+			if (!empty($style)) {
 				// XXX: Is this sufficient?
 				$style_array = explode(';', $style);
 				foreach ($style_array as $item) {

src/Helpers/QueryChecks.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 				// pass an is() on [a, b, c, d]. We use the $seen SPLOS to prevent this.
 				$seen = new SplObjectStorage();
 				foreach ($selector as $item) {
-					if (! $this->matches->contains($item) || $seen->contains($item)) {
+					if (!$this->matches->contains($item) || $seen->contains($item)) {
 						return false;
 					}
 					$seen->attach($item);
@@ -113,7 +113,7 @@ 
 		// ones we add to our new matches.
 		foreach ($nodes as $original_node) {
 			$node = $original_node;
-			while (! empty($node)/* && $node != $node->ownerDocument*/) {
+			while (!empty($node)/* && $node != $node->ownerDocument*/) {
 				if ($this->matches->contains($node)) {
 					$found->attach($node);
 				}
@@ -167,7 +167,7 @@ 
 	public function hasAttr($attrName): bool
 	{
 		foreach ($this->matches as $match) {
-			if (! $match->hasAttribute($attrName)) {
+			if (!$match->hasAttribute($attrName)) {
 				return false;
 			}
 		}

src/Helpers/QueryFilters.php

Spacing +11 added lines, -11 removed lines

@@ -245,7 +245,7 @@ 
 				if (isset($c)) {
 					if (is_array($c) || $c instanceof Iterable) {
 						foreach ($c as $retval) {
-							if (! is_object($retval)) {
+							if (!is_object($retval)) {
 								$tmp              = new stdClass();
 								$tmp->textContent = $retval;
 								$retval           = $tmp;
@@ -253,7 +253,7 @@ 
 							$found->attach($retval);
 						}
 					} else {
-						if (! is_object($c)) {
+						if (!is_object($c)) {
 							$tmp              = new stdClass();
 							$tmp->textContent = $c;
 							$c                = $tmp;
@@ -653,7 +653,7 @@ 
 				$m = $m->parentNode;
 				// Is there any case where parent node is not an element?
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector) > 0) {
 							break;
 						}
@@ -714,7 +714,7 @@ 
 			}
 		} elseif (is_array($selector)) {
 			foreach ($this->matches as $m) {
-				if (! in_array($m, $selector, true)) {
+				if (!in_array($m, $selector, true)) {
 					$found->attach($m);
 				}
 			}
@@ -726,7 +726,7 @@ 
 			}
 		} else {
 			foreach ($this->matches as $m) {
-				if (! QueryPath::with($m, null, $this->options)->is($selector)) {
+				if (!QueryPath::with($m, null, $this->options)->is($selector)) {
 					$found->attach($m);
 				}
 			}
@@ -802,7 +802,7 @@ 
 				$m = $m->parentNode;
 				// Is there any case where parent node is not an element?
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector) > 0) {
 							$found->attach($m);
 							break;
@@ -842,7 +842,7 @@ 
 				$m = $m->parentNode;
 				// Is there any case where parent node is not an element?
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector) > 0) {
 							$found->attach($m);
 						}
@@ -882,7 +882,7 @@ 
 			while (isset($m->nextSibling)) {
 				$m = $m->nextSibling;
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector) > 0) {
 							$found->attach($m);
 							break;
@@ -924,7 +924,7 @@ 
 			while (isset($m->nextSibling)) {
 				$m = $m->nextSibling;
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector) > 0) {
 							$found->attach($m);
 						}
@@ -965,7 +965,7 @@ 
 			while (isset($m->previousSibling)) {
 				$m = $m->previousSibling;
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector)) {
 							$found->attach($m);
 							break;
@@ -1007,7 +1007,7 @@ 
 			while (isset($m->previousSibling)) {
 				$m = $m->previousSibling;
 				if ($m->nodeType === XML_ELEMENT_NODE) {
-					if (! empty($selector)) {
+					if (!empty($selector)) {
 						if (QueryPath::with($m, null, $this->options)->is($selector)) {
 							$found->attach($m);
 						}

src/IOException.php

Spacing +1 added lines, -1 removed lines

@@ -18,6 +18,6 @@
 	public static function initializeFromError($errno, $errstr, $errfile, $errline, $context = null)
 	{
 		$class = __CLASS__;
-		throw new $class($errno, (int) $errstr, $errfile, $errline);
+		throw new $class($errno, (int)$errstr, $errfile, $errline);
 	}
 }

src/ParseException.php

Spacing +1 added lines, -1 removed lines

@@ -45,6 +45,6 @@
 	public static function initializeFromError($errno, $errstr, $errfile, $errline, $context = null)
 	{
 		$class = __CLASS__;
-		throw new $class($errno, (int) $errstr, $errfile, $errline);
+		throw new $class($errno, (int)$errstr, $errfile, $errline);
 	}
 }

examples/docx.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
 
 function readZippedXML($archiveFile, $dataFile)
 {
-	if (! class_exists('ZipArchive', false)) {
+	if (!class_exists('ZipArchive', false)) {
 		return "ZipArchive Class Doesn't Exist.";
 	}
 	// Create new ZIP archive