Fix problem with Document class and html() function (#19)

* 🩹 Remove XML declaration and comments from HTML output in Document class.

* 🩹 Improve comment removal in Document class by using a non-greedy regex match.

* 🎨 Remove comments

* 🩹 Refactor Document class constructor to simplify XML handling and remove unnecessary methods.

* 🩹 Remove unused Str import from Document class

---------

Co-authored-by: Brandon <[email protected]>
Co-authored-by: QWp6t <[email protected]>

Author: 3 people

src/Document.php

@@ -5,7 +5,6 @@
 use DOMDocument;
 use DOMNodeList;
 use DOMXPath;
-use Illuminate\Support\Str;
 
 class Document
 {
@@ -14,26 +13,17 @@ class Document
      */
     protected DOMDocument $document;
 
-    /**
-     * The XML encoding tag.
-     */
-    protected string $encoding = '<?xml encoding="UTF-8">';
-
     /**
      * Initialize the Document instance.
      */
     public function __construct(string $html)
     {
-        $this->document = new DOMDocument;
-
-        $this->suppress();
+        $this->document = new DOMDocument(encoding: 'UTF-8');
 
         $this->document->loadHTML(
-            Str::start($html, $this->encoding),
-            \LIBXML_HTML_NOIMPLIED | \LIBXML_HTML_NODEFDTD
+            $html,
+            LIBXML_HTML_NOIMPLIED | LIBXML_HTML_NODEFDTD | LIBXML_NOXMLDECL | LIBXML_NOWARNING | LIBXML_NOERROR
         );
-
-        $this->clear();
     }
 
     /**
@@ -69,27 +59,7 @@ public function xpath(string $expression): DOMNodeList
      */
     public function html(): string
     {
-        return trim(substr($this->document->saveHTML(), 23));
-    }
-
-    /**
-     * Suppress the XML errors.
-     */
-    protected function suppress(): self
-    {
-        libxml_use_internal_errors(true);
-
-        return $this;
-    }
-
-    /**
-     * Clear the XML errors.
-     */
-    protected function clear(): self
-    {
-        libxml_clear_errors();
-
-        return $this;
+        return trim($this->document->saveHTML());
     }
 
     /**