Review code quality

Signed-off-by: Jérôme TAMARELLE <[email protected]>

composer.json

@@ -16,7 +16,10 @@
         }
     ],
     "require": {
-        "php": ">=7.2.0"
+        "php": ">=7.2.0",
+        "ext-dom": "*",
+        "ext-xsl": "*",
+        "ext-libxml": "*"
     },
     "suggest": {
         "symfony/css-selector": "Provides facilities is you prefers CSS to Xpath"

src/Welldom/Document.php

@@ -84,7 +84,7 @@ static public function create($source, $encoding = null, $version = null)
      * Factory.
      * Create a document from a DOMNode.
      *
-     * @param \Welldom\DOMNode $node
+     * @param \Welldom\NodeInterface $node
      * @param bool $deep When false, $node subtree is not imported.
      * @return \Welldom\Document
      */
@@ -242,10 +242,10 @@ public function validate()
      *
      * @return boolean
      */
-    public function schemaValidate($filename)
+    public function schemaValidate($filename, $options = null)
     {
         XmlErrorHandler::start();
-        $valid = parent::schemaValidate($filename);
+        $valid = parent::schemaValidate($filename, $options);
         $this->lastErrors = XmlErrorHandler::getErrors();
         XmlErrorHandler::clean();
 
@@ -257,10 +257,10 @@ public function schemaValidate($filename)
      *
      * @return boolean
      */
-    public function schemaValidateSource($source)
+    public function schemaValidateSource($source, $flags)
     {
         XmlErrorHandler::start();
-        $valid = parent::schemaValidateSource($source);
+        $valid = parent::schemaValidateSource($source, $flags);
         $this->lastErrors = XmlErrorHandler::getErrors();
         XmlErrorHandler::clean();
 
@@ -459,7 +459,7 @@ public function getNodeValue($expression, $default = false)
      *
      * @param string $expression
      * @param string $value
-     * @return \Welldom\Node
+     * @return \Welldom\NodeInterface
      */
     public function setNodeValue($expression, $value = null)
     {
@@ -530,7 +530,7 @@ public function getNodeInnerXml($expression, $default = false)
      * @param string $expression
      * @param string $xml
      * @param boolean $append
-     * @return \Welldom\Node
+     * @return \Welldom\NodeInterface
      */
     public function setNodeInnerXml($expression, $xml, $append = false)
     {

src/Welldom/DocumentFragment.php

@@ -11,8 +11,6 @@
 
 namespace Welldom;
 
-use Welldom\Exception\XmlLoadingException;
-
 /**
  * DOM Document Fragment extension.
  *

src/Welldom/Xpath.php

@@ -74,7 +74,7 @@ public function query($expression, \DOMNode $contextnode = null, $registerNodeNS
      * @param string $expression Xpath expression
      * @param \DOMNode $contextnode
      * @param bool $registerNodeNS
-     * @return \Welldom\Node The node found or FALSE if the number of results is not 1
+     * @return \Welldom\NodeInterface|false The node found or FALSE if the number of results is not 1
      * @throws \Welldom\Exception\InvalidXpathException
      */
     public function queryOne($expression, \DOMNode $contextnode = null, $registerNodeNS = true)

src/Welldom/XsltProcessor.php

@@ -64,7 +64,7 @@ public function transformToXml(\DOMNode $node, array $parameters = array())
     {
         $this->preTransform($parameters);
         $xml = $this->processor->transformToXml($node);
-        $this->postTransform($parameters);
+        $this->postTransform();
 
         if (false === $xml) {
             return false;
@@ -166,7 +166,7 @@ public function removeParameter(array $name)
     /**
      * Removes a list of parameters.
      *
-     * @param type $names List of names
+     * @param string[] $names List of names
      * @return \Welldom\XsltProcessor
      */
     public function removeParameters(array $names)