Merge pull request #36 from yellowtree/libxmlOptions

vanderlee · web-flow · commit 1e12a7c41f38 · 2019-12-10T21:48:08.000+01:00
Possibility to omit Doctype for hyphenateHtml()
diff --git a/src/Syllable.php b/src/Syllable.php
@@ -62,6 +62,11 @@ class Syllable
     private $excludes = array();
     private $includes = array();
 
+    /**
+     * @var int
+     */
+    private $libxmlOptions = 0;
+
     /**
      * Create a new Syllable class, with defaults
      *
@@ -195,6 +200,16 @@ public function getMinWordLength()
         return $this->min_word_length;
     }
 
+    /**
+     * Options to use for HTML parsing by libxml
+     * @param integer $libxmlOptions
+     * @see https://www.php.net/manual/de/libxml.constants.php 
+     */
+    public function setLibxmlOptions($libxmlOptions)
+    {
+        $this->libxmlOptions = $libxmlOptions;
+    }
+
     private static function initEncoding()
     {
         if (self::$encoding) {
@@ -436,7 +451,7 @@ public function hyphenateHtml($html)
     {
         $dom = new \DOMDocument();
         $dom->resolveExternals = true;
-        $dom->loadHTML($html);
+        $dom->loadHTML($html, $this->libxmlOptions);
 
         // filter excludes
         $xpath = new \DOMXPath($dom);
diff --git a/tests/SyllableTest.php b/tests/SyllableTest.php
@@ -222,6 +222,10 @@ public function testHyphenateHtml()
         $this->assertEquals('<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">'
             . "\n" . '<html><body><p>Ridicu-lous-ly <b class="unsplittable">com-pli-cat-ed</b> meta-text</p></body></html>'
             . "\n", $this->object->hyphenateHtml('Ridiculously <b class="unsplittable">complicated</b> metatext'));
+
+        $this->object->setLibxmlOptions(LIBXML_HTML_NOIMPLIED | LIBXML_HTML_NODEFDTD);
+        $this->assertEquals('<p>Ridicu-lous-ly <b class="unsplittable">com-pli-cat-ed</b> meta-text</p>'
+            . "\n", $this->object->hyphenateHtml('Ridiculously <b class="unsplittable">complicated</b> metatext'));
     }
 
     public function testCaseInsensitivity()
