chillerlan / php-prototype-dom

src/Document.php

Doc Comments +4 added lines, -1 removed lines

@@ -87,6 +87,9 @@ 
 	 * ugly *
 	 ********/
 
+	/**
+	 * @param string $content
+	 */
 	public function _loadDocument($content, $xml = false){
 
 		switch(true){
@@ -108,7 +111,7 @@ 
 	}
 
 	/**
-	 * @param mixed $content
+	 * @param DOMNodeList $content
 	 *
 	 * @return \chillerlan\PrototypeDOM\NodeList
 	 * @throws \Exception

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 chillerlan\PrototypeDOM\Node\{Element, PrototypeNode};
 use chillerlan\PrototypeDOM\Traits\Magic;
 use Symfony\Component\CssSelector\CssSelectorConverter;

Spacing +34 added lines, -34 removed lines

@@ -20,7 +20,7 @@ 
 /**
  * @property string $title
  */
-class Document extends DOMDocument{
+class Document extends DOMDocument {
 	use Magic;
 
 	const NODE_CLASSES = [
@@ -41,14 +41,14 @@ 
 	 * @param string|null $version
 	 * @param string|null $encoding
 	 */
-	public function __construct($content = null, $xml = false, $version = '1.0', $encoding = 'UTF-8'){
+	public function __construct($content = null, $xml = false, $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__.'\\Node\\'.$nodeClass);
 		}
 
-		if(!is_null($content)){
+		if (!is_null($content)) {
 			$this->_loadDocument($content, $xml);
 		}
 
@@ -59,21 +59,21 @@ 
 	 * magic *
 	 *********/
 
-	public function magic_get_title(){
+	public function magic_get_title() {
 		return $this->select('head > title')->item(0)->nodeValue ?? null;
 	}
 
-	public function magic_set_title(string $title){
+	public function magic_set_title(string $title) {
 		$currentTitle = $this->select('head > title')->item(0);
 
-		if($currentTitle instanceof Element){
+		if ($currentTitle instanceof Element) {
 			$currentTitle->update($title);
 		}
-		else{
+		else {
 			$head         = $this->select('head')->item(0);
 			$currentTitle = $this->newElement('title')->update($title);
 
-			if(!$head){
+			if (!$head) {
 				$head = $this->appendChild($this->newElement('head'));
 			}
 
@@ -87,9 +87,9 @@ 
 	 * ugly *
 	 ********/
 
-	public function _loadDocument($content, $xml = false){
+	public function _loadDocument($content, $xml = false) {
 
-		switch(true){
+		switch (true) {
 			case $content instanceof NodeList   : return $this->insertNodeList($content);
 			case $content instanceof DOMNodeList: return $this->insertNodeList(new NodeList($content));
 			case is_string($content)            : return $this->_loadDocumentString($content, $xml);
@@ -97,8 +97,8 @@ 
 		}
 	}
 
-	public function _loadDocumentString(string $documentSource, bool $xml = false){
-		$options = LIBXML_COMPACT|LIBXML_NONET;
+	public function _loadDocumentString(string $documentSource, bool $xml = false) {
+		$options = LIBXML_COMPACT | LIBXML_NONET;
 
 		$xml
 			? $this->loadXML($documentSource, $options)
@@ -115,7 +115,7 @@ 
 	 */
 	public function _toNodeList($content):NodeList{
 
-		switch(true){
+		switch (true) {
 			case $content instanceof NodeList   : return $content;
 			case $content instanceof DOMNodeList: return new NodeList($content);
 			case $content instanceof DOMNode    : return $this->_arrayToNodeList([$content]);
@@ -147,7 +147,7 @@ 
 	public function _arrayToNodeList(array $array):NodeList{
 		$nodelist = new NodeList;
 
-		foreach($array as $node){
+		foreach ($array as $node) {
 			$nodelist[] = $node;
 		}
 
@@ -200,9 +200,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\Node\Element $node */
-			foreach($nodes as $node){
+			foreach ($nodes as $node) {
 				$node->remove();
 			}
 
@@ -219,7 +219,7 @@ 
 	public function insertNodeList(NodeList $nodeList):Document{
 
 		/** @var \DOMNode $node */
-		foreach($nodeList as $node){
+		foreach ($nodeList as $node) {
 			$this->appendChild($this->importNode($node->cloneNode(true), true));
 		}
 
@@ -256,21 +256,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;
 				}
 
@@ -294,15 +294,15 @@ 
 	public function recursivelyCollect(PrototypeNode $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;
 				}
 
@@ -322,14 +322,14 @@ 
 	 *
 	 * @return \chillerlan\PrototypeDOM\Node\PrototypeElement|null
 	 */
-	public function _recursivelyFind(PrototypeNode $element, string $property, string $selector = null, int $index = 0, int $nodeType = XML_ELEMENT_NODE){
+	public function _recursivelyFind(PrototypeNode $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\Node\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;
 				}
 
@@ -351,9 +351,9 @@ 
 	 */
 	public function match(PrototypeNode $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;
 			}
 
@@ -374,7 +374,7 @@ 
 		/** @var \chillerlan\PrototypeDOM\Node\Element $element */
 		$element = $this->createElement($tag);
 
-		if(!is_null($attributes)){
+		if (!is_null($attributes)) {
 			$element->setAttributes($attributes);
 		}
 

Braces +1 added lines, -2 removed lines

@@ -68,8 +68,7 @@
 
 		if($currentTitle instanceof Element){
 			$currentTitle->update($title);
-		}
-		else{
+		} else{
 			$head         = $this->select('head')->item(0);
 			$currentTitle = $this->newElement('title')->update($title);
 

src/Traits/HTMLElementTrait.php

Spacing +10 added lines, -10 removed lines

@@ -21,7 +21,7 @@ 
  * @property string $src
  * @property string $innerHTML
  */
-trait HTMLElementTrait{
+trait HTMLElementTrait {
 	use Magic, ElementTrait;
 
 	protected function magic_get_id():string {
@@ -70,7 +70,7 @@ 
 	public function identify(string $newID = null):string {
 		$oldID = $this->id;
 
-		if(!is_null($newID)){
+		if (!is_null($newID)) {
 			$this->id = $newID;
 		}
 
@@ -85,14 +85,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;
 					}
 
@@ -160,10 +160,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];
 		}
 
@@ -181,11 +181,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.';';
 		}