Skip incomplete forms in nav header

clue · clue · commit c85a4bfc7939 · 2015-04-10T10:24:53.000+02:00
diff --git a/src/Io/Loader.php b/src/Io/Loader.php
@@ -30,11 +30,11 @@ public function loadXmlString($html)
         // - help link in footer not terminated
         // - selected branch/tag in CVS "sticky tag" dropdown has not attribute value
         // - self closing elements with no trailing slash
-        // - excessive form close tags
+        // - remove navheader because of its often incomplete form tags
         $html = str_replace('Help</strong></td>', 'Help</a></strong></td>', $html);
         $html = str_replace('selected>', 'selected="selected">', $html);
         $html = preg_replace('#<((?:input|br|hr|img)[^\/\>]*)>#', '<$1 />', $html);
-        $html = preg_replace('#(</table>\s*)</form>\s*(</div>)#', '$1$2', $html);
+        $html = preg_replace('#\<div class\=\"vc_navheader\"\>.*?\<\/div\>#s','', $html);
 
         // replace named HTML entities with their UTF-8 value
         $html = str_replace(array_values($this->entities), array_keys($this->entities), $html);
diff --git a/tests/Io/LoaderTest.php b/tests/Io/LoaderTest.php
@@ -60,4 +60,13 @@ public function testLoadInvalidMarkupSelectedAttributeNoValue()
 
         $this->assertEquals('selected', (string)$xml['selected']);
     }
+
+    public function testLoadRemovedNavHeader()
+    {
+        $str = '<body><div class="vc_navheader"> invalid <asd> </okay> </div> </body>';
+        $xml = $this->loader->loadXmlString($str);
+
+        $this->assertFalse(isset($xml->div));
+        $this->assertEquals(0, count($xml));
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -60,4 +60,13 @@ public function testLoadInvalidMarkupSelectedAttributeNoValue()`
`60`	`60`
`61`	`61`	`$this->assertEquals('selected', (string)$xml['selected']);`
`62`	`62`	`}`
	`63`	`+`
	`64`	`+ public function testLoadRemovedNavHeader()`
	`65`	`+ {`
	`66`	`+ $str = '<body><div class="vc_navheader"> invalid <asd> </okay> </div> </body>';`
	`67`	`+ $xml = $this->loader->loadXmlString($str);`
	`68`	`+`
	`69`	`+ $this->assertFalse(isset($xml->div));`
	`70`	`+ $this->assertEquals(0, count($xml));`
	`71`	`+ }`
`63`	`72`	`}`