chillerlan / php-prototype-dom

src/NodeList.php

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

@@ -15,7 +15,11 @@
 use chillerlan\PrototypeDOM\Node\{Element, PrototypeNode};
 use chillerlan\Traits\{Enumerable, Interfaces\ArrayAccessTrait, Magic};
 use chillerlan\Traits\SPL\{CountableTrait, SeekableIteratorTrait};
-use ArrayAccess, Countable, DOMNode, DOMNodeList, SeekableIterator;
+use ArrayAccess;
+use Countable;
+use DOMNode;
+use DOMNodeList;
+use SeekableIterator;
 
 /**
  * @property int $length

Braces +2 added lines, -4 removed lines

@@ -51,11 +51,9 @@
 
 		if($nodes instanceof DOMNodeList){
 			$this->array = \iterator_to_array($nodes);
-		}
-		elseif($nodes instanceof NodeList){
+		} elseif($nodes instanceof NodeList){
 			$this->array = $nodes->toArray();
-		}
-		elseif(\is_iterable($nodes)){
+		} elseif(\is_iterable($nodes)){
 			foreach($nodes as $node){
 				if($node instanceof DOMNode || $node instanceof PrototypeNode){
 					$this->array[] = $node;

Spacing +12 added lines, -12 removed lines

@@ -47,17 +47,17 @@ 
 	 *
 	 * @throws \Exception
 	 */
