Normalize spaces in union types in comments ("A | B" -> "A|B")

src/DiagnosticsProvider.php

@@ -9,14 +9,14 @@
 use Microsoft\PhpParser\Node;
 
 class DiagnosticsProvider {
-    /**
-     * Traverses AST to generate diagnostics.
-     * @param \Microsoft\PhpParser\Node $node
-     * @return \Generator | Diagnostic[]
-     */
 
     private static $tokenKindToText;
 
+    /**
+     * Returns the diagnostic for $node, or null.
+     * @param \Microsoft\PhpParser\Node $node
+     * @return Diagnostic|null
+     */
     public static function checkDiagnostics($node) {
         if (!isset(self::$tokenKindToText)) {
             self::$tokenKindToText = \array_flip(\array_merge(
@@ -88,6 +88,11 @@ public static function checkDiagnostics($node) {
         return null;
     }
 
+    /**
+     * Traverses AST to generate diagnostics.
+     * @param \Microsoft\PhpParser\Node $n
+     * @return Diagnostic[]
+     */
     public static function getDiagnostics(Node $n) : array {
         $diagnostics = [];
 

src/Node.php

@@ -16,7 +16,7 @@ abstract class Node implements \JsonSerializable {
     /** @var array[] Map from node class to array of child keys */
     private static $childNames = [];
 
-    /** @var Node | null */
+    /** @var Node|null */
     public $parent;
 
     public function getNodeKindName() : string {
@@ -187,7 +187,7 @@ public function getDescendantNodes(callable $shouldDescendIntoChildrenFn = null)
     /**
      * Gets generator containing all descendant Tokens.
      * @param callable|null $shouldDescendIntoChildrenFn
-     * @return \Generator | Token[]
+     * @return \Generator|Token[]
      */
     public function getDescendantTokens(callable $shouldDescendIntoChildrenFn = null) {
         foreach ($this->getChildNodesAndTokens() as $child) {
@@ -205,7 +205,7 @@ public function getDescendantTokens(callable $shouldDescendIntoChildrenFn = null
      * Gets generator containing all child Nodes and Tokens (direct descendants).
      * Does not return null elements.
      *
-     * @return \Generator | Token[] | Node[]
+     * @return \Generator|Token[]|Node[]
      */
     public function getChildNodesAndTokens() : \Generator {
         foreach ($this::CHILD_NAMES as $name) {
@@ -227,7 +227,7 @@ public function getChildNodesAndTokens() : \Generator {
 
     /**
      * Gets generator containing all child Nodes (direct descendants)
-     * @return \Generator | Node[]
+     * @return \Generator|Node[]
      */
     public function getChildNodes() : \Generator {
         foreach ($this::CHILD_NAMES as $name) {
@@ -425,7 +425,7 @@ private function containsPosition(int $pos): bool {
      * Returns last doc comment in leading comment / whitespace trivia,
      * and returns null if there is no preceding doc comment.
      *
-     * @return string | null
+     * @return string|null
      */
     public function getDocCommentText() {
         $leadingTriviaText = $this->getLeadingCommentAndWhitespaceText();
@@ -446,7 +446,7 @@ public function __toString() {
     }
 
     /**
-     * @return array | ResolvedName[][]
+     * @return array|ResolvedName[][]
      * @throws \Exception
      */
     public function getImportTablesForCurrentScope() {
@@ -547,7 +547,7 @@ public function getImportTablesForCurrentScope() {
     /**
      * Gets corresponding NamespaceDefinition for Node. Returns null if in global namespace.
      *
-     * @return NamespaceDefinition | null
+     * @return NamespaceDefinition|null
      */
     public function getNamespaceDefinition() {
         $namespaceDefinition = $this instanceof NamespaceDefinition

src/Node/ArrayElement.php

@@ -11,13 +11,13 @@
 
 class ArrayElement extends Node {
 
-    /** @var Expression | null */
+    /** @var Expression|null */
     public $elementKey;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $arrowToken;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $byRef;
 
     /** @var Expression */

src/Node/ClassInterfaceClause.php

@@ -13,7 +13,7 @@ class ClassInterfaceClause extends Node {
     /** @var Token */
     public $implementsKeyword;
 
-    /** @var DelimitedList\QualifiedNameList | null */
+    /** @var DelimitedList\QualifiedNameList|null */
     public $interfaceNameList;
 
     const CHILD_NAMES = [

src/Node/ElseClauseNode.php

@@ -14,7 +14,7 @@ class ElseClauseNode extends Node {
     public $elseKeyword;
     /** @var Token */
     public $colon;
-    /** @var StatementNode | StatementNode[] */
+    /** @var StatementNode|StatementNode[] */
     public $statements;
 
     const CHILD_NAMES = [

src/Node/ElseIfClauseNode.php

@@ -18,9 +18,9 @@ class ElseIfClauseNode extends Node {
     public $expression;
     /** @var Token */
     public $closeParen;
-    /** @var Token | null */
+    /** @var Token|null */
     public $colon;
-    /** @var StatementNode | StatementNode[] */
+    /** @var StatementNode|StatementNode[] */
     public $statements;
 
     const CHILD_NAMES = [

src/Node/Expression/AnonymousFunctionCreationExpression.php

@@ -15,7 +15,7 @@
 use Microsoft\PhpParser\Token;
 
 class AnonymousFunctionCreationExpression extends Expression implements FunctionLike {
-    /** @var Token | null */
+    /** @var Token|null */
     public $staticModifier;
 
     use FunctionHeader, FunctionUseClause, FunctionReturnType, FunctionBody;

src/Node/Expression/ArgumentExpression.php

@@ -10,10 +10,10 @@
 use Microsoft\PhpParser\Token;
 
 class ArgumentExpression extends Expression {
-    /** @var Token | null */
+    /** @var Token|null */
     public $byRefToken; // TODO removed in newer versions of PHP. Also only accept variable, not expression if byRef
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $dotDotDotToken;
 
     /** @var Expression */

src/Node/Expression/ArrayCreationExpression.php

@@ -12,7 +12,7 @@
 
 class ArrayCreationExpression extends Expression {
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $arrayKeyword;
 
     /** @var Token */

src/Node/Expression/CallExpression.php

@@ -17,7 +17,7 @@ class CallExpression extends Expression {
     /** @var Token */
     public $openParen;
 
-    /** @var DelimitedList\ArgumentExpressionList | null */
+    /** @var DelimitedList\ArgumentExpressionList|null */
     public $argumentExpressionList;
 
     /** @var Token */

src/Node/Expression/ExitIntrinsicExpression.php

@@ -14,13 +14,13 @@ class ExitIntrinsicExpression extends Expression {
     /** @var Token */
     public $exitOrDieKeyword;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $openParen;
 
-    /** @var Expression | null */
+    /** @var Expression|null */
     public $expression;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $closeParen;
 
     const CHILD_NAMES = [

src/Node/Expression/ObjectCreationExpression.php

@@ -19,25 +19,25 @@ class ObjectCreationExpression extends Expression {
     /** @var Token */
     public $newKeword;
 
-    /** @var QualifiedName | Variable | Token */
+    /** @var QualifiedName|Variable|Token */
     public $classTypeDesignator;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $openParen;
 
-    /** @var DelimitedList\ArgumentExpressionList | null  */
+    /** @var DelimitedList\ArgumentExpressionList|null  */
     public $argumentExpressionList;
 
-    /** @var Token | null */
+    /** @var Token|null */
     public $closeParen;
 
-    /** @var ClassBaseClause | null */
+    /** @var ClassBaseClause|null */
     public $classBaseClause;
 
-    /** @var ClassInterfaceClause | null */
+    /** @var ClassInterfaceClause|null */
     public $classInterfaceClause;
 
-    /** @var ClassMembersNode | null */
+    /** @var ClassMembersNode|null */
     public $classMembers;
 
     const CHILD_NAMES = [

src/Node/Expression/ScopedPropertyAccessExpression.php

@@ -12,13 +12,13 @@
 
 class ScopedPropertyAccessExpression extends Expression {
 
-    /** @var Expression | QualifiedName | Token */
+    /** @var Expression|QualifiedName|Token */
     public $scopeResolutionQualifier;
 
     /** @var Token */
     public $doubleColon;
 
-    /** @var Token | Variable */
+    /** @var Token|Variable */
     public $memberName;
 
     const CHILD_NAMES = [

src/Node/Expression/UnaryExpression.php

@@ -9,7 +9,7 @@
 use Microsoft\PhpParser\Node\Expression;
 
 class UnaryExpression extends Expression {
-    /** @var UnaryExpression | Variable */
+    /** @var UnaryExpression|Variable */
     public $operand;
 
     const CHILD_NAMES = [

src/Node/Expression/Variable.php

@@ -13,7 +13,7 @@ class Variable extends Expression {
     /** @var Token */
     public $dollar;
 
-    /** @var Token | Variable | BracedExpression */
+    /** @var Token|Variable|BracedExpression */
     public $name;
 
     const CHILD_NAMES = [

src/Node/ForeachValue.php

@@ -10,7 +10,7 @@
 use Microsoft\PhpParser\Token;
 
 class ForeachValue extends Node {
-    /** @var Token | null */
+    /** @var Token|null */
     public $ampersand;
     /** @var Expression */
     public $expression;

src/Node/FunctionBody.php

@@ -10,6 +10,6 @@
 use Microsoft\PhpParser\Token;
 
 trait FunctionBody {
-    /** @var CompoundStatementNode | Token */
+    /** @var CompoundStatementNode|Token */
     public $compoundStatementOrSemicolon;
 }

src/Node/FunctionHeader.php

@@ -13,7 +13,7 @@ trait FunctionHeader {
     public $functionKeyword;
     /** @var Token */
     public $byRefToken;
-    /** @var null | Token */
+    /** @var null|Token */
     public $name;
     /** @var Token */
     public $openParen;

src/Node/FunctionReturnType.php

@@ -13,6 +13,6 @@ trait FunctionReturnType {
     public $colonToken;
     /** @var Token|null */
     public $questionToken;
-    /** @var Token | QualifiedName */
+    /** @var Token|QualifiedName */
     public $returnType;
 }

src/Node/FunctionUseClause.php

@@ -7,6 +7,6 @@
 namespace Microsoft\PhpParser\Node;
 
 trait FunctionUseClause {
-    /** @var AnonymousFunctionUseClause | null */
+    /** @var AnonymousFunctionUseClause|null */
     public $anonymousFunctionUseClause;
 }

src/Node/NamespaceUseClause.php

@@ -15,11 +15,11 @@ class NamespaceUseClause extends Node {
     public $namespaceName;
     /** @var NamespaceAliasingClause */
     public $namespaceAliasingClause;
-    /** @var Token | null */
+    /** @var Token|null */
     public $openBrace;
-    /** @var DelimitedList\NamespaceUseGroupClauseList | null */
+    /** @var DelimitedList\NamespaceUseGroupClauseList|null */
     public $groupClauses;
-    /** @var  Token | null */
+    /** @var Token|null */
     public $closeBrace;
 
     const CHILD_NAMES = [

src/Node/Parameter.php

@@ -12,17 +12,17 @@
 class Parameter extends Node {
     /** @var Token|null */
     public $questionToken;
-    /** @var QualifiedName | Token | null */
+    /** @var QualifiedName|Token|null */
     public $typeDeclaration;
-    /** @var Token | null */
+    /** @var Token|null */
     public $byRefToken;
-    /** @var Token | null */
+    /** @var Token|null */
     public $dotDotDotToken;
     /** @var Token */
     public $variableName;
-    /** @var Token | null */
+    /** @var Token|null */
     public $equalsToken;
-    /** @var null | Expression */
+    /** @var null|Expression */
     public $default;
 
     const CHILD_NAMES = [

src/Node/Statement/BreakOrContinueStatement.php

@@ -12,7 +12,7 @@
 class BreakOrContinueStatement extends StatementNode {
     /** @var Token */
     public $breakOrContinueKeyword;
-    /** @var Token | null */
+    /** @var Token|null */
     public $breakoutLevel;
     /** @var Token */
     public $semicolon;

src/Node/Statement/CompoundStatementNode.php

@@ -14,7 +14,7 @@ class CompoundStatementNode extends StatementNode {
     /** @var Token */
     public $openBrace;
 
-    /** @var array | Node[] */
+    /** @var array|Node[] */
     public $statements;
 
     /** @var Token */

src/Node/Statement/DeclareStatement.php

@@ -19,13 +19,13 @@ class DeclareStatement extends StatementNode {
     public $declareDirective;
     /** @var Token */
     public $closeParen;
-    /** @var Token | null */
+    /** @var Token|null */
     public $colon;
-    /** @var StatementNode | StatementNode[] */
+    /** @var StatementNode|StatementNode[] */
     public $statements;
-    /** @var Token | null */
+    /** @var Token|null */
     public $enddeclareKeyword;
-    /** @var Token | null */
+    /** @var Token|null */
     public $semicolon;
 
     const CHILD_NAMES = [

src/Node/Statement/DoStatement.php

@@ -23,7 +23,7 @@ class DoStatement extends StatementNode {
     public $expression;
     /** @var Token */
     public $closeParen;
-    /** @var Token | null */
+    /** @var Token|null */
     public $semicolon;
 
     const CHILD_NAMES = [