Merge pull request #187 from aaronpk/master

fix for parsing empty e- elements

Author: aaronpk

Mf2/Parser.php

@@ -853,7 +853,9 @@ public function parseE(\DOMElement $e) {
 		}
 		$html = $e->ownerDocument->saveHtml($innerNodes);
 		// Put the nodes back in place.
-		$e->appendChild($innerNodes);
+		if($innerNodes->hasChildNodes()) {
+			$e->appendChild($innerNodes);
+		}
 
 		$return = array(
 			'html' => unicodeTrim($html),

tests/Mf2/ParserTest.php

@@ -87,6 +87,17 @@ public function testParseE() {
 		$this->assertEquals('Here is a load of embedded markup', $output['items'][0]['properties']['content'][0]['value']);
 	}
 
+  public function testParseEmptyE() {
+    $input = '<div class="h-entry"><div class="p-name">Name</div> <div class="e-content"></div></div>';
+    //$parser = new Parser($input);
+    $output = Mf2\parse($input);
+
+    $this->assertEquals('Name', $output['items'][0]['properties']['name'][0]);
+    $this->assertArrayHasKey('content', $output['items'][0]['properties']);
+    $this->assertEquals('', $output['items'][0]['properties']['content'][0]['html']);
+    $this->assertEquals('', $output['items'][0]['properties']['content'][0]['value']);
+  }
+
 	public function testParseEResolvesRelativeLinks() {
 		$input = '<div class="h-entry"><p class="e-content">Blah blah <a href="/a-url">thing</a>. <object data="/object"></object> <img src="/img" /></p></div>';
 		$parser = new Parser($input, 'http://example.com');