chillerlan / php-prototype-dom

src/NodeList.php

Doc Comments +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
 	/**
 	 * @param int $offset
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
+	 * @return Node\Element|null
 	 */
 	public function offsetGet($offset){
 		return $this->item($offset);

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

@@ -12,7 +12,10 @@
 
 namespace chillerlan\PrototypeDOM;
 
-use ArrayAccess, Countable, DOMNodeList, Iterator;
+use ArrayAccess;
+use Countable;
+use DOMNodeList;
+use Iterator;
 use chillerlan\PrototypeDOM\Node\PrototypeNode;
 use chillerlan\PrototypeDOM\Traits\{EnumerableTrait, Magic};
 

Spacing +13 added lines, -13 removed lines

@@ -39,9 +39,9 @@ 
 	 *
 	 * @throws \Exception
 	 */
-	public function __construct(DOMNodeList $nodes = null){
+	public function __construct(DOMNodeList $nodes = null) {
 
-		if(!is_null($nodes)){
+		if (!is_null($nodes)) {
 			$this->array = iterator_to_array($nodes);
 		}
 
@@ -71,9 +71,9 @@ 
 	 */
 	public function match(PrototypeNode $node):bool{
 
-		foreach($this->array as $element){
+		foreach ($this->array as $element) {
 
-			if($element->isSameNode($node)){
+			if ($element->isSameNode($node)) {
 				return true;
 			}
 
@@ -119,7 +119,7 @@ 
 	/**
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode
 	 */
-	public function current(){
+	public function current() {
 		return $this->array[$this->offset];
 	}
 
@@ -140,14 +140,14 @@ 
 	/**
 	 *  @return void
 	 */
-	public function next(){
+	public function next() {
 		$this->offset++;
 	}
 
 	/**
 	 * @return void
 	 */
-	public function rewind(){
+	public function rewind() {
 		$this->offset = 0;
 	}
 
@@ -170,7 +170,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function offsetGet($offset){
+	public function offsetGet($offset) {
 		return $this->item($offset);
 	}
 
@@ -180,14 +180,14 @@ 
 	 *
 	 * @return void
 	 */
-	public function offsetSet($offset, $value){
+	public function offsetSet($offset, $value) {
 
-		if($value instanceof PrototypeNode){
+		if ($value instanceof PrototypeNode) {
 
-			if(is_int($offset)){
+			if (is_int($offset)) {
 				$this->array[$offset] = $value;
 			}
-			else{
+			else {
 				$this->array[] = $value;
 			}
 
@@ -200,7 +200,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function offsetUnset($offset){
+	public function offsetUnset($offset) {
 		unset($this->array[$offset]);
 	}
 

Braces +1 added lines, -2 removed lines

@@ -186,8 +186,7 @@
 
 			if(is_int($offset)){
 				$this->array[$offset] = $value;
-			}
-			else{
+			} else{
 				$this->array[] = $value;
 			}
 

src/Traits/ElementTrait.php

Doc Comments +5 added lines, -5 removed lines

@@ -31,9 +31,9 @@ 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/update/
 	 *
-	 * @param string|\DOMNode|\DOMNodeList $content
+	 * @param string $content
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return ElementTrait
 	 */
 	public function update($content):PrototypeElement{
 		$this->purge();
@@ -53,7 +53,7 @@ 
 	 *
 	 * @param string|array|\DOMNode|\DOMNodeList $content
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement
+	 * @return ElementTrait
 	 */
 	public function insert($content):PrototypeElement{
 
@@ -90,7 +90,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{
 
@@ -126,7 +126,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));

Spacing +9 added lines, -9 removed lines

@@ -14,7 +14,7 @@ 
 
 use chillerlan\PrototypeDOM\Node\PrototypeElement;
 
-trait ElementTrait{
+trait ElementTrait {
 	use TraversalTrait;
 
 	/**
@@ -57,18 +57,18 @@ 
 	 */
 	public function insert($content):PrototypeElement{
 
-		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]);
 					}
 
@@ -77,8 +77,8 @@ 
 			}
 
 		}
-		else{
-			foreach($this->ownerDocument->_toNodeList($content) as $node){
+		else {
+			foreach ($this->ownerDocument->_toNodeList($content) as $node) {
 				$this->insert_bottom($node);
 			}
 		}
@@ -94,7 +94,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);
 		}
 

Braces +1 added lines, -2 removed lines

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

src/Traits/EnumerableTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 	 * @link http://api.prototypejs.org/language/Enumerable/prototype/collect/
 	 * @link http://api.prototypejs.org/language/Enumerable/prototype/map/
 	 *
-	 * @param $callback
+	 * @param callable $callback
 	 *
 	 * @return array
 	 * @throws \Exception

Spacing +7 added lines, -7 removed lines

@@ -14,7 +14,7 @@ 
 /**
  * Trait EnumerableTrait
  */
-trait EnumerableTrait{
+trait EnumerableTrait {
 
 	/**
 	 * @link http://api.prototypejs.org/language/Enumerable/prototype/each/
@@ -23,7 +23,7 @@ 
 	 *
 	 * @return $this
 	 */
-	public function each($callback){
+	public function each($callback) {
 		$this->map($callback);
 
 		return $this;
@@ -34,7 +34,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
-	public function first(){
+	public function first() {
 		return $this->item(0);
 	}
 
@@ -43,7 +43,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
-	public function last(){
+	public function last() {
 		return $this->item($this->count() - 1);
 	}
 
@@ -58,13 +58,13 @@ 
 	 */
 	public function map($callback):array {
 
-		if(!is_callable($callback)){
+		if (!is_callable($callback)) {
 			throw new \Exception('invalid callback'); // @codeCoverageIgnore
 		}
 
 		$return = [];
 
-		foreach($this->array as $index => $element){
+		foreach ($this->array as $index => $element) {
 			$return[$index] = call_user_func_array($callback, [$element, $index]);
 		}
 
@@ -87,7 +87,7 @@ 
 	 *
 	 * @return $this
 	 */
-	public function reverse(){
+	public function reverse() {
 		$this->array = array_reverse($this->array);
 		$this->rewind();
 

src/Traits/HTMLElementTrait.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -19,7 +19,7 @@ 
  * @property string $class
  * @property string $innerHTML
  */
-trait HTMLElementTrait{
+trait HTMLElementTrait {
 	use Magic, ElementTrait;
 
 	protected function magic_get_id():string {
@@ -52,7 +52,7 @@ 
 	public function identify(string $newID = null):string {
 		$oldID = $this->id;
 
-		if(!is_null($newID)){
+		if (!is_null($newID)) {
 			$this->id = $newID;
 		}
 
@@ -67,14 +67,14 @@ 
 	public function classNames():array{
 		$currentClassnames = [];
 
-		if($this->hasAttributes()){
+		if ($this->hasAttributes()) {
 			$classnames = explode(' ', $this->class);
 
-			if(!empty($classnames)){
+			if (!empty($classnames)) {
 
-				foreach($classnames as $classname){
+				foreach ($classnames as $classname) {
 
-					if(empty($classname)){
+					if (empty($classname)) {
 						continue;
 					}
 
@@ -142,10 +142,10 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function getStyle(string $property){
+	public function getStyle(string $property) {
 		$currentStyle = $this->getStyles();
 
-		if(array_key_exists(strtolower($property), $currentStyle)){
+		if (array_key_exists(strtolower($property), $currentStyle)) {
 			return $currentStyle[$property];
 		}
 
@@ -163,11 +163,11 @@ 
 	public function setStyle(array $style, bool $replace = false):PrototypeHTMLElement{
 		$currentStyle = $this->getStyles();
 
-		if(!$replace){
+		if (!$replace) {
 			$style = array_merge($currentStyle, $style);
 		}
 
-		foreach($style as $property => $value){
+		foreach ($style as $property => $value) {
 			$style[$property] = $property.': '.$value.';';
 		}
 

src/Traits/NodeTrait.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -33,7 +33,7 @@ 
  * @property string $baseURI
  * @property string $textContent
  */
-trait NodeTrait{
+trait NodeTrait {
 
 	/**
 	 * @link http://api.prototypejs.org/dom/Element/recursivelyCollect/
@@ -78,7 +78,7 @@ 
 	 */
 	public function remove():PrototypeNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			return $this;
 		}
 
@@ -94,7 +94,7 @@ 
 	 */
 	public function replace(PrototypeNode $newnode):PrototypeNode{
 
-		if(!$this->parentNode){
+		if (!$this->parentNode) {
 			return $this;
 		}
 
@@ -106,7 +106,7 @@ 
 	 */
 	public function purge():PrototypeNode{
 
-		while($this->hasChildNodes()){
+		while ($this->hasChildNodes()) {
 			$this->firstChild->remove();
 		}
 
@@ -121,10 +121,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/Traits/TraversalTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -19,8 +19,8 @@ 
 	use  NodeTrait;
 
 	/**
-	 * @param        $selector
-	 * @param        $index
+	 * @param        string|null $selector
+	 * @param        null|integer $index
 	 * @param string $property
 	 * @param int    $nodeType
 	 *
@@ -64,7 +64,7 @@ 
 	 * @param null $expression
 	 * @param int  $index
 	 *
-	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
+	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
 	public function down($expression = null, int $index = null){
 

Spacing +14 added lines, -14 removed lines

@@ -15,7 +15,7 @@ 
 use chillerlan\PrototypeDOM\Node\PrototypeNode;
 use chillerlan\PrototypeDOM\NodeList;
 
-trait TraversalTrait{
+trait TraversalTrait {
 	use  NodeTrait;
 
 	/**
@@ -26,9 +26,9 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|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;
 		}
@@ -66,17 +66,17 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function down($expression = null, int $index = null){
+	public function down($expression = null, int $index = null) {
 
-		if(count(func_get_args()) === 0){
+		if (count(func_get_args()) === 0) {
 			return $this->firstDescendant();
 		}
 
-		if(is_int($expression)){
+		if (is_int($expression)) {
 			$index      = $expression;
 			$expression = '*';
 		}
-		else{
+		else {
 			$index = $index ?? 0;
 		}
 
@@ -91,7 +91,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function up($expression = null, int $index = null){
+	public function up($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'parentNode');
 	}
 
@@ -103,7 +103,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function previous($expression = null, int $index = null){
+	public function previous($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'previousSibling');
 	}
 
@@ -115,7 +115,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeNode|null
 	 */
-	public function next($expression = null, int $index = null){
+	public function next($expression = null, int $index = null) {
 		return $this->_recursivelyFind($expression, $index, 'nextSibling');
 	}
 
@@ -129,11 +129,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;
 				}
 
@@ -187,7 +187,7 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\Element|null
 	 */
-	public function firstDescendant(){
+	public function firstDescendant() {
 		return $this->descendants()->first();
 	}
 

Braces +1 added lines, -2 removed lines

@@ -75,8 +75,7 @@
 		if(is_int($expression)){
 			$index      = $expression;
 			$expression = '*';
-		}
-		else{
+		} else{
 			$index = $index ?? 0;
 		}
 

src/Node/Text.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
 use DOMText;
 use chillerlan\PrototypeDOM\Traits\ElementTrait;
 
-class Text extends DOMText implements PrototypeElement{
+class Text extends DOMText implements PrototypeElement {
 	use ElementTrait;
 }

src/Node/DocumentType.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
 use chillerlan\PrototypeDOM\Traits\NodeTrait;
 use DOMDocumentType;
 
-class DocumentType extends DOMDocumentType implements PrototypeNode{
+class DocumentType extends DOMDocumentType implements PrototypeNode {
 	use NodeTrait;
 }

src/Node/CharacterData.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
 use DOMCharacterData;
 use chillerlan\PrototypeDOM\Traits\ElementTrait;
 
-class CharacterData extends DOMCharacterData implements PrototypeElement{
+class CharacterData extends DOMCharacterData implements PrototypeElement {
 	use ElementTrait;
 }