chillerlan / php-prototype-dom

src/Node/ElementTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 	 *
 	 * @param string|array|\DOMNode|\DOMNodeList $content
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return ElementTrait
 	 */
 	public function insert($content):PrototypeElement{
 
@@ -87,7 +87,7 @@ 
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement      $node
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement|null $refNode
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return ElementTrait
 	 */
 	public function insert_before(PrototypeElement $node, PrototypeElement $refNode = null):PrototypeElement{
 
@@ -129,7 +129,7 @@ 
 	/**
 	 * @param \chillerlan\PrototypeDOM\Node\PrototypeElement $node
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return ElementTrait
 	 */
 	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 ElementTrait{
 
 	/**

Spacing +10 added lines, -10 removed lines

@@ -14,7 +14,7 @@ 
 
 use chillerlan\PrototypeDOM\NodeList;
 
-trait ElementTrait{
+trait ElementTrait {
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/wrap/
@@ -53,28 +53,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]);
 			}
 
@@ -91,7 +91,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);
 		}
 
@@ -105,7 +105,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
 		}
 
@@ -119,7 +119,7 @@ 
 	 */
 	public function insert_top(PrototypeElement $node):PrototypeElement{
 
-		if($this->hasChildNodes()){
+		if ($this->hasChildNodes()) {
 			return $this->firstChild->insert_before($node, $this->firstChild);
 		}
 

src/Node/HTMLElementTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
 	 * @param array $style
 	 * @param bool  $replace
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
+	 * @return HTMLElementTrait
 	 */
 	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,13 +21,13 @@ 
  * @property string $src
  * @property string $innerHTML
  */
-trait HTMLElementTrait{
+trait HTMLElementTrait {
 
 	protected function magic_get_id():string{
 		return \trim($this->getAttribute('id'));
 	}
 
-	protected function magic_set_id($id){
+	protected function magic_set_id($id) {
 		return $this->setAttribute('id', $id);
 	}
 
@@ -35,7 +35,7 @@ 
 		return \trim($this->getAttribute('class'));
 	}
 
-	protected function magic_set_class($class){
+	protected function magic_set_class($class) {
 		return $this->setAttribute('class', $class);
 	}
 
@@ -43,7 +43,7 @@ 
 		return \trim($this->getAttribute('href'));
 	}
 
-	protected function magic_set_href($class){
+	protected function magic_set_href($class) {
 		return $this->setAttribute('href', $class);
 	}
 
@@ -51,7 +51,7 @@ 
 		return \trim($this->getAttribute('src'));
 	}
 
-	protected function magic_set_src($class){
+	protected function magic_set_src($class) {
 		return $this->setAttribute('src', $class);
 	}
 
@@ -69,7 +69,7 @@ 
 	public function identify(string $newID = null):string{
 		$oldID = $this->id;
 
-		if($newID !== null){
+		if ($newID !== null) {
 			$this->id = $newID;
 		}
 
@@ -83,16 +83,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;
 			}
 
@@ -144,7 +144,7 @@ 
 	 */
 	public function toggleClassName(string $classname):PrototypeHTMLElement{
 
-		if($this->hasClassName($classname)){
+		if ($this->hasClassName($classname)) {
 			return $this->removeClassName($classname);
 		}
 
@@ -158,10 +158,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];
 		}
 
@@ -179,11 +179,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/TraversalTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 trait TraversalTrait{
 
 	/**
-	 * @param        $selector
+	 * @param        string|null $selector
 	 * @param        $index
 	 * @param string $property
 	 * @param int    $nodeType https://secure.php.net/manual/dom.constants.php
@@ -65,7 +65,7 @@ 
 	 * @param null $expression
 	 * @param int  $index
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
+	 * @return Element|null
 	 */
 	public function down($expression = null, int $index = null):?PrototypeNode{
 
@@ -188,7 +188,7 @@ 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/firstDescendant/
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
+	 * @return Element|null
 	 */
 	public function firstDescendant():?PrototypeNode{
 		return $this->descendants()->first();

Spacing +14 added lines, -14 removed lines

@@ -18,7 +18,7 @@ 
 /**
  * @property \chillerlan\PrototypeDOM\Document $ownerDocument
  */
-trait TraversalTrait{
+trait TraversalTrait {
 
 	/**
 	 * @param        $selector
@@ -28,9 +28,9 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function recursivelyFind($selector, int $index = null, string $property = null, int $nodeType = \XML_ELEMENT_NODE):?PrototypeNode{
+	public function recursivelyFind($selector, int $index = null, string $property = null, int $nodeType = \XML_ELEMENT_NODE): ?PrototypeNode{
 
-		if(\is_numeric($selector)){
+		if (\is_numeric($selector)) {
 			return $this->ownerDocument->recursivelyFind($this, $property, null, $selector, $nodeType);
 		}
 
@@ -67,19 +67,19 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function down($expression = null, int $index = null):?PrototypeNode{
+	public function down($expression = null, int $index = null): ?PrototypeNode{
 
-		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);
 		}
 
@@ -94,7 +94,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function up($expression = null, int $index = null):?PrototypeNode{
+	public function up($expression = null, int $index = null): ?PrototypeNode{
 		return $this->recursivelyFind($expression, $index, 'parentNode');
 	}
 
@@ -106,7 +106,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function previous($expression = null, int $index = null):?PrototypeNode{
+	public function previous($expression = null, int $index = null): ?PrototypeNode{
 		return $this->recursivelyFind($expression, $index, 'previousSibling');
 	}
 
@@ -118,7 +118,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function next($expression = null, int $index = null):?PrototypeNode{
+	public function next($expression = null, int $index = null): ?PrototypeNode{
 		return $this->recursivelyFind($expression, $index, 'nextSibling');
 	}
 
@@ -132,13 +132,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;
 			}
 
@@ -190,7 +190,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function firstDescendant():?PrototypeNode{
+	public function firstDescendant(): ?PrototypeNode{
 		return $this->descendants()->first();
 	}
 

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

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{
 
-		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\Element|null
 	 */
-	public function item(int $offset):?Element{
+	public function item(int $offset): ?Element{
 		return $this->array[$offset] ?? null;
 	}
 
@@ -140,7 +140,7 @@ 
 	 */
 	public function offsetSet($offset, $value):void{
 
-		if($value instanceof PrototypeNode){
+		if ($value instanceof PrototypeNode) {
 
 			\is_int($offset)
 				? $this->array[$offset] = $value
@@ -155,7 +155,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
-	public function first():?Element{
+	public function first(): ?Element{
 		return $this->__first();
 	}
 
@@ -164,7 +164,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
-	public function last():?Element{
+	public function last(): ?Element{
 		return $this->__last();
 	}
 

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;

src/Node/DocumentFragment.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class DocumentFragment extends \DOMDocumentFragment implements PrototypeElement{
+class DocumentFragment extends \DOMDocumentFragment implements PrototypeElement {
 	use ElementTrait, NodeTrait, TraversalTrait;
 }

src/Node/Notation.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class Notation extends \DOMNotation implements PrototypeNode{
+class Notation extends \DOMNotation implements PrototypeNode {
 	use NodeTrait;
 }

src/Node/ProcessingInstruction.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class ProcessingInstruction extends \DOMProcessingInstruction implements PrototypeNode{
+class ProcessingInstruction extends \DOMProcessingInstruction implements PrototypeNode {
 	use NodeTrait;
 }

src/Node/Entity.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class Entity extends \DOMEntity implements PrototypeNode{
+class Entity extends \DOMEntity implements PrototypeNode {
 	use NodeTrait;
 }

src/Node/Text.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 
 namespace chillerlan\PrototypeDOM\Node;
 
-class Text extends \DOMText implements PrototypeElement{
+class Text extends \DOMText implements PrototypeElement {
 	use ElementTrait, NodeTrait, TraversalTrait;
 }