chillerlan / php-prototype-dom

src/NodeTraversalTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -39,8 +39,8 @@ 
 	}
 
 	/**
-	 * @param        $selector
-	 * @param        $index
+	 * @param        string|null $selector
+	 * @param        null|integer $index
 	 * @param string $property
 	 * @param int    $nodeType
 	 *
@@ -198,7 +198,7 @@ 
 	}
 
 	/**
-	 * @return \DOMNode|null
+	 * @return Element|null
 	 */
 	public function firstDescendant(){
 		return $this->descendants()[0] ?? null;

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 NodeTraversalTrait, NodeManipulationTrait;
 
 	/**
@@ -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/Document.php

Spacing +24 added lines, -24 removed lines

@@ -15,7 +15,7 @@ 
 use DOMDocument, DOMNode, DOMNodeList, DOMXPath;
 use Symfony\Component\CssSelector\CssSelectorConverter;
 
-class Document extends DOMDocument{
+class Document extends DOMDocument {
 
 	/**
 	 * Document constructor.
@@ -23,7 +23,7 @@ 
 	 * @param string|null $version
 	 * @param string|null $encoding
 	 */
-	public function __construct($version = null, $encoding = null){
+	public function __construct($version = null, $encoding = null) {
 		parent::__construct($version, $encoding);
 
 		$this->registerNodeClass('DOMElement', Element::class);
@@ -73,21 +73,21 @@ 
 	 */
 	public function select($selectors = null, DOMNode $contextNode = null, string $axis = 'descendant-or-self::', int $nodeType = XML_ELEMENT_NODE):array{
 
-		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 = [];
 
-		foreach($selectors as $selector){
+		foreach ($selectors as $selector) {
 
-			foreach($this->getElementsBySelector($selector, $contextNode, $axis) as $element){
+			foreach ($this->getElementsBySelector($selector, $contextNode, $axis) as $element) {
 
-				if($element->nodeType === $nodeType){
+				if ($element->nodeType === $nodeType) {
 					$elements[] = $element;
 				}
 
@@ -120,9 +120,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();
 			}
 
@@ -139,21 +139,21 @@ 
 	 */
 	public function _toDOMNodeList($content):DOMNodeList{
 
-		if($content instanceof DOMNodeList){
+		if ($content instanceof DOMNodeList) {
 			return $content;
 		}
 
 		$document = new Document;
 
-		if($content instanceof DOMNode){
+		if ($content instanceof DOMNode) {
 			$document->loadHTML('<html><body id="content"></body></html>');
 
 			$document->getElementById('content')->appendChild($document->importNode($content, true));
 		}
-		elseif(is_string($content)){
+		elseif (is_string($content)) {
 			$document->loadHTML('<html><body id="content">'.$content.'</body></html>');
 		}
-		else{
+		else {
 			throw new \Exception('invalid content'); // @codeCoverageIgnore
 		}
 
@@ -171,15 +171,15 @@ 
 	public function recursivelyCollect(DOMNode $element, string $property, int $maxLength = -1, int $nodeType = XML_ELEMENT_NODE):array{
 		$nodes = [];
 
-		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;
 				}
 
@@ -199,14 +199,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;
 				}
 
@@ -226,9 +226,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;
 			}
 
@@ -247,7 +247,7 @@ 
 		/** @var \chillerlan\PrototypeDOM\Element $element */
 		$element = $this->createElement($tag);
 
-		if($attributes){
+		if ($attributes) {
 			$element->setAttributes($attributes);
 		}