chillerlan / php-prototype-dom

src/TraversalTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -39,8 +39,8 @@
 	}
 
 	/**
-	 * @param        $selector
-	 * @param        $index
+	 * @param        string|null $selector
+	 * @param        null|integer $index
 	 * @param string $property
 	 * @param int    $nodeType
 	 *

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/Attr.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
 
 use DOMAttr;
 
-class Attr extends DOMAttr{}
+class Attr extends DOMAttr {}

src/Comment.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
 
 use DOMComment;
 
-class Comment extends DOMComment{}
+class Comment extends DOMComment {}

src/DocumentFragment.php

Spacing +1 added lines, -1 removed lines

@@ -16,6 +16,6 @@
 
 /**
  */
-class DocumentFragment extends DOMDocumentFragment{
+class DocumentFragment extends DOMDocumentFragment {
 	use TraversalTrait, ManipulationTrait;
 }

src/Document.php

Braces +3 added lines, -6 removed lines

@@ -157,14 +157,11 @@
 
 		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
 		}
 

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/Text.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 use DOMText;
 
-class Text extends DOMText{
+class Text extends DOMText {
 	use ManipulationTrait, TraversalTrait;
 
 }

src/Element.php

Spacing +22 added lines, -22 removed lines

@@ -14,7 +14,7 @@ 
 
 use DOMElement;
 
-class Element extends DOMElement{
+class Element extends DOMElement {
 	use ManipulationTrait, TraversalTrait;
 
 	/**
@@ -25,7 +25,7 @@ 
 	public function id(string $newID = null):string {
 		$oldID = $this->getAttribute('id');
 
-		if(!is_null($newID)){
+		if (!is_null($newID)) {
 			$this->setAttribute('id', $newID);
 		}
 
@@ -38,14 +38,14 @@ 
 	public function getClassNames():array{
 		$currentClassnames = [];
 
-		if($this->hasAttributes()){
+		if ($this->hasAttributes()) {
 			$classnames = explode(' ', trim($this->getAttribute('class')));
 
-			if(!empty($classnames)){
+			if (!empty($classnames)) {
 
-				foreach($classnames as $classname){
+				foreach ($classnames as $classname) {
 
-					if(empty($classname)){
+					if (empty($classname)) {
 						continue;
 					}
 
@@ -76,9 +76,9 @@ 
 	public function addClassNames(array $classnames):Element{
 		$currentClassnames = $this->getClassNames();
 
-		foreach($classnames as $classname){
+		foreach ($classnames as $classname) {
 
-			if(!in_array($classname, $currentClassnames, true)){
+			if (!in_array($classname, $currentClassnames, true)) {
 				array_push($currentClassnames, $classname);
 			}
 
@@ -97,12 +97,12 @@ 
 	public function removeClassNames(array $classnames):Element{
 		$currentClassnames = $this->getClassNames();
 
-		foreach($classnames as $classname){
+		foreach ($classnames as $classname) {
 			$keys = array_keys($currentClassnames, $classname);
 
-			if(!empty($keys)){
+			if (!empty($keys)) {
 
-				foreach($keys as $key){
+				foreach ($keys as $key) {
 					unset($currentClassnames[$key]);
 				}
 
@@ -151,15 +151,15 @@ 
 	public function getStyle():array{
 		$currentStyle = [];
 
-		if($this->hasAttributes()){
+		if ($this->hasAttributes()) {
 			$styles = explode(';', trim($this->getAttribute('style')));
 
-			if(!empty($styles)){
+			if (!empty($styles)) {
 
-				foreach($styles as $style){
+				foreach ($styles as $style) {
 					$s = explode(':', $style);
 
-					if(count($s) === 2){
+					if (count($s) === 2) {
 						$currentStyle[strtolower(trim($s[0]))] = trim($s[1]);
 					}
 
@@ -177,10 +177,10 @@ 
 	 *
 	 * @return bool|string
 	 */
-	public function hasStyle(string $property){
+	public function hasStyle(string $property) {
 		$currentStyle = $this->getStyle();
 
-		if(array_key_exists(strtolower($property), $currentStyle)){
+		if (array_key_exists(strtolower($property), $currentStyle)) {
 			return $currentStyle[$property];
 		}
 
@@ -196,11 +196,11 @@ 
 	public function setStyle(array $style, bool $replace = false):Element{
 		$currentStyle = $this->getStyle();
 
-		if(!$replace){
+		if (!$replace) {
 			$style = array_merge($currentStyle, $style);
 		}
 
-		foreach($style as $property => $value){
+		foreach ($style as $property => $value) {
 			$style[$property] = $property.': '.$value.';';
 		}
 
@@ -215,7 +215,7 @@ 
 	public function getAttributes():array{
 		$attributes = [];
 
-		foreach($this->attributes as $attribute){
+		foreach ($this->attributes as $attribute) {
 			$attributes[$attribute->nodeName] = $attribute->nodeValue;
 		}
 
@@ -229,7 +229,7 @@ 
 	 */
 	public function setAttributes(array $attributes):Element{
 
-		foreach($attributes as $name => $value){
+		foreach ($attributes as $name => $value) {
 			$this->setAttribute($name, $value);
 		}
 
@@ -243,7 +243,7 @@ 
 	 */
 	public function removeAttributes(array $attributes):Element{
 
-		foreach($attributes as $name){
+		foreach ($attributes as $name) {
 			$this->removeAttribute($name);
 		}
 

src/ManipulationTrait.php

Braces +1 added lines, -2 removed lines

@@ -168,8 +168,7 @@
 
 			}
 
-		}
-		else{
+		} else{
 			foreach($this->ownerDocument->_toNodeList($content) as $node){
 				$this->insert_bottom($node);
 			}

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/DocumentType.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
 
 use DOMDocumentType;
 
-class DocumentType extends DOMDocumentType{}
+class DocumentType extends DOMDocumentType {}