chillerlan / php-prototype-dom

src/Document.php

Unused Use Statements +4 added lines, -1 removed lines

@@ -12,7 +12,10 @@
 
 namespace chillerlan\PrototypeDOM;
 
-use DOMDocument,DOMNode, DOMNodeList, DOMXPath;
+use DOMDocument;
+use DOMNode;
+use DOMNodeList;
+use DOMXPath;
 use Symfony\Component\CssSelector\CssSelectorConverter;
 
 class Document extends DOMDocument{

Spacing +26 added lines, -26 removed lines

@@ -12,10 +12,10 @@ 
 
 namespace chillerlan\PrototypeDOM;
 
-use DOMDocument,DOMNode, DOMNodeList, DOMXPath;
+use DOMDocument, DOMNode, DOMNodeList, DOMXPath;
 use Symfony\Component\CssSelector\CssSelectorConverter;
 
-class Document extends DOMDocument{
+class Document extends DOMDocument {
 
 	const NODE_CLASSES = [
 		'Attr',
@@ -34,10 +34,10 @@ 
 	 * @param string|null $version
 	 * @param string|null $encoding
 	 */
-	public function __construct(string $content = null, $version = '1.0', $encoding = 'UTF-8'){
+	public function __construct(string $content = null, $version = '1.0', $encoding = 'UTF-8') {
 		parent::__construct($version, $encoding);
 
-		foreach(self::NODE_CLASSES as $nodeClass){
+		foreach (self::NODE_CLASSES as $nodeClass) {
 			$this->registerNodeClass('DOM'.$nodeClass, __NAMESPACE__.'\\'.$nodeClass);
 		}
 
@@ -84,21 +84,21 @@ 
 	 */
 	public function select($selectors = null, DOMNode $contextNode = null, string $axis = 'descendant-or-self::', int $nodeType = XML_ELEMENT_NODE):NodeList{
 
-		if(is_string($selectors)){
+		if (is_string($selectors)) {
 			$selectors = [trim($selectors)];
 		}
 
-		if(!is_array($selectors) || empty($selectors)){
+		if (!is_array($selectors) || empty($selectors)) {
 			$selectors = ['*'];
 		}
 
 		$elements = new NodeList;
 
-		foreach($selectors as $selector){
+		foreach ($selectors as $selector) {
 
-			foreach($this->querySelectorAll($selector, $contextNode, $axis) as $element){
+			foreach ($this->querySelectorAll($selector, $contextNode, $axis) as $element) {
 
-				if($element->nodeType === $nodeType){
+				if ($element->nodeType === $nodeType) {
 					$elements[] = $element;
 				}
 
@@ -144,9 +144,9 @@ 
 	public function removeElementsBySelector($selectors, DOMNode $contextNode = null, string $axis = 'descendant-or-self::'):Document{
 		$nodes = $this->select($selectors, $contextNode, $axis);
 
-		if(count($nodes) > 0){
+		if (count($nodes) > 0) {
 			/** @var \chillerlan\PrototypeDOM\Element $node */
-			foreach($nodes as $node){
+			foreach ($nodes as $node) {
 				$node->remove();
 			}
 
@@ -163,16 +163,16 @@ 
 	 */
 	public function _toNodeList($content):NodeList{
 
-		if($content instanceof NodeList || $content instanceof DOMNodeList || is_array($content)){
+		if ($content instanceof NodeList || $content instanceof DOMNodeList || is_array($content)) {
 			return new NodeList($content);
 		}
-		elseif($content instanceof DOMNode){
+		elseif ($content instanceof DOMNode) {
 			return new NodeList([$content]);
 		}
-		elseif(is_string($content)){
+		elseif (is_string($content)) {
 			return $this->_loadHTMLFragment($content);
 		}
-		else{
+		else {
 			throw new \Exception('invalid content'); // @codeCoverageIgnore
 		}
 
@@ -189,15 +189,15 @@ 
 	public function recursivelyCollect(DOMNode $element, string $property, int $maxLength = -1, int $nodeType = XML_ELEMENT_NODE):NodeList{
 		$nodes = new NodeList;
 
-		if(in_array($property, ['parentNode', 'previousSibling', 'nextSibling'])){
+		if (in_array($property, ['parentNode', 'previousSibling', 'nextSibling'])) {
 
-			while($element = $element->{$property}){
+			while ($element = $element->{$property}) {
 
-				if($element->nodeType === $nodeType){
+				if ($element->nodeType === $nodeType) {
 					$nodes[] = $element;
 				}
 
-				if(count($nodes) === $maxLength){
+				if (count($nodes) === $maxLength) {
 					break;
 				}
 
@@ -217,14 +217,14 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function _recursivelyFind(DOMNode $element, string $property, string $selector = null, int $index = 0, int $nodeType = XML_ELEMENT_NODE){
+	public function _recursivelyFind(DOMNode $element, string $property, string $selector = null, int $index = 0, int $nodeType = XML_ELEMENT_NODE) {
 
-		if(in_array($property, ['parentNode', 'previousSibling', 'nextSibling'])){
+		if (in_array($property, ['parentNode', 'previousSibling', 'nextSibling'])) {
 
 			/** @var \chillerlan\PrototypeDOM\Element $element */
-			while($element = $element->{$property}){
+			while ($element = $element->{$property}) {
 
-				if($element->nodeType !== $nodeType || !is_null($selector) && !$element->match($selector) || --$index >= 0){
+				if ($element->nodeType !== $nodeType || !is_null($selector) && !$element->match($selector) || --$index >= 0) {
 					continue;
 				}
 
@@ -244,9 +244,9 @@ 
 	 */
 	public function match(DOMNode $element, string $selector):bool{
 
-		foreach($this->select($selector) as $match){
+		foreach ($this->select($selector) as $match) {
 
-			if($element->isSameNode($match)){
+			if ($element->isSameNode($match)) {
 				return true;
 			}
 
@@ -265,7 +265,7 @@ 
 		/** @var \chillerlan\PrototypeDOM\Element $element */
 		$element = $this->createElement($tag);
 
-		if($attributes){
+		if ($attributes) {
 			$element->setAttributes($attributes);
 		}
 

src/ManipulationTrait.php

Spacing +14 added lines, -14 removed lines

@@ -17,7 +17,7 @@ 
 /**
  * @extends \DOMNode
  */
-trait ManipulationTrait{
+trait ManipulationTrait {
 
 	/**
 	 * @link http://php.net/manual/class.domnode.php#domnode.props.ownerdocument
@@ -46,7 +46,7 @@ 
 	 */
 	public function remove():DOMNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			/** @var \DOMNode $this */
 			return $this;
 		}
@@ -62,7 +62,7 @@ 
 	 */
 	public function replace(DOMNode $newnode):DOMNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			/** @var \DOMNode $this */
 			return $this;
 		}
@@ -86,7 +86,7 @@ 
 	 */
 	public function empty():DOMNode{
 
-		while($this->hasChildNodes()){
+		while ($this->hasChildNodes()) {
 			$this->firstChild->remove();
 		}
 
@@ -113,10 +113,10 @@ 
 	public function cleanWhitespace():DOMNode{
 		$node = $this->firstChild;
 
-		while($node){
+		while ($node) {
 			$nextNode = $node->nextSibling;
 
-			if($node->nodeType === XML_TEXT_NODE && empty(trim($node->nodeValue))){
+			if ($node->nodeType === XML_TEXT_NODE && empty(trim($node->nodeValue))) {
 				$node->remove();
 			}
 
@@ -149,18 +149,18 @@ 
 	 */
 	public function insert($content):DOMNode{
 
-		if(is_array($content)){
+		if (is_array($content)) {
 
-			foreach(['before', 'after', 'top', 'bottom'] as $pos){
+			foreach (['before', 'after', 'top', 'bottom'] as $pos) {
 
-				if(array_key_exists($pos, $content)){
+				if (array_key_exists($pos, $content)) {
 					$nodes = $this->ownerDocument->_toNodeList($content[$pos]);
 
-					if($pos === 'top' && $this->hasChildNodes() || $pos === 'after' && $this->nextSibling){
+					if ($pos === 'top' && $this->hasChildNodes() || $pos === 'after' && $this->nextSibling) {
 						$nodes->reverse();
 					}
 
-					foreach($nodes as $node){
+					foreach ($nodes as $node) {
 						call_user_func_array([$this, 'insert_'.$pos], [$node]);
 					}
 
@@ -169,8 +169,8 @@ 
 			}
 
 		}
-		else{
-			foreach($this->ownerDocument->_toNodeList($content) as $node){
+		else {
+			foreach ($this->ownerDocument->_toNodeList($content) as $node) {
 				$this->insert_bottom($node);
 			}
 		}
@@ -187,7 +187,7 @@ 
 	 */
 	public function insert_before(DOMNode $node, DOMNode $refNode = null):DOMNode{
 
-		if($this->parentNode){
+		if ($this->parentNode) {
 			$this->parentNode->insertBefore($this->_importNode($node), $refNode ?? $this);
 		}
 

src/TraversalTrait.php

Spacing +13 added lines, -13 removed lines

@@ -17,7 +17,7 @@ 
 /**
  * @extends \DOMNode
  */
-trait TraversalTrait{
+trait TraversalTrait {
 
 	/**
 	 * @link http://php.net/manual/class.domnode.php#domnode.props.ownerdocument
@@ -46,9 +46,9 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function _recursivelyFind($selector, $index, string $property, int $nodeType = XML_ELEMENT_NODE){
+	public function _recursivelyFind($selector, $index, string $property, int $nodeType = XML_ELEMENT_NODE) {
 
-		if(is_numeric($selector)){
+		if (is_numeric($selector)) {
 			$index    = $selector;
 			$selector = null;
 		}
@@ -91,13 +91,13 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function down($expression = null, int $index = 0){
+	public function down($expression = null, int $index = 0) {
 
-		if(count(func_get_args()) === 0){
+		if (count(func_get_args()) === 0) {
 			return $this->firstDescendant();
 		}
 
-		if(is_numeric($expression)){
+		if (is_numeric($expression)) {
 			$index      = $expression;
 			$expression = '*';
 		}
@@ -111,7 +111,7 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function up($expression = null, int $index = null){
+	public function up($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'parentNode');
 	}
 
@@ -121,7 +121,7 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function previous($expression = null, int $index = null){
+	public function previous($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'previousSibling');
 	}
 
@@ -131,7 +131,7 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function next($expression = null, int $index = null){
+	public function next($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'nextSibling');
 	}
 
@@ -143,11 +143,11 @@ 
 	public function childElements(int $nodeType = XML_ELEMENT_NODE):NodeList{
 		$children = new NodeList;
 
-		if($this->hasChildNodes()){
+		if ($this->hasChildNodes()) {
 
-			foreach($this->childNodes as $child){
+			foreach ($this->childNodes as $child) {
 
-				if($child->nodeType === $nodeType){
+				if ($child->nodeType === $nodeType) {
 					$children[] = $child;
 				}
 
@@ -191,7 +191,7 @@ 
 	/**
 	 * @return \DOMNode|null
 	 */
-	public function firstDescendant(){
+	public function firstDescendant() {
 		return $this->descendants()->first();
 	}
 

src/NodeList.php

Spacing +15 added lines, -15 removed lines

@@ -36,15 +36,15 @@ 
 	 *
 	 * @throws \Exception
 	 */
-	public function __construct($content = null){
+	public function __construct($content = null) {
 
-		if($content instanceof NodeList || $content instanceof DOMNodeList){
+		if ($content instanceof NodeList || $content instanceof DOMNodeList) {
 			$this->nodelist = iterator_to_array($content);
 		}
-		elseif(is_array($content)){
+		elseif (is_array($content)) {
 			$this->nodelist = $content;
 		}
-		else{
+		else {
 			$this->nodelist = [];
 		}
 
@@ -57,9 +57,9 @@ 
 	 */
 	public function match(DOMNode $node):bool{
 
-		foreach($this->nodelist as $element){
+		foreach ($this->nodelist as $element) {
 
-			if($element->isSameNode($node)){
+			if ($element->isSameNode($node)) {
 				return true;
 			}
 
@@ -80,14 +80,14 @@ 
 	/**
 	 * @return \DOMNode|null
 	 */
-	public function first(){
+	public function first() {
 		return $this->nodelist[0] ?? null;
 	}
 
 	/**
 	 * @return \DOMNode|null
 	 */
-	public function last(){
+	public function last() {
 		return $this->nodelist[$this->count() - 1] ?? null;
 	}
 
@@ -157,14 +157,14 @@ 
 	/**
 	 *  @return void
 	 */
-	public function next(){
+	public function next() {
 		$this->key++;
 	}
 
 	/**
 	 * @return void
 	 */
-	public function rewind(){
+	public function rewind() {
 		$this->key = 0;
 	}
 
@@ -187,7 +187,7 @@ 
 	 *
 	 * @return \DOMNode|null
 	 */
-	public function offsetGet($offset){
+	public function offsetGet($offset) {
 		return $this->item($offset);
 	}
 
@@ -197,12 +197,12 @@ 
 	 *
 	 * @return void
 	 */
-	public function offsetSet($offset, $value){
+	public function offsetSet($offset, $value) {
 
-		if(is_int($offset)){
+		if (is_int($offset)) {
 			$this->nodelist[$offset] = $value;
 		}
-		else{
+		else {
 			$this->nodelist[] = $value;
 		}
 
@@ -213,7 +213,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function offsetUnset($offset){
+	public function offsetUnset($offset) {
 		unset($this->nodelist[$offset]);
 	}
 

Braces +3 added lines, -6 removed lines

@@ -40,11 +40,9 @@ 
 
 		if($content instanceof NodeList || $content instanceof DOMNodeList){
 			$this->nodelist = iterator_to_array($content);
-		}
-		elseif(is_array($content)){
+		} elseif(is_array($content)){
 			$this->nodelist = $content;
-		}
-		else{
+		} else{
 			$this->nodelist = [];
 		}
 
@@ -201,8 +199,7 @@ 
 
 		if(is_int($offset)){
 			$this->nodelist[$offset] = $value;
-		}
-		else{
+		} else{
 			$this->nodelist[] = $value;
 		}