-	public function __construct(iterable $nodes = null){
+	public function __construct(iterable $nodes = null) {
 
-		if($nodes instanceof DOMNodeList){
+		if ($nodes instanceof DOMNodeList) {
 			$this->array = \iterator_to_array($nodes);
 		}
-		elseif($nodes instanceof NodeList){
+		elseif ($nodes instanceof NodeList) {
 			$this->array = $nodes->toArray();
 		}
-		elseif(\is_iterable($nodes)){
-			foreach($nodes as $node){
-				if($node instanceof DOMNode || $node instanceof PrototypeNode){
+		elseif (\is_iterable($nodes)) {
+			foreach ($nodes as $node) {
+				if ($node instanceof DOMNode || $node instanceof PrototypeNode) {
 					$this->array[] = $node;
 				}
 			}
@@ -88,9 +88,9 @@ 
 	 */
 	public function match(DOMNode $node):bool{
 		/** @var \chillerlan\PrototypeDOM\Node\Element $element */
-		foreach($this->array as $element){
+		foreach ($this->array as $element) {
 
-			if($element->isSameNode($node)){
+			if ($element->isSameNode($node)) {
 				return true;
 			}
 
@@ -104,7 +104,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|\DOMNode|null
 	 */
-	public function item(int $offset):?DOMNode{
+	public function item(int $offset): ?DOMNode{
 		return $this->array[$offset] ?? null;
 	}
 
@@ -140,7 +140,7 @@ 
 	 */
 	public function offsetSet($offset, $value):void{
 
-		if($value instanceof DOMNode){
+		if ($value instanceof DOMNode) {
 
 			\is_int($offset)
 				? $this->array[$offset] = $value
@@ -155,7 +155,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|\DOMNode|null
 	 */
-	public function first():?DOMNode{
+	public function first(): ?DOMNode{
 		return $this->__first();
 	}
 
@@ -164,7 +164,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|\DOMNode|null
 	 */
-	public function last():?DOMNode{
+	public function last(): ?DOMNode{
 		return $this->__last();
 	}
 

src/Node/PrototypeElement.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @property bool   $schemaTypeInfo
  * @property string $tagName
  */
-interface PrototypeElement extends PrototypeTraversal{
+interface PrototypeElement extends PrototypeTraversal {
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/wrap/

src/Node/PrototypeHTMLElement.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
  * @property string $href
  * @property string $src
  */
-interface PrototypeHTMLElement extends PrototypeElement{
+interface PrototypeHTMLElement extends PrototypeElement {
 
 	/**
 	 * http://api.prototypejs.org/dom/Element/identify/
@@ -80,7 +80,7 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function getStyle(string $property):?string;
+	public function getStyle(string $property): ?string;
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/setStyle/

src/Node/PrototypeNode.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
  * @property string                                $baseURI
  * @property string                                $textContent
  */
-interface PrototypeNode{
+interface PrototypeNode {
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/recursivelyCollect/

src/Node/PrototypeElementTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
 	 *
 	 * @param string|array|\DOMNode|\DOMNodeList $content
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return PrototypeElementTrait
 	 */
 	public function insert($content):PrototypeElement{
 
@@ -88,7 +88,7 @@ 
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement      $node
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement|null $refNode
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return PrototypeElementTrait
 	 */
 	public function insert_before(PrototypeElement $node, PrototypeElement $refNode = null):PrototypeElement{
 
@@ -130,7 +130,7 @@ 
 	/**
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement $node
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return PrototypeElementTrait
 	 */
 	public function insert_bottom(PrototypeElement $node):PrototypeElement{
 		$this->appendChild($this->importNode($node));

Unused Use Statements -2 removed lines

@@ -12,8 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-use chillerlan\PrototypeDOM\NodeList;
-
 trait PrototypeElementTrait{
 	use PrototypeTraversalTrait;
 

Spacing +10 added lines, -10 removed lines

@@ -14,7 +14,7 @@ 
 
 use chillerlan\PrototypeDOM\NodeList;
 
-trait PrototypeElementTrait{
+trait PrototypeElementTrait {
 	use PrototypeTraversalTrait;
 
 	/**
@@ -54,28 +54,28 @@ 
 	 */
 	public function insert($content):PrototypeElement{
 
-		if(!\is_array($content)){
+		if (!\is_array($content)) {
 
-			foreach($this->ownerDocument->toNodeList($content) as $node){
+			foreach ($this->ownerDocument->toNodeList($content) as $node) {
 				$this->insert_bottom($node);
 			}
 
 			return $this;
 		}
 
-		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)) {
 				continue;
 			}
 
 			$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]);
 			}
 
@@ -92,7 +92,7 @@ 
 	 */
 	public function insert_before(PrototypeElement $node, PrototypeElement $refNode = null):PrototypeElement{
 
-		if($this->parentNode){
+		if ($this->parentNode) {
 			$this->parentNode->insertBefore($this->importNode($node), $refNode ?? $this);
 		}
 
@@ -106,7 +106,7 @@ 
 	 */
 	public function insert_after(PrototypeElement $node):PrototypeElement{
 
-		if(!$this->nextSibling && $this->parentNode){
+		if (!$this->nextSibling && $this->parentNode) {
 			return $this->parentNode->insert_bottom($node); // @codeCoverageIgnore
 		}
 
@@ -120,7 +120,7 @@ 
 	 */
 	public function insert_top(PrototypeElement $node):PrototypeElement{
 
-		if($this->hasChildNodes()){
+		if ($this->hasChildNodes()) {
 			return $this->firstChild->insert_before($node, $this->firstChild);
 		}
 

src/Node/PrototypeHTMLElementTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -175,7 +175,7 @@
 	 * @param array $style
 	 * @param bool  $replace
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
+	 * @return PrototypeHTMLElementTrait
 	 */
 	public function setStyle(array $style, bool $replace = null):PrototypeHTMLElement{
 		$currentStyle = $this->getStyles();

Unused Use Statements -2 removed lines

@@ -12,8 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-use chillerlan\Traits\Magic;
-
 /**
  * @property string $id
  * @property string $class

Spacing +14 added lines, -14 removed lines

@@ -21,14 +21,14 @@ 
  * @property string $src
  * @property string $innerHTML
  */
-trait PrototypeHTMLElementTrait{
+trait PrototypeHTMLElementTrait {
 	use PrototypeElementTrait;
 
 	protected function magic_get_id():string{
 		return \trim($this->getAttribute('id'));
 	}
 
-	protected function magic_set_id(string $id){
+	protected function magic_set_id(string $id) {
 		return $this->setAttribute('id', $id);
 	}
 
@@ -36,7 +36,7 @@ 
 		return \trim($this->getAttribute('class'));
 	}
 
-	protected function magic_set_class(string $class){
+	protected function magic_set_class(string $class) {
 		return $this->setAttribute('class', $class);
 	}
 
@@ -44,7 +44,7 @@ 
 		return \trim($this->getAttribute('href'));
 	}
 
-	protected function magic_set_href(string $href){
+	protected function magic_set_href(string $href) {
 		return $this->setAttribute('href', $href);
 	}
 
@@ -52,7 +52,7 @@ 
 		return \trim($this->getAttribute('src'));
 	}
 
-	protected function magic_set_src(string $src){
+	protected function magic_set_src(string $src) {
 		return $this->setAttribute('src', $src);
 	}
 
@@ -70,7 +70,7 @@ 
 	public function identify(string $newID = null):string{
 		$oldID = $this->id;
 
-		if($newID !== null){
+		if ($newID !== null) {
 			$this->id = $newID;
 		}
 
@@ -84,16 +84,16 @@ 
 	 */
 	public function classNames():array{
 
-		if(!$this->hasAttributes()){
+		if (!$this->hasAttributes()) {
 			return [];
 		}
 
 		$classnames        = \explode(' ', $this->class);
 		$currentClassnames = [];
 
-		foreach($classnames as $classname){
+		foreach ($classnames as $classname) {
 
-			if(empty($classname)){
+			if (empty($classname)) {
 				continue;
 			}
 
@@ -145,7 +145,7 @@ 
 	 */
 	public function toggleClassName(string $classname):PrototypeHTMLElement{
 
-		if($this->hasClassName($classname)){
+		if ($this->hasClassName($classname)) {
 			return $this->removeClassName($classname);
 		}
 
@@ -159,10 +159,10 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function getStyle(string $property):?string{
+	public function getStyle(string $property): ?string{
 		$currentStyle = $this->getStyles();
 
-		if(\array_key_exists(\strtolower($property), $currentStyle)){
+		if (\array_key_exists(\strtolower($property), $currentStyle)) {
 			return $currentStyle[$property];
 		}
 
@@ -180,11 +180,11 @@ 
 	public function setStyle(array $style, bool $replace = null):PrototypeHTMLElement{
 		$currentStyle = $this->getStyles();
 
-		if($replace !== true){
+		if ($replace !== true) {
 			$style = \array_merge($currentStyle, $style);
 		}
 
-		foreach($style as $property => $value){
+		foreach ($style as $property => $value) {
 			$style[$property] = $property.': '.$value.';';
 		}
 

src/Node/PrototypeNodeTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -98,7 +98,7 @@ 
 	}
 
 	/**
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode
+	 * @return PrototypeNodeTrait
 	 */
 	public function purge():PrototypeNode{
 
@@ -112,7 +112,7 @@ 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/cleanWhitespace/
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode
+	 * @return PrototypeNodeTrait
 	 */
 	public function cleanWhitespace():PrototypeNode{
 		$node = $this->firstChild;

Spacing +6 added lines, -6 removed lines

@@ -32,7 +32,7 @@ 
  * @property string                                $baseURI
  * @property string                                $textContent
  */
-trait PrototypeNodeTrait{
+trait PrototypeNodeTrait {
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/recursivelyCollect/
@@ -74,7 +74,7 @@ 
 	 */
 	public function remove():PrototypeNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			return $this;
 		}
 
@@ -90,7 +90,7 @@ 
 	 */
 	public function replace(PrototypeNode $newnode):PrototypeNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			return $this;
 		}
 
@@ -102,7 +102,7 @@ 
 	 */
 	public function purge():PrototypeNode{
 
-		while($this->hasChildNodes()){
+		while ($this->hasChildNodes()) {
 			$this->firstChild->remove();
 		}
 
@@ -117,10 +117,10 @@ 
 	public function cleanWhitespace():PrototypeNode{
 		$node = $this->firstChild;
 
-		while($node){
+		while ($node) {
 			$nextNode = $node->nextSibling;
 
-			if($node->nodeType === \XML_TEXT_NODE && $node->empty()){
+			if ($node->nodeType === \XML_TEXT_NODE && $node->empty()) {
 				$node->remove();
 			}
 

src/Node/PrototypeTraversalTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	use PrototypeNodeTrait;
 
 	/**
-	 * @param        $selector
+	 * @param        string|null $selector
 	 * @param        $index
 	 * @param string $property
 	 * @param int    $nodeType https://secure.php.net/manual/dom.constants.php

Spacing +14 added lines, -14 removed lines

@@ -18,7 +18,7 @@ 
 /**
  * @property \chillerlan\PrototypeDOM\Document $ownerDocument
  */
-trait PrototypeTraversalTrait{
+trait PrototypeTraversalTrait {
 	use PrototypeNodeTrait;
 
 	/**
@@ -29,9 +29,9 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeTraversal|\DOMNode|null
 	 */
-	public function recursivelyFind($selector, int $index = null, string $property = null, int $nodeType = \XML_ELEMENT_NODE):?PrototypeTraversal{
+	public function recursivelyFind($selector, int $index = null, string $property = null, int $nodeType = \XML_ELEMENT_NODE): ?PrototypeTraversal{
 
-		if(\is_numeric($selector)){
+		if (\is_numeric($selector)) {
 			return $this->ownerDocument->recursivelyFind($this, $property, null, $selector, $nodeType);
 		}
 
@@ -68,19 +68,19 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeTraversal|null
 	 */
-	public function down($expression = null, int $index = null):?PrototypeTraversal{
+	public function down($expression = null, int $index = null): ?PrototypeTraversal{
 
-		if($expression === null && $index === null){
+		if ($expression === null && $index === null) {
 			return $this->firstDescendant();
 		}
 
 		$index = $index ?? 0;
 
-		if(\is_int($expression)){
+		if (\is_int($expression)) {
 			return $this->select(['*'])->item($expression);
 		}
 
-		if(\is_array($expression)){
+		if (\is_array($expression)) {
 			return $this->select($expression)->item($index);
 		}
 
@@ -95,7 +95,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeTraversal|null
 	 */
-	public function up($expression = null, int $index = null):?PrototypeTraversal{
+	public function up($expression = null, int $index = null): ?PrototypeTraversal{
 		return $this->recursivelyFind($expression, $index, 'parentNode');
 	}
 
@@ -107,7 +107,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeTraversal|null
 	 */
-	public function previous($expression = null, int $index = null):?PrototypeTraversal{
+	public function previous($expression = null, int $index = null): ?PrototypeTraversal{
 		return $this->recursivelyFind($expression, $index, 'previousSibling');
 	}
 
@@ -119,7 +119,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeTraversal|null
 	 */
-	public function next($expression = null, int $index = null):?PrototypeTraversal{
+	public function next($expression = null, int $index = null): ?PrototypeTraversal{
 		return $this->recursivelyFind($expression, $index, 'nextSibling');
 	}
 
@@ -133,13 +133,13 @@ 
 	public function childElements(int $nodeType = \XML_ELEMENT_NODE):NodeList{
 		$children = new NodeList;
 
-		if(!$this->hasChildNodes()){
+		if (!$this->hasChildNodes()) {
 			return $children;
 		}
 
-		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 \chillerlan\PrototypeDOM\Node\PrototypeTraversal|null
 	 */
-	public function firstDescendant():?PrototypeTraversal{
+	public function firstDescendant(): ?PrototypeTraversal{
 		return $this->descendants()->first();
 	}
 

src/Node/CharacterData.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class CharacterData extends \DOMCharacterData implements PrototypeElement{
+class CharacterData extends \DOMCharacterData implements PrototypeElement {
 	use PrototypeElementTrait;
 }