Fix parsing of invalid root class names

Fixes microformats#127
gRegorLove · May 29, 2017 · f139131 · f139131
1 parent e7c0cac
commit f139131
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/Mf2/Parser.php b/Mf2/Parser.php
@@ -846,6 +846,10 @@ public function parseH(\DOMElement $e, $is_backcompat = false) {
 		// Get current µf name
 		$mfTypes = mfNamesFromElement($e, 'h-');
 
+		if (!$mfTypes) {
+			return null;
+		}
+
 		// Initalise var to store the representation in
 		$return = array();
 		$children = array();

diff --git a/tests/Mf2/ParserTest.php b/tests/Mf2/ParserTest.php
@@ -493,6 +493,31 @@ public function testCamelCaseClassNames() {
 	}
 
 
+	/**
+	 * @see https://github.com/indieweb/php-mf2/issues/127
+	 */
+	public function testCamelCaseRootClassNames() {
+		$input = '<div class="h-Entry"> <a href="https://example.com" class="u-url">content</a> </div>';
+		$output = Mf2\parse($input);
+
+		$this->assertEmpty($output['items']);
+	}
+
+
+	/**
+	 * @see https://github.com/indieweb/php-mf2/issues/127
+	 */
+	public function testMisTypedRootClassNames() {
+		$input = '<div class="h-entry>"> <a href="https://example.com" class="u-url">content</a></div>
+		<div class="h-entry1>"> <a href="https://example.com" class="u-url">content</a></div>
+		<div class="h-👍"> <a href="https://example.com" class="u-url">content</a></div>
+		<div class="h-hentry_"> <a href="https://example.com" class="u-url">content</a></div>
+		<div class="h-"> <a href="https://example.com" class="u-url">content</a></div>';
+		$output = Mf2\parse($input);
+
+		$this->assertEmpty($output['items']);
+	}
+
 	public function testClassNameNumbers() {
 		$input = '<div class="h-entry"> <div class="u-column1"> <p class="p-title">Test</p> </div> </div>';
 		$output = Mf2\parse($input);