Merge pull request paquettg#75 from parisholley/whitespace

paquettg · web-flow · commit 48fe89fee331 · 2017-07-23T19:15:56.000-04:00
support html tags with extra whitespace
diff --git a/src/PHPHtmlParser/Dom.php b/src/PHPHtmlParser/Dom.php
@@ -516,8 +516,8 @@ protected function parseTag()
             }
 
             if (empty($name)) {
-                $this->content->fastForward(1);
-                continue;
+				$this->content->skipByToken('blank');
+				continue;
             }
 
             $this->content->skipByToken('blank');
diff --git a/tests/DomTest.php b/tests/DomTest.php
@@ -172,6 +172,15 @@ public function testLoadUtf8()
         $this->assertEquals('Dzień', $dom->find('p', 0)->text);
     }
 
+	public function testLoadFileWhitespace()
+	{
+		$dom = new Dom;
+		$dom->setOptions(['cleanupInput' => false]);
+		$dom->loadFromFile('tests/files/whitespace.html');
+		$this->assertEquals(1, count($dom->find('.class')));
+		$this->assertEquals("<span><span class=\"class\"></span></span>", (string)$dom);
+	}
+
     public function testLoadFileBig()
     {
         $dom = new Dom;
diff --git a/tests/files/whitespace.html b/tests/files/whitespace.html
@@ -0,0 +1 @@
+<span><span class="class" ></span></span>

Original file line number	Diff line number	Diff line change
`@@ -516,8 +516,8 @@ protected function parseTag()`
`516`	`516`	`}`
`517`	`517`
`518`	`518`	`if (empty($name)) {`
`519`		`- $this->content->fastForward(1);`
`520`		`- continue;`
	`519`	`+ $this->content->skipByToken('blank');`
	`520`	`+ continue;`
`521`	`521`	`}`
`522`	`522`
`523`	`523`	`$this->content->skipByToken('blank');`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+<span><span class="class" ></span></span>`