Merge pull request #104 from marcosh/anonymous-alter-parts

Anonymous alter parts

Author: jaapio

src/phpDocumentor/Reflection/NodeVisitor/ElementNameResolver.php

@@ -82,11 +82,12 @@ public function enterNode(Node $node)
             case Class_::class:
             case Trait_::class:
             case Interface_::class:
+                $this->parts->push((string)$node->name);
+
                 if (is_null($node->name)) {
                     return NodeTraverser::DONT_TRAVERSE_CHILDREN;
                 }
 
-                $this->parts->push((string)$node->name);
                 $node->fqsen = new Fqsen($this->buildName());
                 break;
             case Function_::class:

tests/unit/phpDocumentor/Reflection/NodeVisitor/ElementNameResolverTest.php

@@ -14,6 +14,7 @@
 namespace phpDocumentor\Reflection\NodeVisitor;
 
 use PhpParser\Node\Const_;
+use PhpParser\Node\Expr\New_;
 use PhpParser\Node\Name;
 use PhpParser\Node\Scalar\String_;
 use PhpParser\Node\Stmt\Class_;
@@ -77,6 +78,27 @@ public function testDoesNotEnterAnonymousClass()
         );
     }
 
+    /**
+     * @link https://github.com/phpDocumentor/Reflection/issues/103
+     * @covers ::enterNode
+     * @covers ::leaveNode
+     */
+    public function testAnonymousClassDoesNotPopParts()
+    {
+        $anonymousClass = new Class_(null);
+
+        $new = new New_($anonymousClass);
+
+        $namespace = new Namespace_(new Name('ANamespace'), $new);
+
+        $this->fixture->enterNode($namespace);
+        $this->fixture->enterNode($new);
+        $this->fixture->enterNode($anonymousClass);
+        $this->fixture->leaveNode($anonymousClass);
+        $this->fixture->leaveNode($new);
+        $this->fixture->leaveNode($namespace);
+    }
+
     /**
      * @covers ::enterNode
      */