CS fix

Author: dimabdc

composer.json

@@ -14,6 +14,8 @@
 	"require":{
         "php":">=5.4",
         "ext-dom": "*",
+        "ext-libxml": "*",
+        "ext-simplexml": "*",
         "symfony/css-selector": "*"
     },
     "require-dev": {

lib/Document.php

@@ -58,9 +58,9 @@ public function __construct($element = null)
         $this->document = new DOMDocument('1.0', 'UTF-8');
 
         if ($element instanceof Element) {
-            $element = $element->getNode();
+            $node = $element->getNode();
 
-            $domNode = $this->document->importNode($element, true);
+            $domNode = $this->document->importNode($node, true);
             $this->document->appendChild($domNode);
 
             return;
@@ -108,6 +108,9 @@ public function loadHtml($html)
      * @param string $filePath
      *
      * @return Document
+     *
+     * @throws InvalidArgumentException
+     * @throws RuntimeException
      */
     public function loadHtmlFile($filePath)
     {
@@ -162,15 +165,15 @@ public function find($selector, $idx = null)
             $elements[] = new Element($node);
         }
 
-        if (is_null($idx)) {
+        if (null === $idx) {
             return $elements;
-        } else {
-            if ($idx < 0) {
-                $idx = count($elements) + $idx;
-            }
         }
 
-        return (isset($elements[$idx])) ? $elements[$idx] : null;
+        if ($idx < 0) {
+            $idx = count($elements) + $idx;
+        }
+
+        return isset($elements[$idx]) ? $elements[$idx] : null;
     }
 
     /**
@@ -181,7 +184,7 @@ public function find($selector, $idx = null)
     public function html()
     {
         if ($this::$callback !== null) {
-            call_user_func_array($this::$callback, [$this]);
+            call_user_func($this::$callback, $this);
         }
 
         return trim($this->document->saveHTML($this->document->documentElement));
@@ -284,6 +287,8 @@ public function __invoke($selector, $idx = null)
      * @param $arguments
      *
      * @return bool|mixed
+     *
+     * @throws BadMethodCallException
      */
     public function __call($name, $arguments)
     {
@@ -298,16 +303,18 @@ public function __call($name, $arguments)
      * @param $arguments
      *
      * @return bool|Document
+     *
+     * @throws BadMethodCallException
      */
     public static function __callStatic($name, $arguments)
     {
-        if ($name == 'str_get_html') {
+        if ($name === 'str_get_html') {
             $document = new Document();
 
             return $document->loadHtml($arguments[0]);
         }
 
-        if ($name == 'file_get_html') {
+        if ($name === 'file_get_html') {
             $document = new Document();
 
             return $document->loadHtmlFile($arguments[0]);

lib/Element.php

@@ -10,6 +10,7 @@
 
 /**
  * Class Element
+ *
  * @package FastSimpleHTMLDom
  * @property string      outertext Get dom node's outer html
  * @property string      innertext Get dom node's inner html
@@ -80,8 +81,8 @@ protected function replaceNode($string)
 
         $newDocument = new Document($string);
 
-        if ($newDocument->outertext != $string) {
-            throw new RuntimeException("Not valid HTML fragment");
+        if ($newDocument->outertext !== $string) {
+            throw new RuntimeException('Not valid HTML fragment');
         }
 
         $newNode = $this->node->ownerDocument->importNode($newDocument->getDocument()->documentElement, true);
@@ -106,8 +107,8 @@ protected function replaceChild($string)
         if (!empty($string)) {
             $newDocument = new Document($string);
 
-            if ($newDocument->outertext != $string) {
-                throw new RuntimeException("Not valid HTML fragment");
+            if ($newDocument->outertext !== $string) {
+                throw new RuntimeException('Not valid HTML fragment');
             }
         }
 
@@ -160,7 +161,7 @@ public function getDom()
      * Find list of nodes with a CSS selector
      *
      * @param string $selector
-     * @param int    $idx
+     * @param int $idx
      *
      * @return NodeList|Element|null
      */
@@ -477,7 +478,7 @@ public function __toString()
 
     /**
      * @param string $selector
-     * @param int    $idx
+     * @param int $idx
      *
      * @return Element|NodeList|null
      */
@@ -504,6 +505,7 @@ public function __call($name, $arguments)
 
     /**
      * Retrieve an external iterator
+     *
      * @link  http://php.net/manual/en/iteratoraggregate.getiterator.php
      * @return NodeList An instance of an object implementing <b>Iterator</b> or
      * <b>Traversable</b>

lib/NodeList.php

@@ -21,21 +21,21 @@ class NodeList extends \ArrayObject
      */
     public function find($selector, $idx = null)
     {
-        $elements = new NodeList();
+        $elements = new self();
         foreach ($this as $node) {
             foreach ($node->find($selector) as $res) {
                 $elements->append($res);
             }
         }
-        if (is_null($idx)) {
+        if (null === $idx) {
             return $elements;
-        } else {
-            if ($idx < 0) {
-                $idx = count($elements) + $idx;
-            }
         }
 
-        return (isset($elements[$idx])) ? $elements[$idx] : null;
+        if ($idx < 0) {
+            $idx = count($elements) + $idx;
+        }
+
+        return isset($elements[$idx]) ? $elements[$idx] : null;
     }
 
     /**

lib/SelectorConverter.php

@@ -4,10 +4,17 @@
 
 
 use Symfony\Component\CssSelector\CssSelectorConverter;
+
 class SelectorConverter
 {
     protected static $compiled = [];
 
+    /**
+     * @param $selector
+     *
+     * @return mixed|string
+     * @throws \RuntimeException
+     */
     public static function toXPath($selector)
     {
         if (isset(self::$compiled[$selector])){