Merge pull request #42 from hhashimoto/fix-fully-qualified-name

fix parsing class bug
phpowermove · Mar 30, 2017 · 6f10f1e · 6f10f1e
2 parents 841e810 + 1458547
commit 6f10f1e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/src/parser/visitor/ClassParserVisitor.php b/src/parser/visitor/ClassParserVisitor.php
@@ -12,7 +12,11 @@ public function visitClass(Class_ $node) {
 		$struct = $this->getStruct();
 
 		if ($node->extends !== null) {
-			$struct->setParentClassName(implode('\\', $node->extends->parts));
+			if ($node->extends->getType() === 'Name_FullyQualified') {
+				$struct->setParentClassName('\\' . implode('\\', $node->extends->parts));
+			} else {
+				$struct->setParentClassName(implode('\\', $node->extends->parts));
+			}
 		}
 
 		foreach ($node->implements as $name) {

diff --git a/tests/fixtures/MyCollection2.php b/tests/fixtures/MyCollection2.php
@@ -0,0 +1,4 @@
+<?php
+class MyCollection2 extends \phootwork\collection\AbstractCollection implements \phootwork\collection\Collection {
+
+}
diff --git a/tests/parser/ClassParserTest.php b/tests/parser/ClassParserTest.php
@@ -74,6 +74,13 @@ public function testMyCollection() {
 		$this->assertEquals('phootwork\collection\AbstractCollection', $class->getParentClassName());
 		$this->assertTrue($class->hasInterface('phootwork\collection\Collection'));
 	}
+
+	public function testMyCollection2() {
+		$class = PhpClass::fromFile(__DIR__ . '/../fixtures/MyCollection2.php');
+
+		$this->assertEquals('\phootwork\collection\AbstractCollection', $class->getParentClassName());
+		$this->assertTrue($class->hasInterface('\phootwork\collection\Collection'));
+	}
 
 	public function testTypeClass() {
 		$class = PhpClass::fromFile(__DIR__ . '/../fixtures/TypeClass.php');