Merge pull request microsoft#176 from TysonAndre/Node-phpdoc-phan-nits

Make phpdoc of Node and its subclasses more accurate

Author: roblourens

src/Node.php

@@ -78,7 +78,7 @@ public function getParent() {
      * Gets first ancestor that is an instance of one of the provided classes.
      * Returns null if there is no match.
      *
-     * @param array ...$classNames
+     * @param string ...$classNames
      * @return Node|null
      */
     public function getFirstAncestor(...$classNames) {
@@ -139,7 +139,7 @@ public function getFirstDescendantNode(...$classNames) {
 
     /**
      * Gets root of the syntax tree (returns self if has no parents)
-     * @return Node
+     * @return SourceFileNode (expect root to be SourceFileNode unless the tree was manipulated)
      */
     public function getRoot() : Node {
         $node = $this;
@@ -450,7 +450,7 @@ public function __toString() {
      * @throws \Exception
      */
     public function getImportTablesForCurrentScope() {
-        $namespaceDefinition = $namespaceDefinition ?? $this->getNamespaceDefinition();
+        $namespaceDefinition = $this->getNamespaceDefinition();
 
         // Use declarations can exist in either the global scope, or inside namespace declarations.
         // http://php.net/manual/en/language.namespaces.importing.php#language.namespaces.importing.scope

src/Node/Expression/EchoExpression.php

@@ -7,14 +7,15 @@
 namespace Microsoft\PhpParser\Node\Expression;
 
 use Microsoft\PhpParser\Node\Expression;
+use Microsoft\PhpParser\Node\DelimitedList\ExpressionList;
 use Microsoft\PhpParser\Token;
 
 class EchoExpression extends Expression {
 
     /** @var Token */
     public $echoKeyword;
 
-    /** @var Expression[] */
+    /** @var ExpressionList */
     public $expressions;
 
     const CHILD_NAMES = [

src/Node/NumericLiteral.php

@@ -9,7 +9,7 @@
 use Microsoft\PhpParser\Token;
 
 class NumericLiteral extends Expression {
-    /** @var Token[] */
+    /** @var Token */
     public $children;
 
     const CHILD_NAMES = [

src/Node/QualifiedName.php

@@ -157,7 +157,7 @@ public function getLastNamePart() {
     /**
      * @param ResolvedName[] $importTable
      * @param bool $isCaseSensitive
-     * @return null
+     * @return string|null
      */
     private function tryResolveFromImportTable($importTable, bool $isCaseSensitive = false) {
         $content = $this->getFileContents();

src/Node/ReservedWord.php

@@ -9,7 +9,7 @@
 use Microsoft\PhpParser\Token;
 
 class ReservedWord extends Expression {
-    /** @var Token[] */
+    /** @var Token */
     public $children;
 
     const CHILD_NAMES = [

src/Node/StringLiteral.php

@@ -13,7 +13,7 @@ class StringLiteral extends Expression {
     /** @var Token */
     public $startQuote;
 
-    /** @var Token[] | Node[] */
+    /** @var Token[]|Node[]|Token */
     public $children;
 
     /** @var Token */