Merge pull request #181 from gRegorLove/issue156

Fix parsing multiple mf1 roots on same element

Author: aaronpk

Mf2/Parser.php

@@ -1499,6 +1499,8 @@ public function backcompat(DOMElement $el, $context = '', $isParentMf2 = false)
 			$mf1Classes = array_intersect($classes, array_keys($this->classicRootMap));
 		}
 
+		$elHasMf2 = $this->hasRootMf2($el);
+
 		foreach ($mf1Classes as $classname) {
 			// special handling for specific properties
 			switch ( $classname )
@@ -1593,7 +1595,7 @@ public function backcompat(DOMElement $el, $context = '', $isParentMf2 = false)
 				}
 			}
 
-			if ( empty($context) && isset($this->classicRootMap[$classname]) && !$this->hasRootMf2($el) ) {
+			if ( empty($context) && isset($this->classicRootMap[$classname]) && !$elHasMf2 ) {
 				$this->addMfClasses($el, $this->classicRootMap[$classname]);
 			}
 		}

tests/Mf2/ClassicMicroformatsTest.php

@@ -971,5 +971,26 @@ public function testRelTagTrailingSlash() {
 		$this->assertContains('phpmf2', $output['items'][1]['properties']['category']);
 		$this->assertContains('mf2py', $output['items'][1]['properties']['category']);
 	}
+
+	/**
+	 * Upgrade multiple mf1 roots on the same element
+	 * @see https://github.com/indieweb/php-mf2/issues/156
+	 */
+	public function testBackcompatMultipleRoots() {
+		$input = '<article class="vevent hentry">
+    <span class="entry-title">h-entry name</span>
+    <span class="summary">h-event name</span>
+</article>';
+		$result = Mf2\parse($input);
+
+		$this->assertCount(2, $result['items'][0]['type']);
+		$this->assertContains('h-event', $result['items'][0]['type']);
+		$this->assertContains('h-entry', $result['items'][0]['type']);
+		$this->assertArrayHasKey('name', $result['items'][0]['properties']);
+		$this->assertCount(2, $result['items'][0]['properties']['name']);
+		$this->assertContains('h-event name', $result['items'][0]['properties']['name']);
+		$this->assertContains('h-entry name', $result['items'][0]['properties']['name']);
+	}
+
 }